Зачем login повторяется в phpmyadmin?

Question

[Palwanchik]

Здраствуйте, я кодировал форум вход и регистрация. Он работает без ошибки, но login повторяется в phpmyadmin.

Comments

[Everything_is_bad]

плохо кодировал, тупо вставляешь каждый раз, а не проверяешь существования записи

[Василий]

Непосредственно к вопросу это отношения не имеет, но что бы 2 раза не вставать там еще пароль в открытом виде хранится. Это надо бы исправить.
1. Добавить поле salt там хранить соль пароля
2. хранить ни чистый пароль а результат hash(password+salt)
https://www.php.net/manual/ru/function.password-ha...
3. проверять на валиность так https://www.php.net/manual/ru/function.password-ve...

[Rsa97]

Если используется функция password_hash, то никакой соли вручную добавлять не надо, она добавляется автоматически самой функцией и хранится вместе с хэшем пароля.
Например, хэш

$2a$10$vI8aWBnW3fID.ZQ4/zo1G.q1lRps.9cGLcZEiGDMVr5yUP1KUOYTa

2a - алгоритм bcrypt;
10 - стоимость, 2¹⁰ операций;
vI8aWBnW3fID.ZQ4/zo1G.q - соль в Base64;
lRps.9cGLcZEiGDMVr5yUP1KUOYTa - собственно хэш пароля в Base64.

[Василий]

не тем более

Answer 1

[Вадим Освальд]

У тебя отсутствует уникальное ограничение для этого поля.
Есть запрос на установку такого ограничения:
ALTER TABLE users ADD UNIQUE (login);

Так же, чтобы удалить имеющиеся сейчас дубли:

DELETE FROM users
WHERE id NOT IN (
    SELECT MIN(id)
    FROM users
    GROUP BY login
);

В обработке регистрации должна проверка на наличие существующего логина в базе данных перед добавлением

Comments

[Palwanchik]

Также хочу чтобы писал "Данный ползователь уже зарегистрован" при повтора login

[Владислав Лысков]

Palwanchik, ну, кодируй, в чём вопрос?

[Rsa97]

Советую предварительно проверять свои ответы.
Запрос на установку уникального ключа не выполнится, пока в колонке есть дубли.
Duplicate entry 'salam' for key 'users.login'
А ваш запрос на удаление дублей не выполнится, потому что нельзя делать подзапрос по таблице, в которой основной запрос проводит удаление или изменение данных.

You can't specify target table 'users' for update in FROM clause

[Palwanchik]

Владислав Лысков, А как??

Answer 2

[Rsa97]

1. Удаляем дубли:

DELETE u.*
  FROM users AS u
  JOIN users AS d
    ON d.login = u.login AND d.id < u.id;

2. Ставим уникальный индекс на login, чтобы добавить дубль было невозможно даже при ошибке в коде:
ALTER TABLE users ADD UNIQUE (login);
Переписываем код.
3 (необязательно). Прежде, чем добавлять логин в таблицу, делаем запрос на наличие в ней такого логина и, если он есть, то возвращаем сообщение, что такой логин уже есть.
4. Делаем вставку и, если она не удалась, то проверяем ошибку. Если это ошибка 1062 Duplicate entry, то какой-то поток успел вставить такой логин раньше и, опять же, возвращаем сообщение, что такой логин уже есть.