This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Зачада 1 | |
Комментарии по коду (https://gist.github.com/chepe4pi/3c5c2e4bab87fbf155dfdeaa18e2fc8f): | |
- Не соблюден PEP8 | |
- Не соблюден прицип единной ответственности, к примреу одна вью не должна выполнять два разных действия | |
- Бизнес логика не вынесена отдельно | |
- Нейминг моделей и полей в них затрудняет работу. | |
Все модели должны называться в единственном числе так как мы перебираем по записи в БД (переименовать модель Costs в Cost и так далее). | |
Поля в моделях, если возвращают булевый тип, должны начинаться с is_ или has_ (переименовать поле goal if is_goal и так далее) | |
Поля, которые ссылаются на связь с объектом, не должеы называться orderid, stageid, cityid. (переименовать в order,stage,city) |