Форум программистов [Powered by Invision Power Board]

Код


    $index = true;
        if(count($_POST) == 5 && isset($_POST['login']) && isset($_POST['passwd']) && isset($_POST['passwd2']) && isset($_POST['email']) && isset($_POST['captcha'])) {
            $login = htmlspecialchars($_POST['login']);
            $passwd = htmlspecialchars($_POST['passwd']);
            $passwd2 = htmlspecialchars($_POST['passwd2']);
            $email = htmlspecialchars($_POST['email']);
            $captcha = htmlspecialchars($_POST['captcha']);
            if(strlen($login) < 3 || strlen($login) > 32) {
                $errmsg_login = 'Логин должен быть от 3 до 32 символов';
                $index = false;
            }
            if(empty($errmsg_login) and !empty($login) && !eregi("^([A-Za-z0-9А])", $login)) {
                $errmsg_login = 'Неверный формат логина';
                $index = false;
            }
            if(strlen($passwd) < 3 || strlen($passwd) > 32) {
                $errmsg_passwd = 'Пароль должен быть от 3 до 32 символов';
                $index = false;
            }
            if(empty($errmsg_passwd) && $passwd2 != $passwd) {
                $errmsg_passwd = 'Пароли должны быть одинаковы';
                $passwd2 = '';
                $index = false;
            }
            if(empty($errmsg_passwd) && !eregi("^([A-Za-z0-9А])", $passwd) && !empty($passwd)) {
                $errmsg_passwd = 'Неверный формат пароля';
                $passwd2 = '';
                $index = false;
            }
            if($_SESSION['captcha_keystring'] <> $captcha) {
                $errmsg_captcha = 'Не верный код.';
                $index = false;
            }
            if (!eregi("^([0-9a-z]([-_.]?[0-9a-z])*@[0-9a-z]([-.]?[0-9a-z])*\.[a-wyz][a-z](fo|g|l|m|mes|o|op|pa|ro|seum|t|u|v|z)?)$", $email) and $email != "") {
                $errmsg_email = 'неверный формат e-mail';
                $index = false;
            }
            if($index) {
                register_new_user($login, $passwd, $email); // Вызываем функцию регистрации пользователя в базе и перекидываем на главную
                redirect("./index.php");
                exit;
            } else {
                                // Выводим форму регистрации с сообщениями об ошибках
                        }
        }

Цитата

htmlspecialchars

Код


            if($index) {
                register_new_user($login, $passwd, $email); // Вызываем функцию регистрации пользователя в базе и перекидываем на главную
                redirect("./index.php");
                exit;
            } else {
                                // Выводим форму регистрации с сообщениями об ошибках
                        }

Цитата

1. Зачем подсчитывать кол-во данных? При последующем изменении скрипта, месяца через 2, 100% будут непонятки почему данные не воспринимаются скриптом.

Цитата

2. Если данных нет, то не понятно поведение скрипта.

Цитата

3. Ошибки лучше собирать в массив. Тогда и флага index не надо будет, сделал count($errors) и понял что дальше делать. Более того, список ошибок будет полнее, нежели чем сейчас, а значит дружелюбнее к пользователю. Пример с логином:
Человек ввел пробел больший 32 символов не соответствующий допустимой маске пароля. В случае этого скрипта, напишется только про 32, а про недопустимость такого пароля он узнает только отправив форму второй раз.

Цитата

4. Не понятное регулярное выражение A-Za-z0-9А, зачем A два раза.

Цитата

5. Если пишем в базу то http://php.net/mysql_escape_string