1

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. Про нотисы тоже умолчу, хотя про уровень вывода ошибок я говорил.

Больше нуля не поставил бы. Максимум - единицу. Да и то за старание. Надеюсь, что все это выглядело не слишком грубо