Django Rest Framework中Cart序列化器price字段计算异常求助

你的CartSerializer中price字段返回0的核心原因是：

• 你在get_price方法里用了obj.unit_price计算，但数据库中Cart的unit_price字段默认值是0，且创建Cart时没有同步关联MenuItem的price值到该字段。
• 虽然你在Serializer里把unit_price设为ReadOnlyField(source="menuitem.price")，但这只影响返回给前端的展示值，并不会修改数据库中Cart的unit_price字段数据。

方案1：直接用关联MenuItem的price计算总价（最简单） #

修改CartSerializer的get_price方法，跳过数据库中未更新的unit_price，直接从关联的MenuItem取价格计算：

class CartSerializer(serializers.ModelSerializer):
    # ... 其他字段定义
    price = SerializerMethodField()

    def get_price(self, obj):
        # 直接使用关联MenuItem的price计算
        return obj.quantity * obj.menuitem.price

方案2：同步Cart的unit_price和price到数据库（如果需要持久化这些值） #

如果需要把unit_price和price的值存在数据库里，可以在Serializer的创建方法中自动赋值：

class CartSerializer(serializers.ModelSerializer):
    # ... 其他字段定义
    unit_price = serializers.DecimalField(max_digits=6, decimal_places=2, read_only=True)
    price = serializers.DecimalField(max_digits=6, decimal_places=2, read_only=True)

    def create(self, validated_data):
        # 获取选中的MenuItem
        menuitem = validated_data['menuitem']
        # 同步unit_price为MenuItem的价格
        validated_data['unit_price'] = menuitem.price
        # 计算总价
        validated_data['price'] = validated_data['quantity'] * menuitem.price
        return super().create(validated_data)

或者更彻底的方式，重写Cart模型的save方法，确保每次保存时自动同步价格：

class Cart(models.Model):
    user = models.ForeignKey(User, on_delete=models.CASCADE)
    menuitem = models.ForeignKey(MenuItem, on_delete=models.CASCADE)
    quantity = models.SmallIntegerField()
    unit_price = models.DecimalField(max_digits=6, decimal_places=2)
    price = models.DecimalField(max_digits=6, decimal_places=2)

    class Meta:
        unique_together = ('menuitem', 'user')

    def save(self, *args, **kwargs):
        # 每次保存时自动同步价格
        self.unit_price = self.menuitem.price
        self.price = self.quantity * self.unit_price
        super().save(*args, **kwargs)

方案3：移除冗余字段（推荐） #

既然unit_price完全依赖MenuItem的price，price又可以通过quantity * menuitem.price计算，完全可以把这两个字段从Cart模型中移除，避免冗余数据：

class Cart(models.Model):
    user = models.ForeignKey(User, on_delete=models.CASCADE)
    menuitem = models.ForeignKey(MenuItem, on_delete=models.CASCADE)
    quantity = models.SmallIntegerField()

    class Meta:
        unique_together = ('menuitem', 'user')

然后在Serializer中动态返回这两个值：

class CartSerializer(serializers.ModelSerializer):
    user = serializers.HiddenField(default=serializers.CurrentUserDefault())
    unit_price = serializers.DecimalField(source="menuitem.price", max_digits=6, decimal_places=2, read_only=True)
    price = SerializerMethodField()

    class Meta:
        model = Cart
        fields = ['user', 'menuitem', 'unit_price', 'quantity', 'price']

    def get_price(self, obj):
        return obj.quantity * obj.menuitem.price

这样既减少了数据库冗余，又避免了数据不一致的问题。

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