19

ニュースレターに各号に複数の記事が含まれるニュースレターアプリケーションがあります。ニュースレターの年、巻、ラベルをリストした要約ページをオンラインで表示し、次に、その号のすべての記事を順不同のリストで表示したいと思います。私はDjangoを初めて使用するので、これを行うための最良の方法を決定しようとしています。

モデルを定義しました（関連する部分のみ）：

Models.py：

class Newsletter(models.Model):
    volume = models.ForeignKey(Volume)
    year   = models.IntegerField()
    season = models.CharField(max_length=6, choices=VOLUME_SEASON)
    label  = models.CharField(max_length=20)
    number = models.IntegerField()

class Article(models.Model):
    newsletter = models.ForeignKey(Newsletter)
    section    = models.ForeignKey(Section)
    title      = models.CharField(max_length=200)

私がウェブ上で見たいものは次のようになります：

<h2>Spring 2012</h2>
<p>Volume 14, Number 1</p>
<ul>
    <li>Foo</li>
    <li>Bar</li>
    <li>Baz</li>
</ul>

<h2>Winter 2011</h2>
<p>Volume 13, Number 4</p>
<ul>
  <li>Boffo</li>
</ul>

ものすごく単純。しかし、私は自分の意見を書くための最良の方法に混乱しています。使用するかどうか：

  • zip()テンプレートで繰り返し処理する2つのリスト
  • select_related()クエリセットを使用する
  • prefetch_related()クエリセットを使用する

私はそれを最初のオプションを使用して動作させています：

Views.py：

from django.shortcuts import render_to_response, get_object_or_404
from www.apps.newsletter.models import Newsletter, Article

def index(request):
    article_group = []
    newsletter = Newsletter.objects.all().order_by('-year', '-number')
    for n in newsletter:
        article_group.append(n.article_set.all())
    articles_per_newsletter = zip(newsletter, article_group)

    return render_to_response('newsletter/newsletter_list.html',
                              {'newsletter_list': articles_per_newsletter})

次に、次のテンプレートを使用してレンダリングします。

Newsletter_list.html：

{% block content %}
  {% for newsletter, articles in newsletter_list %}
    <h2>{{ newsletter.label }}</h2>
    <p>Volume {{ newsletter.volume }}, Number {{ newsletter.number }}</p>
    <p>{{ newsletter.article }}</p>
    <ul>
    {% for a in articles %}
      <li>{{ a.title }}</li>
    {% endfor %}
    </ul>
  {% endfor %}
{% endblock %}

非常に簡単ですが、私はDjangoにかなり慣れていないので、私が行っていることは、その強力なORMの点で完全に非効率的であるかどうか疑問に思いました。zip()より速い方法があれば、オンザフライでリストを作成してから、2つのリストを一緒に作成する必要はありません。

TIA。

4

1 に答える 1

49

現在行っているアプローチは、クエリ数が1 + Nになるため、非常に非効率的です。n.article_set.all()つまり、すべてのニュースレターのクエリに対して1つ、次にそれらの結果を評価するたびに1つです。したがって、最初のクエリに100個のNewletterオブジェクトがある場合、101個のクエリを実行することになります。

これは、を使用する優れた理由prefetch_relatedです。結果は2つのクエリになります。1つはニュースレターを取得し、もう1つは関連記事をバッチで取得します。あなたはまだzipそれらを整理するために完全にやり続けることができますが、それらはすでにキャッシュされているので、実際にはクエリを直接テンプレートに渡してそれをループすることができます。：

見る

newsletters = Newsletter.objects.prefetch_related('article_set').all()\
                    .order_by('-year', '-number')

return render_to_response('newsletter/newsletter_list.html',
                          {'newsletter_list': newsletters})

レンプレート

{% block content %}
  {% for newsletter in newsletter_list %}
    <h2>{{ newsletter.label }}</h2>
    <p>Volume {{ newsletter.volume }}, Number {{ newsletter.number }}</p>
    <p>{{ newsletter.article }}</p>
    <ul>
    {% for a in newsletter.article_set.all %}
      <li>{{ a.title }}</li>
    {% endfor %}
    </ul>
  {% endfor %}
{% endblock %}
于 2012-08-18T00:53:01.500 に答える