Django问答应用金币流转异常排查：提问者金币已扣除但回答者未获对应金币

首先，你遇到的核心问题非常明确：Answer模型根本没有coins_given字段，但在回答的信号处理函数里却尝试访问instance.coins_given，这会直接抛出AttributeError，导致回答者的金币增加逻辑完全执行失败，同时提问者的金币已经完成扣除（因为提问的信号函数逻辑是正常的）。

1. 核心错误修复 #

把回答者加金币的信号函数修改为从关联的Question对象中获取金币数额，这才是正确的数据源：

@receiver(post_save, sender=Answer)
def add_coins_to_user(sender, instance, created, **kwargs):
    if created:
        # 从当前回答关联的Question实例中读取预设金币数
        coins_amount = instance.question.coins_given
        if coins_amount is not None:  # 提前判断避免空值异常
            profile = instance.user
            profile.coin += coins_amount
            profile.save()

2. 后续优化建议 #

除了修复当前报错，还有几个细节可以优化，避免后续出现更多业务逻辑问题：

（1）用数据库事务保证金币流转的原子性 #

金币增减属于敏感的资金类操作，必须保证操作的原子性——要么全部成功，要么全部回滚。可以用Django的事务装饰器包裹信号函数：

from django.db import transaction

# 提问扣金币的函数加上事务
@receiver(post_save, sender=Question)
@transaction.atomic
def subtract_coins_from_user(sender, instance, created, **kwargs):
    if created:
        if instance.coins_given is None:
            raise ValueError("提问必须设置金币数额")
        # 提前检查用户金币是否足够
        if instance.user.coin < instance.coins_given:
            raise ValueError("用户金币不足，无法发起提问")
        profile = instance.user
        profile.coin -= instance.coins_given
        profile.save()

# 回答加金币的函数同样加上事务
@receiver(post_save, sender=Answer)
@transaction.atomic
def add_coins_to_user(sender, instance, created, **kwargs):
    if created:
        coins_amount = instance.question.coins_given
        if coins_amount is not None:
            profile = instance.user
            profile.coin += coins_amount
            profile.save()

（2）修正Question模型的字段约束 #

你的Question模型中coins_given设置了blank=False, null=True，这会导致数据库允许该字段为NULL，但表单提交时又强制要求填写，逻辑矛盾。如果金币是提问的必填项，建议改为：

coins_given = models.PositiveSmallIntegerField(choices=COINS_CHOICES, blank=False, null=False)

从数据库层面强制每个提问都有明确的金币数额，避免后续处理时出现空值问题。

（3）明确多回答场景的业务规则 #

如果你的业务规则是每个回答问题的用户都能获得金币，当前修复后的逻辑没问题；但如果是只有第一个回答的用户能获得金币，需要在信号函数中添加判断：

@receiver(post_save, sender=Answer)
@transaction.atomic
def add_coins_to_user(sender, instance, created, **kwargs):
    if created:
        question = instance.question
        # 检查当前回答是否是该问题的第一个回答
        if Answer.objects.filter(question=question).count() == 1:
            coins_amount = question.coins_given
            if coins_amount is not None:
                profile = instance.user
                profile.coin += coins_amount
                profile.save()

3. 验证错误来源 #

你可以查看Django的控制台日志或错误追踪信息，应该能看到类似'Answer' object has no attribute 'coins_given'的报错提示，这也能直接印证我们的分析结论。

内容的提问来源于stack exchange，提问作者obodoma uzondu vincent