Django/Python求和报错：只能将元组与元组拼接，不能拼接整数

问题背景 #

开发待办清单系统时遵循以下规则：

• 每次创建ValidationModel对象验证Action，该Action得1分
• 当Task下所有Action都得1分，判定Task完成

在分数累加环节循环报错，错误信息：can only concatenate tuple (not "int") to tuple

现有代码 #

模型代码 #

class Action(models.Model):
    name = models.CharField(verbose_name="Name",max_length=100, blank=True)

class ValidationModel(models.Model):
    user = models.ForeignKey(UserProfile, blank=True, null=True, on_delete=models.CASCADE)
    venue = models.ForeignKey(Action, blank=True, null=True, on_delete=models.CASCADE)
    created_at = models.DateTimeField(auto_now_add=True, null=True, blank=True)

class Task(models.Model):
    title = models.CharField(verbose_name="title",max_length=100, null=True, blank=True)
    venue = models.ManyToManyField(Action, blank=True)
    created_at = models.DateTimeField(auto_now_add=True, null=True, blank=True)
    
class TaskAccepted(models.Model):
    name = models.ForeignKey(Task,null=True, blank=True, on_delete=models.SET_NULL, related_name='task_accepted')
    user = models.ForeignKey(User, null=True, blank=True, on_delete=models.CASCADE)
    accepted_on = models.DateTimeField(auto_now_add=True, null=True, blank=True)

视图代码 #

def function(request, taskaccepted_id):
    instance_1 = Action.objects.filter(task__id=taskaccepted_id)
    action_count = instance_1.count()
    instance_2 = get_object_or_404(Task, pk=taskaccepted_id)
sum_completed =()

    for action in instance1:
        
        for in_action in action.validationmodel_set.all()[:1]:
            latest_point = in_action.created_at
            action_completed = in_action
            if latest_point > instance_2.accepted_on:
                action_completed = 1   
            else:
                action_completed = 0
            sum_completed += venue_completed #<-- 报错位置

错误原因 #

1. 变量类型错误：sum_completed = () 将求和变量初始化为元组，元组是不可变类型，使用+=时会尝试拼接元组，但要添加的是整数，触发类型错误。
2. 变量名笔误：代码中instance1应为instance_1，venue_completed应为action_completed，属于低级拼写错误。
3. 逻辑冗余：循环嵌套for in_action in action.validationmodel_set.all()[:1]可简化为直接取最新一条记录，无需循环。

基础修复方案 #

修正变量类型和笔误，简化逻辑：

def function(request, taskaccepted_id):
    instance_1 = Action.objects.filter(task__id=taskaccepted_id)
    action_count = instance_1.count()
    task = get_object_or_404(Task, pk=taskaccepted_id)
    sum_completed = 0  # 初始化为整数0，而非元组

    for action in instance_1:
        # 取该Action最新的一条验证记录
        latest_validation = action.validationmodel_set.order_by('-created_at').first()
        # 验证记录存在且创建时间晚于任务接受时间，加1分
        if latest_validation and latest_validation.created_at > task.accepted_on:
            sum_completed += 1
    
    # 判断任务是否完成
    is_task_completed = sum_completed == action_count

更优实现方案（数据库层面优化） #

上述循环会多次访问数据库（每个Action一次查询），数据量大时效率低下。可通过Django关联查询直接统计符合条件的Action数量，减少数据库访问次数：

def function(request, taskaccepted_id):
    task = get_object_or_404(Task, pk=taskaccepted_id)
    # 获取任务下所有Action的总数
    total_actions = task.venue.count()
    # 统计满足条件的Action数量：存在验证记录且验证时间晚于任务接受时间
    completed_actions = Action.objects.filter(
        task__id=taskaccepted_id,
        validationmodel__created_at__gt=task.accepted_on
    ).distinct().count()
    
    is_task_completed = completed_actions == total_actions

额外建议 #

• 模型中ValidationModel的外键venue命名不直观，建议改为action，避免与Task的venue字段混淆，提升代码可读性。
• 可在Action模型中添加is_completed属性或方法，封装验证逻辑，避免在视图中写过多业务代码。

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