php 独自のルールを基に配列の並び替えを行いたい。

Question

自作FW的なものを作っているのですが、ファイルの読み込み順に問題があり、配列の順番を変えたいのですが、これになかなか苦戦しています。

2018-06-25 00:40:22: array(5)
Array
(
    [0] => /core/classes/controller/base/baseAdminController.php
    [1] => /core/classes/controller/base/baseController.php
    [2] => /core/classes/model/baseModel.php
    [3] => /core/classes/object/baseObject.php
    [4] => /core/classes/repository/baseRepository.php
)

ファイルの命名ルールを決めていて、以下になります。

• 管理画面で使うcontrollerの親classのファイル名は、adminとbaseという文字列が入ってなければなりません。
• フロント画面で使うcontrollerの親classのファイル名は、frontとbaseという文字列が入ってなければなりません。
• 親classの親class(基底class)のファイル名は、baseという文字列が入ってなければなりません。

手動なら問題ないんですが、scandir関数で指定したディレクトリ配下のファイルを取得する関数を作ってファイル取ってきているので、上記の形式で先にbaseAdminController.phpがくるんですね。

baseAdminController.phpはbaseController.phpを継承しているのでforeachでincludeするとbaseControllerは存在しないってエラーがでるんですね。

その措置として、上記ファイル配列の順番を変更する必要があります。

wordpressなんですけど、is_adminで管理画面じゃなければbaseAdminController.phpじゃなくてbaseFrontController.phpを取得してきます。
人間がわかるように説明すると、
ファイル名に（（Admin か　Front） と　Base）を含んでいるファイルは、
ファイル名に（Admin か　Front）を含まずBaseを含んでいるファイルよりも先に存在してはいけない。
です。

単なるsortではなく、システム固有のルールがあるのでそんな関数はないので自作しないとってところなんですが、このロジックが思い浮かばずです。

条件はわかっているのですが、単に「いるファイル、いらないファイル」なら条件に応じてcontinueと$files[] = $crrentFileでいいのですが、並び替えなのでちょっと苦戦しています。詰めるところをどうするかです。

とりあえず書いていったらifififififとかなりそうで相談がてら投稿しました。
以下、作りかけです。

    /*
     * 親classを継承しているclassファイルが先に読み込まれないように配列の順番を変更する。
     *
     * @param    array   $fullPathFiles
     *
     * @return   array
     */
    public static function changingTheOrderOfCoreFiles($fullPathFiles)
    {
        if (!is_array($fullPathFiles)) {
            return false;
        }
        $getFiles  = array();
        $loopCount = 0;
        $previousIndexName = '';
        foreach ($fullPathFiles as $index => $fullPathFile) {

            if (!$loopCount) {
                continue;
            }

            // 前回のファイル名。
            $previousFileName = mb_strtolower(self::getFileName($fullPathFiles[$previousIndexName]));
            // 現在のファイル名。
            $currentFileName  = mb_strtolower(self::getFileName($fullPathFile));


            // 前回のファイル名に'Front or Admin' と 'Base'が含まれているか？
            if (
                (strpos($previousFileName, mb_strtolower(self::FRONT_IDENTIFIER)) !== false ||
                 strpos($previousFileName, mb_strtolower(self::ADMIN_IDENTIFIER)) !== false
                )
                && strpos($previousFileName, mb_strtolower(self::BASE_IDENTIFIER)) !== false
            ) {

                // 現在のファイル名に'Front or Admin' と 'Base'が含まれているか？
                if (
                    (strpos($currentFileName, mb_strtolower(self::FRONT_IDENTIFIER)) !== false ||
                     strpos($currentFileName, mb_strtolower(self::ADMIN_IDENTIFIER)) !== false
                    )
                    && strpos($currentFileName, mb_strtolower(self::BASE_IDENTIFIER)) !== false
                ) {



                }


            }

            ++$loopCount;
            $previousIndexName = $index;
        }
        return $getFiles;
    }

