HettТут явно нужно проверку entity сначала делать, а потом уже opendir.По хорошему, ранее ещё проверка is_dir() должна быть.

CTYDEHTа зачем

Мда... Иногда проще купить сервер помощнее (и подороже), чем найти толкового программиста.

CTYDEHTНет чтобы ответить нормально, к чему это может привести, они клоунаду тут устроили.Вы и не пытались спрашивать "к чему это может привести", Вы лишь утверждали, что "вот тут усё работает, зачем что-то ещё". В каком-то смысле Вы правы. Когда у проекта околонулевая посещаемость, то можно не заморачиваться ни с контролем над ресурсами, ни с оптимизацией алгоритмов и времени выполнения. Оно, действительно, "сойдет и так", ибо, скорее всего, никакой разницы не заметите. Вот когда посещаемость проекта приличная, а ресурсы сервера не резиновые - тогда подобного рода вещи вполне могут показать себя не с лучшей стороны.

Проанализируйте, что будет делать скрипт, если в $entry придут последовательно значения ".", ".." и имя существующего файла. Тогда увидите, что скрипт трижды попытается заставить сервер "пошкрябать головками жесткого диска" и один раз заставит PHP создать в памяти две структуры данных (ресурсы . и ..), выработать исключение (передано имя файла) и затем этим всем добром _не_ воспользуется. Это вкратце. До закрытия ресурсов не дошли ещё.

Hett указал Вам, что проверки следует делать раньше, но Вы же утверждать стали что "это работает". Да, работает, никто же с эти утверждением и не спорит. Разница только в том, _как_ оно работает. Вероятно, Вам не интересны детали. Вы о деталях не спрашивали - Вам никто и не отвечал. Так чего Вы обижаетесь то?