django 查询、反向查询和谓词中的 DRY

Question

让我感到沮丧的是，在 Django 中，我经常不得不在自定义管理器上编写方法：

class EntryManager(Manager):
    def filter_beatle(self, beatle):
        return self.filter(headline__contains=beatle)

... 并在不同的管理器中重复几乎相同的方法进行反向查询：

class BlogManager(Manager):
    def filter_beatle(self, beatle):
        return self.filter(entry__headline__contains=beatle)

... 以及 Entry 上的谓词：

def headline_contains(self, beatle):
    return self.headline.find(beatle) != -1

[注意Entry 上的谓词将适用于尚未保存的 Entry 对象。]

这感觉像是违反了 DRY。有没有什么方法可以表达一次并在所有三个地方使用它？

我想要做的是写一些类似的东西：

q = Q(headline__contains="Lennon")
lennon_entries = Entry.objects.filter(q)
lennon_blogs = Blog.objects.filter(q.reverse(Entry))
is_lennon = entry.would_filter(q)

... 其中 'headline__contains="Lennon"' 恰好表达了一次“关于“Lennon”的条目”的含义，这可以用来构造反向查询和谓词。

Answer 1

最好的地方是自定义管理器。根据 django 的指南，管理器类是影响类中多个对象的代码的最佳位置。

class EntryManager(models.Manager):
    def filter_lennons(self):
        return self.get_query_set().filter(headline__contains='Lennon')

class Entry(models.Model):
    headline = models.CharField(max_length=100)

    objects = EntryManager()

lennons = Entry.objects.filter_lennons()

Answer 2

您应该永远很少需要执行以下操作：

if entry.headline.find('Lennon') >= 0:

因为过滤器应该负责将结果集限制为您感兴趣的实例。

如果您要多次使用相同的过滤器，您可以创建一个 自定义管理器 或一个简单的类方法。

class Entry(models.Model):
    ...
    # this really should be on a custom manager, but this was quicker to demonstrate
    @classmethod
    def find_headlines(cls, text):
        return cls.objects.filter(headline__contains=text)

entries = Entry.find_headlines('Lennon')

但实际上，DRYness 已经包含在 Queryset API 中。您真正需要将字符串“Lennon”硬编码到查询中的频率是多少？通常，搜索参数将从 GET 或 POST 传递到视图中。完美干燥。

那么，真正的问题是什么？除了探索查询集 API 之外，您是否曾经需要在多个查询（如您的问题）中对查找值进行硬编码？

Answer 3

对于“反向过滤器”情况，您可以使用子查询：

Blog.objects.filter(entries__in=Entry.objects.filter_beatle("Lennon"))

重用或生成谓词是不可能的（通常），因为有些谓词无法表示为查询，而查询则无法表示为没有数据库访问的谓词。

Answer 4

我对谓词最常见的用法似乎是在断言中。通常是这样的：

class Thing(Model):
    class QuerySet(query.QuerySet):
        def need_to_be_whacked():
            # ... code ...

    def needs_to_be_whacked(self):
        return Thing.objects.need_to_be_whacked().filter(id=self.id).exists()

    def whack(self):
        assert self.needs_to_be_whacked()

for thing in Thing.objects.need_to_be_whacked():
    thing.whack()

我想确保没有其他代码在不需要被敲击的状态下调用 whack() 。虽然它会影响数据库，但它确实有效。