【Django】views.pyにおいて、filterで絞り込んだクエリセットの合計値を算出したい

Question

Djangoで工事台帳を作成しており、
工事ごとの請負金額を取りまとめる表を作成したいと考えています。

例えば、下記のように、当初、追加1、追加2を入力すると（Adminsiteで入力し、これはできています。）、計を計算して表示されるようにしたいです。
当初 10,000,000円
追加1 2,000,000円
追加2 1,000,000円
計 13,000,000円

model.pyは、下記の通りで問題ないと思います。

#models.py
# -*- coding: utf-8 -*-
from django.db import models
class ContractPrice(models.Model):
    site_id = models.ForeignKey(Site)    
    price = models.IntegerField(u'税込金額')

views.pyがうまくいっていないと思います。

#views.py
# -*- coding: utf-8 -*-
from django.shortcuts import render_to_response, get_object_or_404
from django.http import HttpResponse
from django.template import RequestContext
from cls.models import ContractPrice, Owner, Site

def site_detail(request, id):
    if id:
	site = get_object_or_404(Site, pk = id)
    else:
	id = Site()
    site = Site.objects.filter(id = id)
    prices = ContractPrice.objects.filter(site_id = id)
    totalsales = sum(prices.price)
    return render_to_response('cls/site_detail.html',
	{'site' : site, 'contractprice' : price, 'totalsales' : totalsales},
    	context_instance = RequestContext(request)
    )

上記のように、
totalsales という変数に、請負金額の合計を代入するために、
totalsales = sum(prices.price)
としました。
pricesは、idで絞り込んだ要素を意味するので、そのpriceアトリビュートを合計すればよいと思ったためです。
しかし、
「AttributeError
'QuerySet' object has no attribute 'price'」
というエラーが出ます。
sumの使い方が誤っているのかと思い、外してみましたが、結果は同じでした。

ちなみに、annotateを使えば解決するのかと思い、
totalsales = prices.annotate(Sum('price'))
としてみましたが、
「NameError
global name 'Sum' is not defined」
というエラーが出ました。

私の考え方のどこかが誤っているのだと思いますが、お分かりの方、ご教示いただけないでしょうか。
お手数をお掛けしますが、よろしくお願いいたします。

Answer 1

先ほどURLについて回答させていただいた者です。
また回答させていただきたいと思います。

今回のエラーは「QuerySetにはpriceという属性はありません」という事だと思います。
この「QuerySet」は簡単いうと「1つ1つのモデルインスタンス（データベースでいうレコードだと思ってください）をリストに格納したもの」だと思ってください。
「prices = ContractPrice.objects.filter(site_id = id)」という箇所で使用している「filter」メソッドがQuerySetを返すメソッドになります。

よって、そのまま変数pricesに「.price」と指定しても、リストに属性を指定しているようなものなので上記のエラーが発生したと思います。

修正箇所は以下のようになると思います。

totalsales = sum([p.price for p in prices])

for文を使用し、1つ1つモデルのインスタンスを取り出し、その属性であるpriceを格納したリストを元にsum関数で合計を出すようにすると大丈夫だと思います。

今回は少々説明が難しくになってしまい、わかりづらいかと思いますが、もし解決できなかった場合はまたコメントいただければと思います。

よろしくお願いいたします。

Comments

[mulberryfields]

ありがとうございました。
ご指摘頂いたとおりでうまくいきました。
丁寧にご説明いただき助かりました。