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

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

新規登録して質問してみよう
ただいま回答率
85.50%
Python 3.x

Python 3はPythonプログラミング言語の最新バージョンであり、2008年12月3日にリリースされました。

Python

Pythonは、コードの読みやすさが特徴的なプログラミング言語の1つです。 強い型付け、動的型付けに対応しており、後方互換性がないバージョン2系とバージョン3系が使用されています。 商用製品の開発にも無料で使用でき、OSだけでなく仮想環境にも対応。Unicodeによる文字列操作をサポートしているため、日本語処理も標準で可能です。

Q&A

解決済

1回答

207閲覧

python classとforの組み合わせ

MasaKoba

総合スコア18

Python 3.x

Python 3はPythonプログラミング言語の最新バージョンであり、2008年12月3日にリリースされました。

Python

Pythonは、コードの読みやすさが特徴的なプログラミング言語の1つです。 強い型付け、動的型付けに対応しており、後方互換性がないバージョン2系とバージョン3系が使用されています。 商用製品の開発にも無料で使用でき、OSだけでなく仮想環境にも対応。Unicodeによる文字列操作をサポートしているため、日本語処理も標準で可能です。

0グッド

0クリップ

投稿2017/10/16 14:06

編集2017/10/16 14:51
import csv import datetime # 辞書にデータを読み取る date_dict = {} with open("moving_train.csv", "r", newline="") as f: reader = csv.reader(f) for row in reader: date = map(int, row[0].split('/')) date_dict[datetime.date(*date)] = int(row[1]) class Year: def __init__(self, x): self.x = x def month_ave(self): for key in list(date_dict): if datetime.date(self.x, 4, 1) <= key < datetime.date(self.x, 5, 1): pass else: del date_dict[key] total = sum(date_dict.values()) average = 1.0 * total / len(date_dict) print(total, average) if __name__ == '__main__': year_2011 = Year(2011) year_2012 = Year(2012) year_2013 = Year(2013) year_2011.month_ave() year_2012.month_ave() year_2013.month_ave()

お世話になります。以下の手順で作業を行おうとしたところエラーが出ました。

(手順)
・csvから以下のようなデータを取得(データ自体は2011-2015くらいまでございます)
2011/4/1,48
2011/4/2,29
2011/4/3,28
2011/4/4,23
2011/4/5,25
・同じ月で年ごとの平均を計算(例えば、2011年の4月と2012年の4月のデータの平均を計算)

以下のようなエラーが出たのですが、もし理由がわかれば教えてほしいです。
何卒よろしくお願いいたします。

C:\Users.....py" 710 23.666666666666668 Traceback (most recent call last): File "C:/Users/.....py", line 35, in <module> year_2012.month_ave() File "C:/Users/.....py", line 25, in month_ave average = 1.0 * total / len(date_dict) ZeroDivisionError: float division by zero

まだ初心者で本やウェブを見ながら作っているため、トンチンカンなところもあるかと思いますが、何卒よろしくお願いいたします。

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

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

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

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

MasaKoba

2017/10/16 14:29

申し訳ございません。ご指摘を踏まえてコードの形式を修正してみました。追加で修正が必要な場合は再度ご教示願います。
LouiS0616

2017/10/16 14:32 編集

なお、ご提示のコードではこの機能がお節介をして、__init__がinitの斜体として表示されてしまっていますし、Pythonのコメント行は見出しとして扱われています。
GOTA77

2017/10/16 14:43

記載しているコードと実行しているコードが合っているでしょうか?エラーはyeが定義されていいないためのように見えます。
MasaKoba

2017/10/16 14:55

皆様、貴重なご指摘ありがとうございます。少しずつ勉強していきます。お手数をおかけしますが引き続き何卒よろしくお願いいたします。
MasaKoba

2017/10/16 14:55

取り急ぎ御礼までです。
guest

回答1

0

ベストアンサー

クラス内でインスタンス変数を参照する際には、selfを付けねばなりません。

Python

1def month_ave(self): 2 for key in list(date_dict): 3 if datetime.date(self.x, 4, 1) <= key < datetime.date(self.x, 5, 1): 4 pass 5 else: 6 del date_dict[key] 7 8 total = sum(date_dict.values()) 9 average = 1.0 * total / len(date_dict) 10 print(total, average)

ただ、これを付けていなくても、ご提示のエラーは出ないかと思います。
実際にはこのようなエラーが出るのでは。

Traceback (most recent call last): File "hoge.py", line 29, in <module> year.month_ave() File "hoge.py", line 18, in month_ave if datetime.date(x, 4, 1) <= key < datetime.date(x, 5, 1): NameError: name 'x' is not defined

投稿2017/10/16 14:30

LouiS0616

総合スコア35658

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

MasaKoba

2017/10/16 14:58

ありがとうございます!selfをつけると大丈夫でした!ただ、elseのときにデータを削除してしまっているため、forで2011, 2012, 2013と回した場合、2週目からはすべてデータが削除されてしまい、その結果、len(date_dict)がゼロとなり、ZeroDivisionError: float division by zeroといったエラーが出てしまいます。
MasaKoba

2017/10/16 14:59

現在、これをどうにか解決できないか検討中でございます。
LouiS0616

2017/10/16 15:01

メソッドmonth_ave内で直接date_dictを破壊的に変更しなければ大丈夫でしょう。 ・ディープコピーしたものから要素を削除する ・特定の要素で新しく辞書を作る そのどちらかで対応できます。
MasaKoba

2017/10/16 15:21

ありがとうございます。メソッドmonth_aveの下に一行 date_dict_x = copy.deepcopy(date_dict) を加えると、上手く計算できました! 本当に助かります。ありがとうございます。(しかし、最近プログラミングを始めたのですが、プログラミングってすごく大変ですね…。。。根気よく頑張ります。。)
LouiS0616

2017/10/16 15:28

頑張ってくださいね。応援しています。 ある程度思い通りにいじれるようになると物凄く楽しいですよ。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.50%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問