質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

87.96%

Djangoでデータベースからデータをとってくる

解決済

回答 2

投稿

  • 評価
  • クリップ 0
  • VIEW 903

score 378

 開発環境

  • Python 3.6.5
  • Django 2.1.2

 直面している問題

次のようなモデルを作って、利用しています。

from django.db import models

# Create your models here.
class Money(models.Model):
    use_date = models.DateField('日付')
    detail = models.CharField(max_length=200)
    cost = models.IntegerField(default=0)
    category = models.CharField(max_length=10)

    def __str__(self):
        return self.detail + ' ¥' + str(self.cost)

ビューを作るときにここからデータをとってこようと考えています。問題が生じているのは、特定の月の支出の合計額をデータベースから計算するコードです。

def index(request, year=TODAY[0], month=TODAY[1]):
    money = Money.objects.filter(use_date__year=year,
            use_date__month=month).order_by('use_date')
    total = 0
    for m in money:
        total += m.cost
#...

これだとうまくいきます。しかし合計額を計算する部分を関数として抽出すると途端にエラーが出てしまいます。

def calc_month_pay(money):
    total = 0
    for m in money:
        total += m.cost

    return cost


def index(request, year=TODAY[0], month=TODAY[1]):
    money = Money.objects.filter(use_date__year=year,
            use_date__month=month).order_by('use_date')
    total = calc_month_pay(money)

#...
NameError: name 'cost' is not defined

なぜでしょうか?

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 2

checkベストアンサー

0

……エラー文を読んだそのままですが、 cost という未定義の変数を導入されたから、かと思います。

def calc_month_pay(money):
    total = 0
    for m in money:
        total += m.cost

-    return cost
+    return total

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2018/11/14 00:15

    実際に上のうまく動いているほうのコードでは「m.cost」でアクセスできているのに、同じものを引数に渡した瞬間にNameErrorが出る理由が知りたいです。

    キャンセル

  • 2018/11/14 07:23

    完全に勘違いしていました。returnのほうですね。解決しました。ありがとうございます。

    キャンセル

  • 2018/11/14 10:39

    そうですか。すぐに反応ができませんでしたが……ご解決されたとのことでよかったです!

    余談ですが、エラーが出たときには

    > NameError: name 'cost' is not defined

    の近くに、エラーが出た行(今回の場合ですと return cost の行)を示す出力が出たのではないかと思います。エラーが出たときはエラー内容とあわせて該当行を必ずチェックするようにされると、問題の絞り込みがしやすくなるかと思います。ご参考になさってください。

    キャンセル

0

    return cost

この cost という変数が未定義です

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 87.96%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る