Django模板for循环内if条件判断异常问题求助（CS50w项目2）

错误原因 #

你在模板中使用top_bids.listing.id的方式完全错误。top_bids是Python字典，键是Listing的id（整数类型），但Django模板的点语法top_bids.listing.id会尝试访问字典的listing键，再访问该值的id属性——这和你要通过当前循环的listing id获取对应出价的逻辑完全不符，所以永远得不到正确结果。

解决方案（推荐：优化视图逻辑） #

直接给每个Listing对象附加top_bid属性，模板中直接访问对象属性更简洁高效：

修改views.py的index函数 #

def index(request):
    listings = Listings.objects.all()
    # 遍历每个listing，添加最高出价属性
    for listing in listings:
        # 获取当前listing的最高出价，没有则设为None
        top_bid_obj = Bids.objects.filter(listing=listing).order_by("-amount").first()
        listing.top_bid = top_bid_obj.amount if top_bid_obj else None
    return render(request, "auctions/index.html", {
        "listings": listings
    })

修改模板中的判断逻辑 #

{% extends "auctions/layout.html" %}
{% block body %}
    <h2>Active Listings</h2>
    <div class="container">
    {% for listing in listings %}
    <div class="card mb-3 mt-4">
        <div class="row g-0 align-items-center">
            <div class="col-md-4" style="max-width: 300px; max-height: 300px;">
                <img src="{{listing.image_url}}" class="img-fluid rounded-start" alt="{{listing.title}}">
            </div>
            <div class="col-md-8 ">
                <div class="card-body">
                    <h3 class="card-title">{{listing.title}}</h3>
                    <p class="card-text">{{listing.description}}</p>
                    {% if listing.top_bid is None %}
                    <p class="card-text">Start Price: ${{listing.start_bid}}</p>
                   {% else %}
                    <p class="card-text">Top Bid: ${{ listing.top_bid }}</p>
                    {% endif %}
                 <p class="card-text"><small class="text-body-secondary">Created: {{listing.date_created}}</small></p>
                    <div class="mt-4">
                        <a href=""><button class="btn btn-primary">View</button></a>
                    </div>
                </div>
            </div>
        </div>
    </div>
    {% endfor %}
    </div>
{% endblock %}

备选方案：使用自定义过滤器访问字典 #

如果不想修改视图逻辑，可以通过自定义模板过滤器实现字典键值访问：

1. 在你的App目录下创建templatetags文件夹，添加__init__.py和custom_filters.py文件：

# custom_filters.py
from django import template

register = template.Library()

@register.filter
def get_item(dictionary, key):
    return dictionary.get(key)

2. 在模板顶部加载过滤器，修改判断逻辑：

{% extends "auctions/layout.html" %}
{% load custom_filters %} <!-- 加载自定义过滤器 -->
{% block body %}
    <h2>Active Listings</h2>
    <div class="container">
    {% for listing in listings %}
    <div class="card mb-3 mt-4">
        <!-- 其他代码不变 -->
        <div class="card-body">
            <!-- 其他代码不变 -->
            {% if top_bids|get_item:listing.id is None %}
            <p class="card-text">Start Price: ${{listing.start_bid}}</p>
           {% else %}
            <p class="card-text">Top Bid: ${{ top_bids|get_item:listing.id }}</p>
            {% endif %}
            <!-- 其他代码不变 -->
        </div>
    </div>
    {% endfor %}
    </div>
{% endblock %}

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