Правильно ли выглядит структура таблиц в MySQL?

Question

[DarkUser1337]

Здравствуйте! Правильно ли настроены эти таблицы с точки зрения грамотного хранения данных и оптимизации? Какие ошибки и недочеты могут возникнуть? Что исправить?

TINYINT использую вместо bool, значение всегда будет 1/0
DATETIME естественно для даты и времени. Может лучше использовать UNIX time с целыми числами?

таблица restore для восстановления пароля

`id` int(11) NOT NULL,   
 `user_id` int(11) NOT NULL DEFAULT 0,  
 `email` varchar(100) COLLATE utf8_unicode_ci NOT NULL,   
 `restore_key` varchar(191) COLLATE utf8_unicode_ci NOT NULL, -- ключ, который отправится на email   
 `changed` tinyint(1) NOT NULL DEFAULT 0, -- при следующем переходе по той же ссылке, проверяется это значение, менялся ли уже пароль
 `created_at` datetime NOT NULL

таблица users

`user_id` int(11) UNSIGNED NOT NULL,
  `username` varchar(64) COLLATE utf8_unicode_ci NOT NULL,
  `email` varchar(100) COLLATE utf8_unicode_ci NOT NULL,
  `password` varchar(128) COLLATE utf8_unicode_ci NOT NULL,
  `password_salt` varchar(64) COLLATE utf8_unicode_ci NOT NULL,   -- динамическая соль
  `token` varchar(191) COLLATE utf8_unicode_ci NOT NULL,          -- ключ для доступа к API
  `hwid` varchar(128) COLLATE utf8_unicode_ci NOT NULL,           -- хеш
  `activation_key` varchar(191) COLLATE utf8_unicode_ci NOT NULL, -- ключ для подтверждения email'a
  `status` tinyint(1) NOT NULL DEFAULT 0,                         -- подтвержден ли email
  `password_changed_date` datetime NOT NULL DEFAULT current_timestamp(),
  `join_date` datetime NOT NULL

таблица user_privileges привилегии для пользователей (админ, модератор и т.п.)

`id` int(11) NOT NULL,
  `user_id` int(11) UNSIGNED NOT NULL,
  `level` int(11) NOT NULL DEFAULT 1

Индексы

таблицы restore, user_privileges
ADD PRIMARY KEY (`id`);

таблица users

ADD PRIMARY KEY (`user_id`),
  ADD KEY `username` (`username`),
  ADD KEY `email` (`email`),
  ADD KEY `hwid` (`hwid`),
  ADD KEY `token` (`token`);

Атрибут AUTO_INCREMENT

таблицы restore, user_privileges
MODIFY `id` int(11) NOT NULL AUTO_INCREMENT;

таблица users

MODIFY `user_id` int(11) UNSIGNED NOT NULL AUTO_INCREMENT;

Comments

[John Didact]

Я может ошибаюсь, но bool в MySQL это и есть tinyint(1), на данный момент, по крайней мере…
Я, раньше, вместо bool использовал char(0), где пустая строка означала истину, а значение NULL - ложь. Было удобненько… но до тех пор, пока мне не пришлось использовать поле в условии (where) SELECT. Не знаю, зачем написал тебе об этом…

[ThunderCat]

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

Например для приложения на пхп поле password_salt излишне. В остальном есть несколько явно специфичных полей, и большинство стандартных.
Важно: Не морочьте себе голову идеальностью таблиц на этапе проектирования. Важно чтобы к моменту готовности к работе с реальными наборами данных вы их спроектировали. А это далеко не сразу. По мере необходимости можно менять или настраивать таблицы/индексы под задачи.

[DarkUser1337]

ThunderCat, Можете пожалуйста рассказать, почему поле password_salt излишне? И где в таком случае хранить динамическую соль? Спасибо.

[ThunderCat]

DarkUser1337, Есть нативная функция password_hash() и соответствующая парная password_verify(). Там уже весь механизм реализован без дополнительных полей.

[DarkUser1337]

ThunderCat, этого недостаточно безопасно, насколько хотелось бы. Особенно, когда видишь на официальном мануале вот такую строчку, которая жирным шрифтом на красном фоне выделяется:

"Внимание
Эта опция была объявлена устаревшей начиная с PHP 7.0.0. Рекомендуется использовать автоматически генерируемую соль."

Источник: https://www.php.net/manual/ru/function.password-ha...

[ThunderCat]

DarkUser1337, угу, только еще важно ПОНИМАТЬ что вы читаете, в доке как раз смысл "не лезь своими кривыми ручонками с солью, используй нативную функцию"