#Вопрос Https://Dpaste.Org/QAxR Посмотрите, пожалуйста, на статический метод в модели. По-хорошему, методу get_raster_image место

Question

#Вопрос Https://Dpaste.Org/QAxR Посмотрите, пожалуйста, на статический метод в модели. По-хорошему, методу get_raster_image место

в файле utils.

Да вот беда. Я боюсь импортов, как чумы. Одна возможность кольцевого импорта просто меня может парализовать. Однажды я целый проект выбросил и с нуля переписал. Просто потму что на поздней стадии не смог справиться с кольцевым импортом.

Так вот, почему метод засунут в модель - да потому что модель можно отовсюду достать через get_model. Код стал малость уродским, но я готов это пережить - лишь бы только не импорты.

Скажите, пожалуйста, это полная дрисня или нормальный прием?

#backend #django #programming #python #russian

0

20.08.2021

12 ответов

9 просмотров

Nonverbis Автор вопроса

Проблема в читаемости кода. Код стал малость уродливым. Во-первых, могут засмеять. А во-вторых, нарушена логика. Вообще-то говоря, это, конечно, не про модель. Модель работает с базой данных. Хотя, тут тоже более-менее связано с базой данных. Но вот так делать можно или это полная чушь?

0

20.08.2021

Pavel Aksenov

Только я бы сделал classmethod, а не staticmethod

0

20.08.2021

Dan Tyan

RasterImage = apps.get_model(app_label="images", model_name="RasterImage") obj = RasterImage.objects.get(pk=id)

0

20.08.2021

Nonverbis Автор вопроса

Пожалуй, пример неудачный. Речь идет об утилитах. сейчас перебью пример.

0

20.08.2021

Dan Tyan

мне кажется лучше и правильнее разобраться с импортами ну и подружиться с ними

0

20.08.2021

Nonverbis Автор вопроса

Я уже зубы об него сломал.

0

20.08.2021

Nonverbis Автор вопроса

В общем, так делать не хочется. Потому что это хардкод. Ну, хорошо - там pk. А если там было бы написано foo? А потом это foo поменяется на trololo. И я в 1000 мест буду перебивать все. В общем, пример я мог бы привести. Но это любая утилита. Вот вообще любая. Просто мне ее не хочется импортировать импортом.

0

20.08.2021

Nonverbis Автор вопроса

Можно я вам в личку напишу?

0

20.08.2021

Pavel Aksenov

Давай

0

20.08.2021

Dan Tyan

какой то ложный посыл во первых что там захардкожено ? во вторых ну хорошо у тебя в одном месте будет вызов модели в остальных вызов твоей утилиты и ты в утилите(в одном месте) меняешь поле выборки а это значит что во всех остальных местах ты должен поменять данные для выборки

0

20.08.2021

Nonverbis Автор вопроса

да, возможно, ложный посыл. спасибо за помощь. буду дальше думать.

0

20.08.2021

Dan Tyan · Accepted Answer

Dan Tyan

так а в чем проблема raster_image_model = apps.get_model(app_label="images", model_name="RasterImage") obj = raster_image_model.objects.get(pk=id)

0

20.08.2021

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

#Вопрос Https://Dpaste.Org/QAxR Посмотрите, пожалуйста, на статический метод в модели. По-хорошему, методу get_raster_image место

12 ответов

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