Question

I have a function which returns an array of files in a folder recursive.

protected function getFiles($base) {

    $files = array();       
    if(!is_dir($base)) return $files;

    if (($handle = opendir($base)) != false) {
        while (false !== ($file = readdir($handle))) {
            if ($file == "." || $file == "..") continue;

            if(is_dir("$base/$file")) {                 
                $subfiles = $this->getFiles("$base/$file");
                $files = array_merge($files, $subfiles);                    
            } else {                    
                if(File::type($file,false) == "xml") 
                    $files[] = "$base/$file";
            }
        }
        closedir($handle);
    }

    return $files;
}

How can I modify this so that it always lists the root files first before any sub folders? At the moment by default the folders always come first.

Answer 1

Just create an array of sub-directories while you loop the directory pointer, and iterate the array of directories at the end:

protected function getFiles($base) {

    $files = $dirs = array();       
    if(!is_dir($base)) return $files;

    if (($handle = opendir($base)) != false) {

        while (false !== ($file = readdir($handle))) {
            if ($file == "." || $file == "..") continue;

            if(is_dir("$base/$file")) {    
                $dirs[] = "$base/$file";
            } else {                    
                if(File::type($file,false) == "xml") 
                    $files[] = "$base/$file";
            }
        }
        closedir($handle);

        foreach ($dirs as $dir) {
            $subfiles = $this->getFiles($dir);
            $files = array_merge($files, $subfiles);                    
        }

    }

    return $files;
}