comments app #11
comments app #11
Conversation
core/enums.py
Outdated
| from enum import IntEnum | ||
|
|
||
|
|
||
| class Limits(IntEnum): |
There was a problem hiding this comment.
Это что?
Какую задачу решает?
Почему max_length это не константа в settings.py?
There was a problem hiding this comment.
Коллекция перечислений.
Большой разницы нет, если не нужно перебирать или получить имя по значению.
Когда добавлял, ещё констант не было в settings.
Так код немного понятнее, на мой взгляд.
Импорт класса Limits более понятен, чем импорт settings и лаконичнее чем импорт констант из settings.
Удалю, перенесу константой в settings.
There was a problem hiding this comment.
Я вроде разобрался, интересное решение.
Я бы попробовал, но переносить константы из settings уже лениво.
В идеале такое должно быть установлено как правильно на старте проекта.
PS: А почему в MAX_LEN_COMMENT_TEXT = (200,) 200 внутри кортежа?
На первый взгляд должно быть MAX_LEN_COMMENT_TEXT = 200
There was a problem hiding this comment.
С кортежем, я ошибся.
Перенес константой в settings.
Модель ирегистарция в админке.