Можно ревью кода на Python?

Question

[ArconB]

Пытался реализовать MVC паттер, посмотрев ролик соера. Можно небольшой ревью кода?
https://gist.github.com/ApppleTurnover/65399a8a9a5...

И еще вопрос. Где лучше всего искать ревью? Хочется чаще видеть фидбек о своем коде, а куда скидывать - я хз

Comments

[Влад Григорьев]

явно прослеживается, что вы не понимаете зачем используется dataclass
ну и функции кажется правильнее в методы перевести

[ArconB]

Влад Григорьев, метод какого класса? Выделить отдельный класс под действия или же в классе человек?

[Влад Григорьев]

AthonD, в класс человек

[Andy_U]

Type annotations частично отсутствуют, docstrings (например, в Google-стандарте - нет... Нафига наследование, непонятно. Или у вас пьющие животные будут? Та же фигня с наследованием бутылки от сосуда и его испольщование в качестве параметра в функциях. Нет проверки на отрицательное количества жидкости в бутылке.

[ArconB]

Andy_U, это тонкости, которые я не учитывал. Наследование - для маштабирование. Пьют не только люди, но животные, птицы, насекомые. Из сосудов есть еще чашки, кружки и тд и тп. Я просто попытался быстро накидать и коротко мое виденье решения задачи в MVC формате

[Andy_U]

AthonD, Ну и чем процесс питья у животных отличается от человека? Т.е., добавьте в Entity тип существа и хватит пока. Та же фигня с чашками, стаканами и пр. Добавьте к Vessel тип посуды и опять пока хватит.
Не надо преждевременно усложнять.