0

У меня есть 2 файла с моделями. Пусть будет models_a и models_b. И есть функция, которая выполняет похожие действия с этими моделями. Поэтому функция на все модели одна. К какой именно модели нужно применить эти действия, я узнаю по названию модели, которое попадает в функцию в виде строки. И мне нужно по этому названию получить нужную модель. Я сделал это вот так

from models_a import *
from models_b import *

def my_func(model_name):
    model = globals().get(model_name)
    qs = model.objects.all() 

В принципе, это работает. Подскажите - сильно ли плох этот вариант? Как эффективнее можно решить такую задачу?

10
  • Зачем globals использовать? Можно саму модель в качестве аргумента передать вроде как. Commented 13 нояб. 2019 в 9:48
  • @МихаилАлексеевич ну, может и можно, но мне этот вариант не подходит. Мне нужно получить модель по ее названию. Commented 13 нояб. 2019 в 9:53
  • 1
    Ничего плохого в этом варианте нет. А учитывая, что он не использует "приватные" методы языка - очевидно, что данная реализация близка к рекомендуемой под Вашу задачу. Commented 13 нояб. 2019 в 9:56
  • Где то в исходниках самой Django видел подобные конструкции, это признак "правильности" имхо :) Commented 13 нояб. 2019 в 9:57
  • Есть метод get_model для этого. Commented 13 нояб. 2019 в 10:07

1 ответ 1

1

Способ 1

import sys

from django.apps import apps


def my_func(request, model_name):
    Model = apps.get_model(app_label=sys.modules[resolve(request.path_info).func.__module__].__package__, model_name=model_name)

Способ 2 - работает намного быстрее и является предпочтительным.

  1. Добавить в <app>.urls.py строку
app_name='<app_name>'
  1. В функции
from django.apps import apps


def my_func(request, model_name):
    Model = apps.get_model(request.resolver_match.app_name, model_name)    

Проверил Ваш метод. Он работает быстрее всего, но что-то мне подсказывает, что это не самое лучшее решение.

4
  • ____module__.__package__ не есть хороший стиль. Commented 14 нояб. 2019 в 6:57
  • @МихаилАлексеевич поэтому я и советую второй вариант. Commented 14 нояб. 2019 в 10:28
  • вроде как apps.get_model работает через то же global() внутри, а ввиду того, что это еще дополнительный вызов - не факт, что такой код оптимальнее того, что в вопросе в части производительности. Commented 14 нояб. 2019 в 10:31
  • Вроде как, apps.get_model не работает через global() - github.com/django/django/blob/master/django/apps/config.py#L167 Commented 14 нояб. 2019 в 10:50

Ваш ответ

Нажимая «Отправить ответ», вы соглашаетесь с условиями пользования и подтверждаете, что прочитали политику конфиденциальности.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.