Стоит ли использовать двойные foreach?

Question

[Человек]

Добрый вечер, как бороться с двойными foreach? Цель достать file.php, но директории отличаются, поэтому приходилось использовать foreach. Ниже код описывает суть.

// path1/name1/path/file.php
// path2/name2/path/file.php

$paths = scandir('path');
foreach ($paths as $path){
    $name = scandir($path);
    foreach ($names as $name){
        ...
    }
}

Можно ли эту конструкцию упростить, улучшить и правильная ли она (новичок, учусь...)?

Comments

[Дмитрий]

Добрый вечер.
Посмотрите в сторону итераторов, рекурсивных функций.
Ну и как написал Sergey - подумайте над хранением файлов и путей к ним.

[alexalexes]

Два уровня каталогов ваш код будет обходить, если для решения какой-то практической задачи этого достаточно, то это достаточно.
А вот, если потребуется обойти все дерево вложенных каталогов, то нужна рекурсивная функция.
PS: Тут у вас закралась опечатка. Scandir извлекает перебираемый объект, а в foreach вы ее ставите не на месте перебираемого объекта (до as), а в качестве текущего элемента (после as).

....
$names = scandir($path);
    ^    
foreach ($names as $name){
             ^
....

[AUser0]

Ещё можно так:

foreach (scandir('path') as $path){
    foreach (scandir($path) as $name){
        ...
    }
}

[Lander]

$name = scandir($path);
Тут, видимо, переменная должна называться $names

Answer 1

[Sergey]

Правильно организовать хранение файлов, чтобы чтение их было однозначным

Comments

[Алишер Айдархан]

Согласен. Лучше упростить путь, чем код

[Человек]

Упростить не получится никак, хотя была идея... Сейчас объясню почему на примере плагинов:

Plagins/
AuthorName/PlaginName/ и тут путь к файлу
AuthorName/PlaginName/ и тут путь к файлу

Автор и имя плагина различные, и надо добраться до пути к файлу...