Django视图遍历ManyToMany字段求和报错：仅获取首个查询结果

看起来你的问题主要来自两个关键点：累加变量未初始化，以及字段允许为Null导致的类型不兼容错误——你觉得循环“仅获取到第一个查询结果”，其实是因为第一次循环就触发了报错，导致代码直接中断，并不是真的只遍历了第一个关联对象。下面一步步帮你解决：

1. 修复基础错误：初始化变量+处理Null值 #

你的代码里trip_fees没有预先定义赋值，第一次循环时它的默认值是None，而None和Decimal类型的价格做加法，就会触发operand + can not be between none and decimal values错误。同时你的trip_Price_EGP设置了null=True，如果某个Trip的这个字段是Null，也会引发同样的问题。

修改后的views.py代码：

from decimal import Decimal

def PayNowTourPackageView(request, tour_id, buyer_id):
    tour = TourPackage.objects.get(id=tour_id)
    trips = tour.trips.all()
    # 初始化累加变量为Decimal类型的0，避免None值问题
    trip_fees = Decimal('0.00')
    for trip in trips:
        # 处理可能的Null值，用0替代空值
        trip_price = trip.trip_Price_EGP or Decimal('0.00')
        trip_fees += trip_price
    # 后续可以使用trip_fees完成支付逻辑

2. 更高效的方案：用Django聚合函数直接求和 #

其实完全不需要手动循环遍历，Django的Sum聚合函数可以直接在数据库层面计算总和，既高效又能避免循环中的各种问题：

from django.db.models import Sum, DecimalField
from decimal import Decimal

def PayNowTourPackageView(request, tour_id, buyer_id):
    tour = TourPackage.objects.get(id=tour_id)
    # 使用aggregate直接求和，用or处理没有关联trip时的空结果
    total_fees = tour.trips.aggregate(
        total=Sum('trip_Price_EGP', output_field=DecimalField())
    )['total'] or Decimal('0.00')
    # total_fees就是所有关联trip的价格总和

这种方式不需要写循环，数据库直接完成计算，性能更优，也从根源上避免了变量初始化和Null值的问题。

补充说明 #

你感知到的“循环仅获取第一个结果”，本质是报错导致代码执行中断——第一次循环就触发了None + Decimal的错误，循环还没来得及遍历后续对象就停止了。修复初始化和Null值问题后，循环会正常遍历所有关联的Trip对象。

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