| 2006-07-17 16:26:22 | |
| Работа: Новостной скрипт | |
Ниже плинтуса. По-другому оценить, извините, сложно. Это были амбиции, извините А теперь конструктив:1. Начнем с того, что в коде присутствуют нарушения синтаксиса: в некоторых блоках (например строки 31-32, 88... дальше не смотрел, но не исключаю, что могут быть) echo "..." не экранируются ковычки. 2. Программа написана при Register_globals = On. Сложно найти сервер, где это так. Хотя бы в .htaccess вынесли что-то вроде php_flag register_globals 1 3. Объявленный в .htaccess php_value error_reporting 7 переопределяется на нулевой уровень (отсутствие вывода сообщений об ошибках). Учитывая то, что в скрипте полно косяков с нарушением синтаксиса (см. п.1), то неподготовленный пользователь увидит просто белый экран. 4. Про грубейшее нарушение трациций MVC (отделение биснес-логики от логики отображения, или грубо говоря, отсутствие внятных шаблонов) умолчу 5. Про использование текстовых файлов вместо базы данных умолчу 6. Про то, что если изменить имя файла базы в переменной, скрипт работать будет не так, умолчу 7. Про нотисы тоже умолчу, хотя про уровень вывода ошибок я говорил. Больше нуля не поставил бы. Максимум - единицу. Да и то за старание. Надеюсь, что все это выглядело не слишком грубо |