БД SQL — нормальны ли magic numbers в БД, которые имеют константы в back end коде? Или лучше сделать таблицу и отношение?

Question

[sorry_i_noob]

Здравствуйте. Я пишу интернет магазин. И вот у товара есть статусы - скрыт, показан. Или у тега - скрыт, показан, запрещен к созданию (какое-нибудь матерное слово, например).
В БД в поле это 0 - скрыт, 1 - показан, 2 - запрещен.
У меня в php есть класс Status и у него константы, которые раскрывают эти значения:

const STATUS_HIDDEN = 0;
const STATUS_VISIBLE = 1;
const STATUS_BANNED = 2;

Вопрос - нормально ли то, что константы есть в php, но в БД это просто цифры (magic numbers) - 0, 1, 2? Нужно ли создавать таблицу в БД - statuses, где будут статусы?

Comments

[Евгений]

очевидно же, что у того кому вы передадите проект будет ряд вопросов) запилите табличку, не жадничайте

[sorry_i_noob]

Евгений, тогда при работе со статусами будет еще одно лишнее обращение - к таблице statuses. А чем меньше обращений, тем лучше ведь, разве нет? Насколько это критично?

[iljaGolubev]

sorry_i_noob, Зависит от проекта. Но в любом случае, всегда можно 1 раз прочитать из базы при инициализации скрипта и положить в переменную.

Answer 1

[d-stream]

Как показала практика в живущих проектах "1,2,3 и никогда не будет расширятся" - достаточно быстро расширяется)

Как примеры: еще недавно пол был М или Ж и можно было хардкорить… уже типа нет...

Ну и референсная таблица - хорошее место для хранения вариантов описаний ("запр.", "запрещен к продаже") для разных случаев отображения.

p.s. Кстати почему бы не быть товару показываемым только неким группам? (вот и 4 вариант для того что не может меняться)

Comments

[sorry_i_noob]

А что мешает добавить новый пол в качестве константы в класс? Как и новый статус. Это ведь просто число.

[d-stream]

sorry_i_noob, потом этих полов становится 100500, они обрастают дополнительными атрибутами....
В итоге через 10-15 лет получается франкенштейн, которого рано или поздно приходится на ходу и с оглядками на версионность все-таки превращать в related-table...

Естественно пол - это слегка утрированно, но общий смысл именно таков

Answer 2

[Rsa97]

Если дальнейшего расширения не предполагается, сделайте поле с типом ENUM.

Comments

[sorry_i_noob]

Спасибо. А если сделать таблицу и отношение. При работе со статусом придется обращаться к БД, чтобы получить возможные статусы. Насколько это критично? Насколько хуже, чем обращаться к константе в классе php?

[Rsa97]

sorry_i_noob, Просто лишний запрос к базе или JOIN в запросе. Пока ваш сайт не сильно нагружен, влияние особого не окажет.

Answer 3

[Дмитрий Энтелис]

Я считаю что нормально иметь такие вещи целиком в php, а в базе держать просто tinyint