Django RawQueryset参数绑定错误及ORM优化实践

sizes = Variants.objects.raw("SELECT * FROM core_variants WHERE product_id=%s GROUP BY size_id",[id])

# views.py
if product.variant is not None: # 优化了条件判断，使用 is not None 更符合Python习惯
    variants = Variants.objects.filter(product_id=product.id)
    colors = Variants.objects.filter(
        product_id=product.id, size_id=variants[0].size_id
    )
    sizes = Variants.objects.raw(
        'SELECT * FROM core_variants WHERE product_id=%s GROUP BY size_id',
        [product.id], # 将 id 替换为 product.id
    )
    variant = Variants.objects.get(id=variants[0].id)
context = {
    'sizes': sizes,
    'colors': colors,
    'variant': variant,
}

return render(request, 'core/product-details.html', context)

# 假设 Variants 模型中 size_id 是指向 Size 模型的 ForeignKey
# 获取与当前产品关联的所有不重复的尺寸对象
from core.models import Size # 假设 Size 模型在 core 应用中

# 方案一：通过 Variants 反向查询 Size
# 这种方式直接获取 Size 对象，更符合语义
distinct_sizes = Size.objects.filter(variants__product_id=product.id).distinct()

# 方案二：如果只需要 size_id 和 size_id.title，可以通过 Variants 模型进行聚合
# 注意：这种方式返回的是字典列表，不是 Variants 对象
# 如果 size_id 是 ForeignKey，则 size_id__title 可以访问关联 Size 模型的 title 字段
distinct_sizes_data = Variants.objects.filter(product_id=product.id)\
                                  .order_by('size_id__title')\
                                  .values('size_id', 'size_id__title')\
                                  .distinct()

# 如果 HTML 模板期望的是一个拥有 .size_id 和 .size_id.title 属性的对象
# 可以考虑创建一个简单的DTO (Data Transfer Object) 或直接使用 distinct_sizes
# 如果 distinct_sizes 是 Size 对象的 QuerySet，则在模板中 rs.id 和 rs.title 即可
# 假设 Size 模型有 id 和 title 字段
# 此时在模板中循环 distinct_sizes，使用 rs.id 和 rs.title

<select name="size" id="size" class="form-control">
    {% for rs in sizes %} {# 这里的 rs 现在是 Size 对象 #}
    <option {% if variant.size_id == rs.id %}selected{% endif %} value="{{rs.id}}">{{rs.title}}</option>
    {% endfor %}
</select>

# views.py 优化后的代码示例
if product.variant is not None:
    # 一次性获取所有与产品相关的变体
    all_product_variants = Variants.objects.filter(product_id=product.id)

    if all_product_variants.exists(): # 确保有变体存在，避免索引错误
        # 获取第一个变体作为基准
        first_variant = all_product_variants.first()

        # 根据第一个变体的 size_id 过滤颜色
        colors = all_product_variants.filter(size_id=first_variant.size_id)

        # 获取去重尺寸信息（假设 size_id 是 ForeignKey 到 Size 模型）
        # 这里使用 ORM 获取 Size 对象，更符合 Pythonic 风格
        from core.models import Size # 假设 Size 模型在 core 应用中
        sizes = Size.objects.filter(variants__product_id=product.id).distinct()

        # 获取第一个变体对象
        variant = first_variant
    else:
        # 处理没有变体的情况，例如设置默认值或跳转
        colors = []
        sizes = []
        variant = None # 或者一个默认的 Variants 对象

context = {
    'sizes': sizes,
    'colors': colors,
    'variant': variant,
}

return render(request, 'core/product-details.html', context)