ololo__schoolboy
Сообщений: 118
Оценки: 0
Присоединился: 2011-06-14 12:42:52.573333
|
Скрипт достаточно сырой, дабы не быть голословным предлагаю детально разобраться.
1 Первое что бросается в глаза это отсутствие отступов и коментариев.
2 Отсутствие процедуроного ситиля
3 preg_match("/^[0-9]{1}/", $_REQUEST['rr']) в реглярном выражении дапущена ошибка:
javascript:gg('Exa', 99, 2) Enter - и рейтинг уже 100 из 10:)
нужно использовать знак $ чтобы декларировать конец строки, "/^[0-9]{1}$/" так-же можно использовать модификатор \d вместо [0-9], но это уже дело вкуса.
4 Вместо того, чтобы хранить данные черти как, а потом проявлять чудеса инженерной мысли, можно пользоваться сериализацией
5 if($cf=='n'){
$k=$rr;
$u[2]='1';
$file=fopen('rt.txt', 'a+');
fputs($file,"$rt|$rr|1|\r\n");
fclose($file);
} rt не проверялось совсем, да и rr через пень колоду, xss на лицо.
6 $st="<style>.rat {font-size:13px;font-family:arial;} ";
/* .... */
лучше вынести в отдельный css и пользоваться link
Думаю прежде чем говрить об оценке, стоит, осуществить немеренный объем работы
|
Оцените скрипт рейтинга - 
