0

Подскажите как можно улучшить этот код, а то мне кажется что с ним что-то не так.

foreach ($newModules['MODULES'] as $newModule) {
        foreach(self::getInstallModules() as $installModule) {
            if ($installModule['TITLE'] == $newModule['TITLE']) {
                if($newModule['INFO']['VERSION'] > $installModule['VERSION']){
                    $results[] = [
                        'NAME' => $newModule['NAME'],
                        'OLD_VERSION' => $installModule['VERSION'],
                        'NEW_VERSION' => $newModule['INFO']['VERSION'],
                        'DESCRIPTION' => $newModule['DESCRIPTION']
                    ];
                }
            }
        }
    }
2
  • Что-то не так скорее всего из-за $newModule['INFO']['VERSION'] > $installModule['VERSION'])
    – Peresada
    6 апр 2018 в 10:56
  • двойной цикл тут костылем кажется для поиска соответствий в массивах. title бы сделать ключами, пересечь массивы, и потом отфильтровать остатки. но по объему кода меньше не станет. так что смысла нет особого
    – teran
    6 апр 2018 в 11:15

1 ответ 1

0

большую часть вашего кода тут занимает поиск соответствий в двух массивах. Поскольку пхп имеет и встроенные функции для поиска схожений массивов, то можно воспользоваться ими.

Пример исходных данных:

$newModules = [
            ['title' => "module-2", 'version' => 1],
            ['title' => "module-3", 'version' => 2],
            ['title' => "module-4", 'version' => 3],
        ];

$oldModules = [
            ['title' => "module-1", 'version' => 1],
            ['title' => "module-2", 'version' => 1],
            ['title' => "module-3", 'version' => 1],
            ['title' => "module-4", 'version' => 1],
            ['title' => "module-5", 'version' => 1],
];

и используем array_uintersect не совсем по назначению:

$result = [];
$data = array_uintersect($newModules, $oldModules, function($n, $o) use (&$result){
                $c = strcmp($n['title'], $o['title']);

                if($c == 0 && $n['version'] > $o['version']){
                    $result[] = [
                        'title' => $o['title'],
                        'old_version' => $o['version'],
                        'new_version' => $n['version'],
                        'description' => null
                    ];                      
                }
                return $c;
            });

print_r($result);

но честно сказать, не знаю, лучше ли будет этот код, чем ваш. фактически просто спрятали двойной цикл за функцией.

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.