Блог

Вот два примера моделей Django. Обратите особое внимание на метод has_pet.

 class Person(models.Model):
    name = models.CharField(max_length=255)

    def has_pet(self):
        return bool(self.pets.all().only('id'))

class Pet(models.Model):
    name = models.CharField(max_length=255)
    owner = models.ForeignKey(Person, blank=True, null=True, related_name="pets")
  

Проблема здесь в том, что метод has_pet всегда генерирует запрос. Если вы сделаете что-то подобное.

 p = Person.objects.get(id=1)
if p.has_pet():
    ...
  

Тогда вы фактически будете выполнять дополнительный запрос, просто чтобы проверить, есть ли у одного человека домашнее животное. Это большая проблема, если вам приходится проверять нескольких человек. Он также будет генерировать запросы, если используется в шаблонах, подобных этому.

 {% for person in persons %}
    {% if person.has_pet %}
        {{ person.name }} owns a pet
    {% else %}
        {{ person.name }} is petless
    {% endif %}
{% endfor %}
  

Этот пример фактически выполнит дополнительный запрос для каждого пользователя в наборе запросов persons во время рендеринга шаблона.

Есть ли способ сделать это всего одним запросом или, по крайней мере, выполнить меньше одного дополнительного запроса на человека? Возможно, есть другой способ спроектировать это, чтобы вообще избежать проблемы.

Я подумал о добавлении логического поля к Person и о том, чтобы это поле обновлялось всякий раз, когда домашнее животное сохраняется или удаляется. Это действительно правильный путь?

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

Если вам нужен список всех людей с домашними животными, вы можете сделать это в одном запросе:

 Person.objects.exclude(pets=None)
  

Похоже, вы хотите выполнить итерацию по одному списку пользователей, использование аннотаций, вероятно, имело бы смысл:

 for person in Person.objects.annotate(has_pet=Count('pets')):
     if person.has_pet: # if has_pet is > 0 this is True, no extra query
  

Было бы неплохо, если бы в Django был Exists агрегат, но его нет, и я не знаю, насколько сложно было бы его добавить. Вы, конечно, должны профилировать и выяснить, работает ли это для вас.

Лично я бы, вероятно, просто сохранил has_pets в качестве логического значения в модели, это, вероятно, наиболее эффективный подход.