search

Danstiv / cm-assistant

1 stars 1 forks source link

Enum вместо обычных констант #20

Open TheZavitaev opened 2 years ago

TheZavitaev commented 2 years ago

https://github.com/Danstiv/cm-assistant/blob/master/tgbot/constants.py#L1

Я бы Enum завел

Danstiv commented 2 years ago

Хорошая идея.

Danstiv commented 2 years ago

Хм, не, не очень хорошая. Эти константы используются в качестве user_id для gui-окон, значения в базу пишутся, и если сделать enum, то нужно будет проверки писать при добавлении в базу, при получение из базы и т.п, так что, вероятно, enum-ы не подойдут.