Django电商项目：商品折扣计算与模型优化咨询

现有代码的问题 #

• 数据冗余与一致性风险：Products模型中的discount_price属于冗余存储，当商品原价price或折扣率discount变更时，若未同步更新该字段，会导致数据不一致。
• 关联命名错误：OfferProduct中OneToOneField的related_name设为category_offers，但这是商品级别的优惠，命名与实际用途不符，易造成混淆。
• 缺少自动计算逻辑：没有实现管理员添加/修改折扣时，自动计算并更新折扣价的机制。
• 字段类型不合理：使用IntegerField存储价格，无法处理带小数的金额（如9.99元），不符合电商场景需求。
• 缩进错误：Meta类、get_url方法、__str__方法的缩进不正确，不属于Products类的内部成员，会导致Django无法识别这些配置和方法。

改进方案 #

方案1：动态计算折扣价（推荐，避免数据冗余） #

去掉Products中的discount_price字段，通过动态属性实时计算，确保数据始终一致。

修改后的模型代码：

from django.db import models
from django_extensions.db.fields import AutoSlugField
from django.core.validators import MinValueValidator, MaxValueValidator
from django.urls import reverse

# 假设Category模型已定义
class Category(models.Model):
    name = models.CharField(max_length=100)
    slug = AutoSlugField(populate_from=['name'], unique=True)
    
    def __str__(self):
        return self.name

class Products(models.Model):
    product_name   = models.CharField(max_length=200, unique=True)
    slug           = AutoSlugField(populate_from=['product_name'], unique=True)
    description    = models.TextField(max_length=500, blank=True)
    # 改用DecimalField处理小数金额
    price          = models.DecimalField(
        max_digits=10, 
        decimal_places=2, 
        validators=[MinValueValidator(0.00)]
    )
    image1         = models.ImageField(upload_to='photos/Products')
    image2         = models.ImageField(upload_to='photos/Products')
    image3         = models.ImageField(upload_to='photos/Products')
    image4         = models.ImageField(upload_to='photos/Products')
    stock          = models.IntegerField(validators=[MinValueValidator(0)])
    is_available   = models.BooleanField(default=True)  # 统一小写命名规范
    category       = models.ForeignKey(Category, on_delete=models.CASCADE)
    created_date   = models.DateTimeField(auto_now_add=True)
    modified_date  = models.DateTimeField(auto_now=True)

    class Meta:
        verbose_name = 'product'
        verbose_name_plural = 'products'

    def get_url(self):
        return reverse('product_page', args=[self.category.slug, self.slug])

    # 动态计算折扣价
    @property
    def discount_price(self):
        if hasattr(self, 'product_offer') and self.product_offer.is_active:
            discount = self.product_offer.discount
            return self.price * (1 - discount / 100)
        return self.price

    def __str__(self):
        return self.product_name

class OfferProduct(models.Model):
    product = models.OneToOneField(
        Products, 
        related_name='product_offer',  # 修正关联名称
        on_delete=models.CASCADE
    )
    discount = models.IntegerField(
        validators=[MinValueValidator(0), MaxValueValidator(99)],
        default=0
    )
    is_active = models.BooleanField(default=True)

    class Meta:
        verbose_name = 'Offer Product'
        verbose_name_plural = 'Offer Products'

    def __str__(self):
        return f"{self.product.product_name} - {self.discount}% off"

方案2：保留discount_price字段，通过信号自动更新 #

如果业务场景需要持久化存储折扣价，可以使用Django的信号机制，在OfferProduct保存时自动计算并更新Products的discount_price。

添加信号处理代码（可放在models.py或单独的signals.py中）：

from django.db.models.signals import post_save
from django.dispatch import receiver

@receiver(post_save, sender=OfferProduct)
def update_product_discount_price(sender, instance, **kwargs):
    product = instance.product
    if instance.is_active:
        discount_amount = product.price * (instance.discount / 100)
        product.discount_price = product.price - discount_amount
    else:
        product.discount_price = product.price
    product.save()

同时修改Products模型的价格字段：

price = models.DecimalField(
    max_digits=10, 
    decimal_places=2, 
    validators=[MinValueValidator(0.00)]
)
discount_price = models.DecimalField(
    max_digits=10, 
    decimal_places=2, 
    validators=[MinValueValidator(0.00)],
    null=True,
    blank=True
)

额外优化点 #

• 统一命名规范：将Is_available改为is_available，符合Python PEP8命名规范。
• 优化OfferProduct的__str__方法，显示折扣信息，便于后台管理查看。
• 若使用方案1，模板中直接调用{{ product.discount_price }}即可获取实时计算的折扣价。

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