Ссылка
нажмите — покажем
нажмите — покажем
Товарищи! Только что докончил работу над своим первым (не)полноценным проектом. Прошу рецензий, да советов с пистонами от Вас, дорогие мои профи https://github.com/IamDimitriy/mathematical-modeling.git
ВеткаТоварищи! Только что докончил работу над своим первым (не)полноценным проектом. Прошу рецензий, да советов с пистонами о…
29 сообщений · –Д D R A
T
V Ссылка
нажмите — покажемПо верхам посмотрел: 0) Entry-point-а в виде main.py в проект просто нет. Как запустить? Я чо, чтоб запустить твой проект, должен все файлики посмотреть? 1) Однотипные константы в Enum-ы собирай (вот эти, например). 2) Вот это жуть. Путь до проекта через pathlib доставай. Хардкодить пути вообще никогда нельзя. 3) Кириллические символы убирай. Всё на латинице. И названия спрайтов тоже. 4) data = [] — не константа, а глобал. Никогда не используй глобалы. (@advice17 — там хорошо приведены аргументы). Что она делает в константах — непонятно. 5) Именование переменных не очень. Что такое quantity_fac — непонятно без думания и прыгания по коду. 6) Юзай тайпхинты. Передаешь какой-то плохо заименованный df в build_a_graph — ваще непонятно что происходит. 7) В bacanimal: мега-жирный класс. Нехорошо. Функции огромные и непонятно что делающие. Функция — название из глагола+опционального_существительного. Функция длиннее 15 строк — плохая функция (чаще всего). Функция, выполняющая больше 1 примитивной задачи — плохая функция. У тебя же функции — отдельные сущности. 8) В README должен быть гайдлайн по установке. Нет requirements.txt. 9) Не соблюден пеп почти везде. 10) ... кто-нибудь может принять эстафету и продолжить. Для начала хватит =)V R Д Фотография
нажмите — покажемАндрей, спасибо большое тебе за рекомендации, очень полезно для начинающего. Но у меня тем не менее созрел ряд вопросов: 0) Насколько критично, что у меня в директории нет папки __init__.py и что и зачем я могу в нее поместить (инициализация данных из constant без того работает)? 1) Как использовать Enum-ы в моем случае? 2) Как достать через pathlib путь к частным файлам, да такой чтобы они были 'WindowsPath' и в последующем подлежали конкатенации и на получившийся путь программа могла ссылаться? На скрине ниже я создаю новые файлы, или делаю то, что нужно?Д
Д
🍉 Д