Conversation
| if username in BANNED_USERNAMES: | ||
| return HttpResponse('User banned') | ||
| else: | ||
| return HttpResponse('User not banned') |
There was a problem hiding this comment.
Тут ты в ифе не только сообщение выбираешь, но и копипастишь тип ответа и делаешь два ретурна. В этом случае будет лучше, если ты ифом будешь только текст ответа выбирать, а формирование и возврат ответа будет один раз уже после ифа.
А ещё можно воспользоваться тренарным оператором вместо ифа, будет ещё чище и понятнее.
| 'Октябрь', | ||
| 'Ноябрь', | ||
| 'Декабрь' | ||
| ][month_number - 1] |
There was a problem hiding this comment.
Форматирование не очень. Вынеси список в отдельную переменную, тогда будет нормас. А ещё можешь глянуть модуль calendar, авось там найдётся чего полезного :)
| def get_month_title_by_number(month_number: int): | ||
| pass # код писать тут | ||
| locale.setlocale(locale.LC_ALL, "ru_RU") | ||
| months = list(calendar.month_name) |
|
|
||
| def get_month_title_by_number(month_number: int): | ||
| pass # код писать тут | ||
| locale.setlocale(locale.LC_ALL, "ru_RU") |
There was a problem hiding this comment.
А вот это неправильно: сейчас у тебя функция получения названия месяца неявно меняет локаль, это не очень.
Когда мне надо временно переопределить локаль, а потом вернуть старую, я вот этим способом пользуюсь: https://stackoverflow.com/questions/50737783/
2 participants
No description provided.