如何在Django ORM中优化到期画廊的筛选与删除逻辑？

首先纠正你当前实现的核心问题：硬编码日期的方式无法适配每日运行的定时任务，且代码冗余。下面提供两种更优的实现方案，优先推荐第一种基于Django ORM日期函数的方案。

方案一：使用Django内置日期函数动态计算到期时间（推荐） #

利用Django 3.2+提供的AddMonths函数，直接在数据库层面计算画廊的到期日期，无需手动构造条件：

from django.db.models import F
from django.db.models.functions import AddMonths
from datetime import date
from myapp.models import Gallery

def clean_expired_galleries():
    # 筛选条件：term不为空（非永久），且date + term个月 <= 当前日期
    expired_galleries = Gallery.objects.filter(
        term__isnull=False,
        AddMonths(F('date'), F('term')) <= date.today()
    )
    # 批量删除到期画廊
    expired_galleries.delete()

说明： #

• F('date')和F('term')直接引用数据库字段，避免将数据加载到内存计算，性能更优
• AddMonths函数在数据库层面完成日期运算，兼容不同数据库的日期处理逻辑
• 自动适配所有TERMS_CHOICES中的期限值，无需手动维护条件列表

方案二：生成式动态拼接Q对象（适配旧版Django） #

如果你的Django版本低于3.2，无法使用AddMonths，可以用生成式结合reduce动态拼接Q对象：

from functools import reduce
from operator import or_
from datetime import date
from dateutil.relativedelta import relativedelta
from django.db.models import Q
from myapp.models import Gallery

def clean_expired_galleries():
    # 获取所有可选的期限值
    term_options = [choice[0] for choice in Gallery.TERMS_CHOICES]
    
    # 生成每个期限对应的筛选条件：date <= 当前日期 - term个月
    condition_list = [
        Q(term=term, date__lte=date.today() - relativedelta(months=term))
        for term in term_options
    ]
    
    # 用reduce组合所有OR条件，再加上term不为空的限制
    expired_galleries = Gallery.objects.filter(
        term__isnull=False,
        reduce(or_, condition_list)
    )
    expired_galleries.delete()

说明： #

• 使用dateutil.relativedelta处理月份偏移（需先安装python-dateutil：pip install python-dateutil）
• 通过列表生成式自动生成所有期限的条件，避免手动逐个拼接Q对象
• reduce(or_, ...)将多个Q对象用OR逻辑组合，等价于你原来的Q(**data[0]) | Q(**data[1])...写法

额外优化：修复模型date字段的默认值 #

你当前模型中date字段的默认值存在问题：datetime.now().strftime('%Y-%m-%d')会在项目启动时计算一次，后续创建的画廊都会使用这个固定日期。建议修改为：

from datetime import date

class Gallery(models.Model):
    # ... 其他字段 ...
    date = models.DateField(default=date.today, editable=False)
    # 或者用auto_now_add自动设置创建日期：
    # date = models.DateField(auto_now_add=True, editable=False)

内容的提问来源于stack exchange，提问作者Anna Berezko