ちょっとだけ近づいてきたけどこれじゃ無限ループになってしまう。。。
プロパティーにflag的なものは持たせたくない。

    public static function changingTheOrderOfCoreFiles($fullPathFiles)
    {
        $getFiles = array();
        foreach ($fullPathFiles as $fullPathFile) {

            $fileName  = mb_strtolower(self::getFileName($fullPathFile));
            // ファイル名に'Front or Admin' が含まれておらず、 'Base'は含まれているか？
            if (
                (strpos($fileName, mb_strtolower(self::FRONT_IDENTIFIER)) === false ||
                 strpos($fileName, mb_strtolower(self::ADMIN_IDENTIFIER)) === false
                )
                && strpos($fileName, mb_strtolower(self::BASE_IDENTIFIER)) !== false
            ) {
                $getFiles[] = $fullPathFile;
            }
            // ファイル名に'Front or Admin' が含まれいて、 'Base'も含まれているか？
            if (
                (strpos($fileName, mb_strtolower(self::FRONT_IDENTIFIER)) !== false ||
                    strpos($fileName, mb_strtolower(self::ADMIN_IDENTIFIER)) !== false
                )
                && strpos($fileName, mb_strtolower(self::BASE_IDENTIFIER)) !== false
            ) {
                $getFiles = array_merge($getFiles, self::changingTheOrderOfCoreFiles(array($fullPathFile)));
            }
        }
        return $getFiles;
    }

Comments

[m.ts10806]

「自作FW的なもの」と「wordpress」はどのような関係性なのでしょうか。

[earnest_gay]

リリースまで乞うご期待！！！

[m.ts10806]

あいえ。申し訳ないけどそういう興味はないので。

Answer 1

条件だけ見ると「Admin か Front」を含むファイルを、配列の最後にもっていくだけで良い気がします。

Answer 2

ちょっとぶち壊しな提案をしちゃいますが、
もしクラスの依存関係云々で読み込みの順番を調整したいってことなら
単純にspl_autoload_registerを使用することを提案します

autoload_functionで管理画面であるか否かによって
読みこむファイルを切り替えれば、
フロントと管理画面で使うクラスを切り替えることも簡単でしょう

Answer 3

私もタイトルだけ見て、usortだなと思って↓みたいのを書いてみました。

<?php
$list = [
    "/core/classes/controller/base/baseAdminController.php",
    "/core/classes/controller/base/baseController.php",
    "/core/classes/model/baseModel.php",
    "/core/classes/object/baseObject.php",
    "/core/classes/repository/baseRepository.php",
];

usort($list, function (string $a, string $b) {
    $fnameA = strtolower(basename($a));
    $fnameB = strtolower(basename($b));
    $priorA = (strpos($fnameA, "base") !== false && (strpos($fnameA, "admin") !== false || strpos($fnameA, "front") !== false)) ? -1 : 0;
    $priorB = (strpos($fnameB, "base") !== false && (strpos($fnameB, "admin") !== false || strpos($fnameB, "front") !== false)) ? -1 : 0;
    if($priorA === $priorB) { return $fnameA - $fnameB; }
    return ($priorA > $priorB) ? -1 : 1;
});
var_export($list);
/*
array (
  0 => '/core/classes/model/baseModel.php',
  1 => '/core/classes/controller/base/baseController.php',
  2 => '/core/classes/object/baseObject.php',
  3 => '/core/classes/repository/baseRepository.php',
  4 => '/core/classes/controller/base/baseAdminController.php',
)
*/

無名関数のところを適宜かえていけば、まあ色々とやれるのではないかとは思います。

ただ、フレームワークを意図しているのだとしたら、ファイル名もそうですが、ディレクトリの階層とかで制御したほうがいいようにも思いました。

Comments

[earnest_gay]

質問に合わせたサンプルがあって非常に良い回答であるといえます。
ありがとございます。

Answer 4

リストからふたつを取り出して比較、前にあるか後ろにあるかを判別するだけで良い場合

function cmp($a, $b) {
	//条件
}
usort($fullPathFiles,"cmp");

PHP: usort - Manual

比較結果が等しくなる二つの要素があった場合、ソートした配列におけるそれらの並び順は不定となります。

とあるので、順番を極力維持したままソートしたい場合はforeach等を使う必要があります。

恐らくusortで事足りると思うので簡単な説明に留めますが、
for i=0 リストの0番目から最後まで
for j=i+1 リストのi+1番目から最後まで(i+1が範囲を超える時は実行しない)
// iよりもjの方を前に置きたい時は入れ替え
とすることで順番を維持したままソートは出来ます。

Comments

[earnest_gay]

以下のindex 0と１を入れ替えたい場合はどうなりますか？

[0] => /core/classes/controller/base/baseAdminController.php
[1] => /core/classes/controller/base/baseController.php
[2] => /core/classes/model/baseModel.php
[3] => /core/classes/object/baseObject.php
[4] => /core/classes/repository/baseRepository.php