Как лучше сделать проверку на занятость логина?

Question

[Ivan Ivanovich]

Здравствуйте.

Есть таблица users

* id

* login

* name

* password

Как сделать при регистрации проверку на занятость логина?

Есть код:

$login = $_POST['login'];
  $name = filter_var(trim($_POST['name']),FILTER_SANITIZE_STRING);
  $password = filter_var(trim($_POST['password']),FILTER_SANITIZE_STRING);


  if (mb_strlen($login) < 3 || mb_strlen($login) > 91) {
    echo "Логин должен быть от 3 до 91 символов.";
    exit();
  } else if (mb_strlen($name) < 3 || mb_strlen($name) > 99) {
    echo "Имя должно быть от 3 до 99 символов.";
    exit();
  } else if (mb_strlen($password) < 6 || mb_strlen($password) > 99) {
    echo "Пароль должен быть от 6 до 99 символов.";
    exit();
  }

  $mysqli = new mysqli('localhost', 'root', '123', 'db');
  $mysqli->query("INSERT INTO `users` (`login`, `password`, `name`)
    VALUES('$login', '$password', '$name')");


    $mysqli->close();
    header("Location: ".$_SERVER["HTTP_REFERER"]);

Подскажите пожалуйста, как лучше сделать? Расположить в блоке if или ещё как-то. И вообще какую лучше дать команду в БД для этого. Облазил достаточно много сайтов, но на большинстве из них только устаревшая информация.

Заранее спасибо каждому, кто поможет.

Comments

[Одиночка Айс]

Ищите перед регистрацией логин в БД, при этом приведя строку в нижний регистр, если есть такой, шлете лесом.

[Ivan Ivanovich]

Одиночка Айс, Можете пожалуйста запрос к БД начеркать?

[ThunderCat]

Ivan Ivanovich, че там черкать, селект вэа логин=$login

[Ivan Ivanovich]

ThunderCat, "SELECT * FROM `users` WHERE `login`=$login";

[ThunderCat]

Ivan Ivanovich, ну, я бы только не *, а конкретное поле выбирал, например id, а так - все верно, ну за исключением того что логин может содержать иньекцию, и неплохо бы использовать подготовленные выражения.

[Ivan Ivanovich]

ThunderCat, А как сделать проверку? Например я делаю

else if ($login == $logincheck) {
    echo "Логин занят";
    exit();
  }

но это не работает.

[Ivan Ivanovich]

ThunderCat, $r = "SELECT * FROM `users` WHERE `login`=$logincheck"; сдеал logincheck чтобы не путаться пока

[ThunderCat]

Ivan Ivanovich, это у вас какое-то программирование методом тыка?
Проверяйте результат выборки - если пустая выборка - логин свободен, если есть значение - значит занят.

[Ivan Ivanovich]

ThunderCat, через while?

[ThunderCat]

Ivan Ivanovich, зависит, теоретически возвращается не более 1 значения, курсор установлен в первое значение, по этому можно просто вытащить 1 значение фетчем прям внутри ифа.

[Ivan Ivanovich]

ThunderCat, Хорошо, буду пробовать, спасибо!

Answer 1

[Stalker_RED]

$login = filter_input(INPUT_POST, 'login', FILTER_SANITIZE_STRING);

Заблуждения программистов об именах
Заблуждения программистов об именах — с примерами

Также рекомендую: о времени и о адресах

Правила ты можешь устанавливать на логин и пароль, но не на имя.
У логина проверяй только длину и допустимые символы, это все можно записать одной регуляркой или несколькими if'ами.

Дальше пытаешся вставить в базу, там конечно unicue на поле login, и при ошибке duplicate показывай клиенту, что уже занято.

Comments

[ThunderCat]

уник на логин ок, но обрабатывать через ошибку вставки не есть гуд. Логичнее сделать запрос на существование логина в базе.

Answer 2

[Asjacker]

$rs_username = $mysqli->query("SELECT * FROM users WHERE login='$login'")->rowCount();

проверяем, есть ли такой логин
если запрос отдал 0, то идет запись в бд

if($rs_username == 0) {
 $mysqli->query("INSERT INTO `users` (`login`, `password`, `name`)
    VALUES('$login', '$password', '$name')");
}

если такой логин уже существует то вылезает ошибка:

else {
                echo "<script>alert('Такой логин уже существует!')</script>";
            }

Вот код полностью :

if(isset($_POST['login'])) {
$login = $_POST['login'];
  $name = filter_var(trim($_POST['name']),FILTER_SANITIZE_STRING);
  $password = filter_var(trim($_POST['password']),FILTER_SANITIZE_STRING);


  if (mb_strlen($login) < 3 || mb_strlen($login) > 91) {
    echo "Логин должен быть от 3 до 91 символов.";
    exit();
  } else if (mb_strlen($name) < 3 || mb_strlen($name) > 99) {
    echo "Имя должно быть от 3 до 99 символов.";
    exit();
  } else if (mb_strlen($password) < 6 || mb_strlen($password) > 99) {
    echo "Пароль должен быть от 6 до 99 символов.";
    exit();
  }

  $mysqli = new mysqli('localhost', 'root', '123', 'db');
 $rs_username = $mysqli->query("SELECT * FROM users WHERE login='$login'")->rowCount();
  if($rs_username == 0) {
 $mysqli->query("INSERT INTO `users` (`login`, `password`, `name`)
    VALUES('$login', '$password', '$name')");
}{
$mysqli->close();
    header("Location: ".$_SERVER["HTTP_REFERER"]);}
else {
                echo "<script>alert('Такой логин уже существует!')</script>";
            }
    }