А можно код ревью попросить?) Это рендерилка для блендера которая бекенды

https://github.com/CNC5/GlacierRender/blob/main/glacier-backend/database.py#L91 тут тоже все плохо. Класс делает все и сразу, при том делает это максимально плохо. 1. Класс лезет в переменные окружения. 2. Класс зачем-то внутри __init__ регистрирует таблицы, при уже существующих моделях... нахрена? 3. Класс еще и вызывает create_all в __init__

0

07.05.2023

Lol ava Автор вопроса

Alex
https://github.com/CNC5/GlacierRender/blob/main/gl...

1. Я правильно понимаю что этим должен заниматся отдельный файл типа config.py с датаклассами? 2.Читал, документацию, делал вроде как там написано, хотя не исключаю что не нашел чего-то и полез в гугл, потыкаю, исправлю 3.Это связано с пунктом 2...

0

07.05.2023

Alex

Lol ava
1. Я правильно понимаю что этим должен заниматся о...

Конфигурацию загружаешь отдельно, как угодно, один из вариантов в виде датаклассов и передаешь ее уже тем классам, которым они нужны. Сейчас твой класс зависит от глобального состояния, которым является окружение твоего процесса.

0

07.05.2023

Alex · Accepted Answer

Alex

Сразу что бросается в глаза это странная (верней отсутствующая) структура проекта. У тебя в корне репозитория валяется __init__.py с кодом, однако есть еще папка glacier-backend со свалкой скриптов внутри

0

07.05.2023

170 похожих чатов

А можно код ревью попросить?) Это рендерилка для блендера которая бекенды

8 ответов

Похожие вопросы