|
| |
|
|
| |
для: Akira
(06.07.2005 в 18:37)
| | | Я не стал затрагивать эту тему... | |
| |
|
|
| |
|
|
| |
для: cheops
(06.07.2005 в 22:04)
| | | я же говорил что это не только пробный вариаен, все будет изменено на читаемое для непосвещенного пользователя, пока это только набросок.
Спиб за критику :) код несколько переработал...
2 isset : логические ошибки? можно указать?)
.. | |
| |
|
|
| |
|
|
| |
для: Гость
(06.07.2005 в 17:36)
| | | Из фразы
/*Так теперь мы уверены - поля для ввода не пустые,
**и в них что-то есть %)
**Теперь следует обрезать длину этих полей,
**на тот случай если пользователь
**локально изменит параметр maxlength
|
не понятно чем это может грозить - хорошо бы расшифровать зачем об этом следует беспокоиться. | |
| |
|
|
| |
|
|
| |
для: Гость
(06.07.2005 в 17:36)
| | | Сленг, вроде "бутафорс" лучше явно расшифровывать, так как например я впервые встречаю это слово и если бы не знал, для чего нужна задержка не понял бы назначение функции sleep().
Причём фразу
//Используем sleep, для чего? - чтобы исключить
|
Лучше заменить на
//Используем задержку в 2 секунды, для чего? - чтобы исключить
|
Так как читатель может не знать, об существовании функции sleep() и её предназначении и из первой фразы ничего не поймёт, а из второй фразы он сможет сразу догадаться о том, для чего предназначена функция. | |
| |
|
|
| |
|
|
| |
для: Гость
(06.07.2005 в 17:36)
| | | Наличие конструкции elseif тяжело воспринимается и плохо читается - для увеличения читабельности кода лучше заменить её на if | |
| |
|
|
| |
|
|
| |
для: Гость
(06.07.2005 в 17:36)
| | | Не совсем чёткая проверка
<?php
$user1=mysql_fetch_array($user);
//Далее смотрим вернулся ли запрос - если нет значит такого пароля в БД нет
if (!$user1)
?>
|
лучше переписать
<?php
//Далее смотрим вернулся ли запрос - если нет ни одной строки значит такого пароля в БД нет
if (mysql_num_rows($user)==0)
?>
|
Кроме того использование переменных с цифрой на конце ($user1) является плохим тоном и не желательно прибегать к нему в стать - код лучше вылизать и дать переменным осмысленные имена | |
| |
|
|
| |
|
|
| |
для: Гость
(06.07.2005 в 17:36)
| | | Неплохо строки
<?php
$login=$_POST['Login'];
$pass=$_POST['Pass'];
?>
|
Явно прокомментировать, что это логин и пароль, которые передаются из HTML-формы. | |
| |
|
|
| |
|
|
| |
для: Гость
(06.07.2005 в 17:40)
| | | Посмотри мою %) | |
| |
|
|
| |
|
|
| |
для: Гость
(06.07.2005 в 17:36)
| | | При беглом осмотре не нашел не куки не сессий...Так и надо? | |
| |
|
|
| |
|
|
| |
для: Гость
(06.07.2005 в 17:36)
| | | Лучше не выкладывать эту статью, я уже около трех логических ошибок нашел + сам код очень странный | |
| |
|
|
|
