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

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

ただいまの
回答率

90.49%

  • Python

    12220questions

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

  • if

    258questions

    if文とは様々なプログラミング言語で使用される制御構文の一種であり、条件によって処理の流れを制御します。

[python]if文を使った集計について

解決済

回答 3

投稿

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

 pythonで自動販売機の月別売上高の集計を行うプログラムを作成したい

<前提>

ファイル'input/report.csv'の中に、自動販売機の売上データが入っている。形式は以下の通り
2014-06-20 10:22:13,コーヒー,120
2014-06-21 18:10:02,オレンジジュース,110
2014-06-28 12:03:40,コーラ,120
2014-07-04 13:47:10,ミネラルウォーター,100
2014-07-08 22:10:28,コーラ,120
2014-07-10 06:55:22,お茶,150
2014-07-12 12:10:40,お茶,150
2014-07-13 10:29:33,コーラ,120
2014-07-13 11:18:08,コーヒー,120
2014-07-14 13:36:59,オレンジジュース,110

=>このファイルを読み込んで、月ごと(6月、7月)に売上金額のを集計し、画面に出力するプログラムを作成する。

 先に正解を記載します

from datetime import datetime

sales ={}

with open('input/report.csv', encoding='utf-8') as f:
    for row in f:
        date, menu, price = row.rstrip().split(',')
        solddate = datetime.strptime(date, '%Y-%m-%d %H:%M:%S')
        if solddate.month not in sales:
            sales[solddate.month] = 0
        sales[solddate.month] += int(price)

for key,value in sales.items():
    print(key,'月: ', value, sep='')

 よくわからない点

9〜11行目の条件分岐について
辞書salesにsolddate.monthが入っていなかったら、値を0とする項目を追加するということでしょうか?
{solddate.month:0, ...}というイメージですか?
条件分岐の外(11行目)でsales[solddate.month]の値にpriceを自己代入しているのも理解できません。

--------------9行目から-----------------

if solddate.month in sales:
    sales[solddate.month] += int(price)
else:
    sales[solddate.month] = 0

--------------以下省略------------------

上のように記述した方が自分の理解としてはわかりやすく、
形式上も綺麗なのにと思ったら異なる実行結果が出てしまいました。

正解の方の条件分岐がなぜ正しく、2つめに記載したコードの条件分岐が何故誤っているのか、
2つめのコードの条件分岐は何をどう分岐しているのか、
初心者にも分かるようご教示いただけるとありがたいです。

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 3

checkベストアンサー

+2

質問者さんは勘違いなさってますね。

        if solddate.month not in sales:      # s1
            sales[solddate.month] = 0        # s2
        sales[solddate.month] += int(price)  # s3

if solddate.month in sales:
    sales[solddate.month] += int(price)
else:
    sales[solddate.month] = 0

では動作が違います。

質問者さんのelse:が勘違いしていますね。
正しいものは、sales[solddate.month] += int(price)が前のifにかかわらず実行されます。ですから、質問者さんのだとelseの場合に実行するので動作が違います。
フローチャートを書いてみると、一目瞭然ですよ。
また、正解の文、s1とs2はキー(solddate.month)がなければ、初期設定する文で当然と思われる文です。かつ、文s3の前になければなりません。
こんな感じでわかっていただけるでしょうか。
自分で数値をあてはめてみてみて、机上で流れを追って動作を確認してみてはいかがでしょうか。デバッガを使ってもよろしいですが。

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2018/08/18 22:35

    この回答を見て、ようやく自分のコードがめちゃくちゃなのが分かりました。
    非常に分かりやすい解説をありがとうございました。

    キャンセル

+1

書き直したコードだと、
ループ一回目の「120円のコーヒー」はどこで集計するんですかね。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2018/08/18 22:36

    間違った方のコードだと先頭の120円のコーヒーはifを通り過ごしてelseで処理されますね。
    素早いご回答ありがとうございました。

    キャンセル

+1

sales = {}  # 空っぽの辞書を作る

# 8月の分を初期化しないと、値を取り出せない
print(sales[8])
# KeyError

# 足すこともできない。
sales[8] += 120
# KeyError

# 初期化すると
sales[8] = 0

# 足せるようになる。
sales[8] += 120

書き直したコードは次のようにすると正しくなる。けど、おんなじ内容の行が重複してしまうでしょう。

if solddate.month in sales:
    sales[solddate.month] += int(price)
else:
    sales[solddate.month] = 0
    sales[solddate.month] += int(price)

豆知識

Pythonにはdefaultdictという初期値を設定できる便利な辞書があります。上の間違いが分かったら試して見て下さい。

from datetime import datetime
from collections import defaultdict
sales = defaultdict(int)

with open('input/report.csv', encoding='utf-8') as f:
    for row in f:
        date, menu, price = row.rstrip().split(',')
        solddate = datetime.strptime(date, '%Y-%m-%d %H:%M:%S')
        sales[solddate.month] += int(price)

for key,value in sales.items():
    print(key,'月: ', value, sep='')

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2018/08/18 22:23

    初期化について、非常に分かりやすくご説明ありがとうございました。
    defaultdictは、今回の例でいうと
    sales={}

    sales[solddate.month] = 0
    の記述が1行で済むということですね?
    非常に勉強になります。ありがとうございます。

    キャンセル

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

  • Python

    12220questions

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

  • if

    258questions

    if文とは様々なプログラミング言語で使用される制御構文の一種であり、条件によって処理の流れを制御します。