2012-02-26 48 views
3

我是Django的新手,至今我的能力给我留下了深刻的印象。我正在玩更复杂的模型,我有问题正确使用它们。使用Django 1.3,我正在尝试编写一个摘要页面,它将使用以下结构呈现下面的三个模型。换句话说,一个旅行目的地和活动的列表。Django和具有多个外键的模型

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

模型

  • 旅行< - > TripDestination < - >目标(一个行程可以有多个目的地)
  • 活动 - >跳闸,活动 - >目标(一个活动是定义为在特定位置/目的地旅行)
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 %} 

简单地说,有人可以帮我把所有的行程和活动的总结?什么是最好的方式来实现这一目标?模型是否正确?

谢谢!

回答

13

有很大的改进空间。通过在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了一下,加print_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 

最后改进的模板:

{% 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 %} 

还有一些提升空间性能明智。注意我使用了select_related。这将在访问访存时预取所有目标,以便visit.destination.city_name不会引发另一个数据库调用。但是,这不适用于反向ManyToMany关系(在我们的例子中是activity_set的所有成员)。 Django 1.4将推出一种名为prefetch_related的新方法,它也可以解决这个问题。

与此同时,请阅读Efficient reverse lookups了解如何进一步减少数据库命中数。在评论中,也提到了几种现成可用的解决方案。

+0

谢谢digivampire!这正是我所期待的。 – Martin 2012-02-27 22:08:03