Question

我是Django的新手，到目前为止，我的能力给我留下了深刻的印象。我正在玩更复杂的模型，我有问题正确使用它们。使用Django 1.3，我正在尝试编写一个摘要页面，它将使用以下结构显示下面的三个模型。换句话说，列出了目的地和活动的旅行。

旅行1
- 目的地1
- 目的地2
- 活动1
旅行2
- 目的地1
- 活动2

模型

旅行＆lt; - ＆gt; TripDestination＆lt; - ＆gt;目的地（旅行可以有多个目的地）
活动 - ＆gt;旅行，活动 - ＆gt;目的地（为特定地点/目的地的旅行定义活动）

    class Destination(models.Model):
        city_name=models.CharField()

    class Trip(models.Model):
        departing_on=models.DateField()
        returning_on=models.DateField()
        destinations=models.ManyToManyField(Destination)

    class Activity(models.Model):
        destination=models.ForeignKey(Destination, null=False)
        trip=models.ForeignKey(Trip, null=False)

我正在尝试编写一个视图，该视图将生成具有上述结构的页面。我现在面临的主要问题是显示特定旅行和目的地的活动。正如您在下面的代码中看到的，我正在构建一个字典，我怀疑这是正确的做法。此外，视图变为

查看

def list_trip(request, template_name = 'trip-list.html'):
    trips = Trip.objects.all()

    # Build a dictionary for activities -- Is this the right thing to do?
    activities = Activity.objects.filter(trip__in=trips)
    activities_by_trips = dict()
    for activity in activities:
        if activity.trip_id not in activities_by_trips:
            activities_by_trips[activity.trip_id] = dict()

        if activity.destination_id not in activities_by_trips[activity.trip_id]:
            activities_by_trips[activity.trip_id][activity.destination_id] = []

        activities_by_trips[activity.trip_id][activity.destination_id].append(activity)

    return render_to_response(template_name, {
        'page_title': 'List of trips',
        'trips': trips,
        'activities_by_trips': activities_by_trips,
    })

模板


{% block content %}
    {% for trip in trips %}
        {{ trip.id }} - {{ trip.name }}

        {% for destination in trip.destinations.all %}
            {{ destination.city_name }}

            ** This is terrible code -- How to fix that **
            {% for key, value in activities_by_trips|dict_lookup:trip.id %}
                {% if value %}
                    {% for key_prime, value_prime in value|dict_lookup:destination.id %}
                       {{ value_prime.description }}
                    {% endfor %}
                {% endif %}
            {% endfor %}
        {% endfor %}
    {% endfor %}
{% endblock %}

简而言之，有人可以帮助我总结所有的旅行和活动吗？实现这一目标的最佳方法是什么？模型是否正确？

谢谢！

Answer 1

还有很大的改进空间。通过在ManyToManyField上使用through，您可以明确定义连接表，我们可以方便地将其视为在特定旅行期间对城市的单次访问。在那次访问期间我们有活动，所以活动应该有一个外国访问。

对于表中的每个外键，Django将为关系的另一侧的对象集添加API便利管理器。 Destination将visit_set，Trip也是visit。同样，由于Activity中的activity_set外键，每次访问都会有from django.db import models # Create your models here. class Destination(models.Model): city_name=models.CharField(max_length=50) class Trip(models.Model): departing_on=models.DateField() returning_on=models.DateField() destinations=models.ManyToManyField(Destination, through='Visit') class Visit(models.Model): destination=models.ForeignKey(Destination) trip=models.ForeignKey(Trip) class Activity(models.Model): name=models.CharField(max_length=50) visit=models.ForeignKey(Visit)。

首先从模型开始：

list_trip

然后让我们稍微更改def list_trip(request, template_name = 'trip-list.html'): return render_to_response(template_name, { 'page_title': 'List of trips', 'trips': Trip.objects.all(), }) def print_trips(): for trip in Trip.objects.all(): for visit in trip.visit_set.select_related().all(): print trip.id, '-', visit.destination.city_name for act in visit.activity_set.all(): print act.name，添加print_trip以清楚模板中发生的事情：

{% block content %}
    {% for trip in trips %}
        {{ trip.id }} - {{ trip.name }}

        {% for visit in trip.visit_set.select_related.all %}
            {{ visit.destination.city_name }}

            {% for act in visit.activity_set.all %}
                 {{ act.name }}
            {% endfor %}
        {% endfor %}
    {% endfor %}
{% endblock %}

最后是改进的模板：

{{1}}

还有一些改进性能的改进空间。注意我使用了select_related。这将在获取访问时预取所有目标，因此visit.destination.city_name不会引发另一个db调用。但是，这不适用于反向ManyToMany关系（在我们的例子中是activity_set的所有成员）。 Django 1.4将推出名为prefetch_related的新方法，它也可以解决这个问题。

同时，请阅读Efficient reverse lookups，了解如何进一步减少数据库命中数。在评论中也提到了一些现成的解决方案。

Django和具有多个外键的模型

1 个答案: