Новостной скрипт
Автор: RapteR*
Дата: 05.05.2006  03:28
Комментариев: 10
Просмотров: 553
Оценка: 2 (0|1|1|0|0) [0]
JeRR* Неофит (ур.7) ::: комментарий от 17.03.2007 | 22:30
отлично, только через регуляры следовло бы вырезать из новостей ява-скрипты (чтобы смотрящий не пострадал от рук размещающего). И думаю не плохо бы было подумать о "PHP отдельно, HTML отдельно"
mr. Twister Герой (ур.14) ::: комментарий от 24.02.2007 | 17:51
ан нет рабочий он, вот токо где его третья часть?


--------------------
1
mr. Twister Герой (ур.14) ::: комментарий от 24.02.2007 | 17:43
мне кажется, что не рабочий скрипт


--------------------
1
Videot* Неофит (ур.7) ::: комментарий от 28.01.2007 | 17:13
Странный какойта скрипт..


--------------------
Punk not DEAD!
nopox Дух (ур.1) ::: комментарий от 17.07.2006 | 16:26
Ниже плинтуса. По-другому оценить, извините, сложно. Это были амбиции, извините А теперь конструктив:



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



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


--------------------
Я - часть той силы, что вечно хочет зла и вечно совершает благо
wolong Искатель (ур.5) ::: комментарий от 18.06.2006 | 15:59
Zloy прав if ($name != "") : длиннее чем emerty()

но я бы сам if написал - проще это, привычнее...


--------------------
$_="1K8}3v3w8(5f1o2q5y6n3h1s5%2R0e9{7s9)5m1b6i2m6k3u";s/(.)(.)/pack('C',ord($2)-$1)/eg;print;
RapteR* Приверженец (ур.6) ::: комментарий от 22.05.2006 | 18:32
ты чрезвычайно туп, что кароче empty() или !="" ??? зачем программе веса добавлять?


--------------------
Хочеш сделать хорошо, сделай сам!
Zloy Человек (ур.3) ::: комментарий от 16.05.2006 | 14:30
if ($name != "")

ты про empty() слышал?
RapteR* Приверженец (ур.6) ::: комментарий от 15.05.2006 | 13:00
Ну на счёт матов я проглядел, их было больше)

на счёт разделителей, лучше как бы избегать текстовиков в принципе, а если уж на то пошло то фильтровать поля на наличие | чем то вроде:

$date = str_replace( "|"," ",$date);

$autor = str_replace( "|"," ",$autor);

$news = str_replace( "|"," ",$news);

всё это можно сунуть в 17 строку после if`ака.


--------------------
Хочеш сделать хорошо, сделай сам!
Chronos Приверженец (ур.6) ::: комментарий от 06.05.2006 | 13:19
Позволю себе несколько коменариев ;-)



Имеет смысл не использовать не нормативную лексику для вывода исключительных ситуаций .

///print "И где бл*** файл???";

Особенно это касаеться веб приложений, так как увидеть его могут разные люди, а у некоторых с людей с которыми ты как то можешь быть связан может не оказаться юмора ;-) например у заказчика разрабатываемого тобой сайта :о)



И второй совет, у тебя в качестве разделителя используеться символ "|", если пользователь передаст скрипту данные сожержащие такой символ, то при отображении данных обязательно будут искажения ;-). Рекомендую использовать более длинный сплитер(разделитель), что то вроде т.е. такая комбинация которая с высокой вероятностью не будет переданна из формы ;-)



А в целом вполне одекватная реализация ;-)


--------------------
Хорошо смееться тот кто стреляет последним