PHPで特定の拡張子のファイル一覧を再帰的に取得するにはどうすればよいでしょうか？

Question

質問内容

タイトルの通りです

指定ディレクトリ直下のファイルしか取得できない

/var/www/php/直下のPHPファイルしか取得できません。
/var/www/php/conf/や/var/www/php/batch/detail/などの深い階層のPHPファイルが取得できません。

$php_path = /var/www/php;
$list = getFileList($php_path);
var_dump($list);exit;

function getFileList($dir) {
        $files = glob($dir.'/{*.php}',GLOB_BRACE);
	$list = array();
	foreach ($files as $file) {
		if (is_file($file)) {
			$list[] = $file;
		}
		if (is_dir($file)) {
			$list = array_merge($list, getFileList($file));
		}
	}
	return $list;
}

Answer 1

<?php

/**
 * @param string $dir
 * @param strint $ext
 * @return \CallbackFilterIterator|\SplFileInfo[]
 */
function getFilesOfExt(string $dir, string $ext): \CallbackFilterIterator
{
    // 再帰可能イテレータを作成し
    $iter = new \RecursiveDirectoryIterator($dir);

    // リーフノードだけに絞り込むようにラップ
    $iter = new \RecursiveIteratorIterator($iter);

    // $ext が拡張子にマッチするファイルだけに絞り込むようにラップ
    $filter = fn (\SplFileInfo $f) => !$f->isDir() && $f->getExtension($ext);
    return new \CallbackFilterIterator($iter, $filter);
}

// これで実行
foreach (getFilesOfExt('/var/www/php', 'php') as $file) {
    // $file は SplFileInfo オブジェクト
    var_dump($file->getFilename());
}

// 一気に配列化したい場合はこれで
$files = iterator_to_array(getFilesOfExt('/var/www/php', 'php'), false);

// 配列化+ファイル名だけ取り出す場合はこれで
$files = array_map(
    fn (\SplFileInfo $file) => $file->getFilename(),
    iterator_to_array(getFilesOfExt('/var/www/php', 'php'), false)
);

Comments

[mpyw]

PHP 7.4 想定で書いています
PHP 7.3 以下の場合は Arrow Functions は使わないでください

Answer 2

現在のものがうまく動かない理由は、以下の箇所にあります。

 $files = glob($dir.'/{*.php}',GLOB_BRACE);

拡張子.phpのみを取得しているので、ディレクトリが取得できない（/var/www/php/xxx.php/ みたいなディレクトリは例外）ことが原因です。
この問題を解消するためには、m.ts10806さんが指摘しているように「いきなり絞り込まずにまず全フォルダ、ファイルを取得してループしつつis_dir()で判定」するとよいと思います。

Answer 3

まず、あなたのそのコードを関数とします
その時、検索するパスを引数とする関数とします。
そのうえで、

• ディレクトリの一覧を取得
• 一覧をforeach
• ファイルであればリストに追加
• ディレクトリであれば、引数のパスにその名前を追加して自関数を再帰呼び出し
• foreach終了時点で全てのファイルの取得が完了

というシーケンスでどうでしょうか

Answer 4

いきなり絞り込まずにまず全フォルダ、ファイルを取得してループしつつis_dir()で判定してはどうでしょう。
フォルダなら自身を呼び出し、ファイルなら拡張子.phpかを判定する

Comments

[bibouroku]

以下の修正をすることで解決できました！

修正①
$files = glob($dir.'/{*.php}',GLOB_BRACE);
↓
$files = glob($dir.'/*');

修正②
if (is_file($file))の中に
if (strrchr($file, '.') === '.php') {
continue;
}

[m.ts10806]

個人的にはファイル名のみで持ってくるのはセキュリティ的に弱いかなとは思いますが（ファイル名についた拡張子は飾りという意味）それは次の段階ですね