Skip to content

Instantly share code, notes, and snippets.

View defenitionofreal's full-sized avatar
:octocat:
{"status": "coding"}

Erik Arakelian defenitionofreal

:octocat:
{"status": "coding"}
  • Russia
View GitHub Profile
Зачада 1
Комментарии по коду (https://gist.github.com/chepe4pi/3c5c2e4bab87fbf155dfdeaa18e2fc8f):
- Не соблюден PEP8
- Не соблюден прицип единной ответственности, к примреу одна вью не должна выполнять два разных действия
- Бизнес логика не вынесена отдельно
- Нейминг моделей и полей в них затрудняет работу.
Все модели должны называться в единственном числе так как мы перебираем по записи в БД (переименовать модель Costs в Cost и так далее).
Поля в моделях, если возвращают булевый тип, должны начинаться с is_ или has_ (переименовать поле goal if is_goal и так далее)
Поля, которые ссылаются на связь с объектом, не должеы называться orderid, stageid, cityid. (переименовать в order,stage,city)