Добрый день!
Изучаю PHP, оцените пожалуйста код и укажите недостатки.

 spl_autoload_register ('autoload');
  
 function autoload($className)
    {   
 $files = Array();
$dirs = array(); 
$dirs[]= 'moduls/';
$dirs[]= 'php/';   
foreach ($dirs as $dir){
$dir = DOCUMENT_ROOT.'/system/includes/'.$dir;
$odir = opendir($dir);    
while($rdir =  readdir($odir)){ 
$path_parts = pathinfo($rdir); 

if($path_parts['extension'] == 'php'){
$files[ substr($rdir,0, strpos($rdir, '.'))] = $dir.$rdir;  
}else{
if (preg_match('/^[a-z0-9\-]+$/', $rdir) == true){
   $ferdir=$dir.$rdir.'/'.$rdir.'.php';
if(file_exists($ferdir)){
    $files[$rdir] = $ferdir;
 }
}     
}
unset($path_parts);
}
closedir($odir); 
}
unset($dirs);

 foreach($files as $key =>$file){
   if($className == $key){
  include_once($file);
 }
 }
    unset($files);
unset($dirs); 
    }