Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

фидбек #1

Open
asundukov opened this issue Jan 9, 2018 · 1 comment
Open

фидбек #1

asundukov opened this issue Jan 9, 2018 · 1 comment

Comments

@asundukov
Copy link

  1. на messages отсутствует primary key - это несколько антипаттерн оставлять бизнесовую сущность без идентификатора - тогда мы не сможем манипулировать ей, например нельзя удалить конкретное сообщение т.к. если два сообщения будут содержать один текст и относить к одному response - то получим проблему

  2. нейминг внешних ключей - в одном случае usr_id, в другом - vacancy_id . Стоит, разумеется, всегда придерживаться единого стайлинга, иначе создаешь проблему не только для чтения запросов глазами но и в будущем когда база сильно вырастет - будут проблемы с поиском нужных столбцов при написании новых запросов

  3. active_days - очень спорное решение, получается его нужно обновлять ежедневно в 0-00,
    лучше в таких случаях использовать пару publish_start + publish_end с типом времени, либо просто publish_end если старт не нужен

  4. в генерации usrid явно завязан на то что скрипт отрабатывает на свежей бд, иначе если там часть пользователей была удалена - он может не выполниться, лучше конечно добавлять во все связанные таблицы точно существующие значения, по остальным аналогично

@asundukov
Copy link
Author

Да, теперь очень неплохо получилось!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant