Код: Выделить всё
// Удалить все ресайзы
$rezised_images = glob($this->config->root_dir.$this->config->resized_images_dir.$file.".*x*.".$ext);
if(is_array($rezised_images)) {
foreach (glob($this->config->root_dir.$this->config->resized_images_dir.$file.".*x*.".$ext) as $f) {
@unlink($f);
}
}
Сразу видно целых два недостатка:
1. Дважды повторяется один и тот же вызов glob(...), хотя при первом вызове результат запомнен в переменной. Явно нерационально и лишняя нагрузка.
2. Если удаляется файл с базовым именем, например, a.jpg, то будут удаляться, наряду с a.100x200.jpg, также и файлы с названиями вида apple.100x200.jpg, относящиеся к ДРУГОМУ изображению. Беда, конечно, небольшая, так как файлы будут созданы автоматически заново. Но опять же получаем лишнюю нагрузку.
Два очевидных ляпа в таком простом действии - это много. Неаккуратность разработчиков налицо...