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

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

ただいまの
回答率

90.38%

  • Python

    12873questions

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

  • Python 3.x

    10746questions

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

pythonにおけるループ構造を使った辞書の作成について

解決済

回答 1

投稿

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

T.Tom

score 1

 前提・実現したいこと

Pythonの辞書型を使ったデータ管理の練習をしています。keyに学生番号、valueにいくつかの教科の点数をまとめたlistを定義します。それぞれの学生について、各教科の点数から評価値を求めます。次に、keyが学生番号でvalueが評価値になるdictionaryを定義しようとしています。for文の中で、ひとりひとりの点数から評価を求めて、都度dictionaryに追加していく、というコードをかきました。

 発生している問題・エラーメッセージ

for文の中で新たにdictionaryに要素を追加する度に、指定していないkeyのvalueもかわってしまいます。最終的に、全ての学生のvalueが、for文で最後に計算した学生の評価値になってしまいます。

 該当のソースコード

dic = { # 学生番号: [3教科の点数]
    '001': [1, 1, 1],
    '002': [2, 2, 2],
}
evaluate = [0, 0, 0]
test = { # 学生番号: [3教科の評価] ;output
'001': [0,0,0]
}

for index in dic: 
    points = dic[index] # 学生一人の3教科の点数
    evaluate.clear() #評価値を代入するlistの初期化
    for i, value in enumerate(points):
        evaluate.append(points[i]) # 点数をそのまま評価値とする
    test[index] = evaluate # outputに{ある学生番号: [3教科の評価値]}を追加
    print('test =',test)
print('test =',test) # {'001': [2,2,2], '002': [2,2,2]}

 試したこと

点数をそのまま評価値としているので、正しく複製できているのか確認するためにtest[index] = evaluateの行をtest[index] = pointsにしてみました。すると、test = {'001': [1,1,1], '002': [2,2,2]}となり正しくdictionaryがつくれました。また、evaluate == pointsを確認すると真でした。

計算速度の観点からみるとそもそもfor文を使うべきではないかもしれませんが、それは置いておいて、このコードは何が問題なのでしょうか。

 補足情報(FW/ツールのバージョンなど)

Python3.5.2を使用

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 1

checkベストアンサー

+4

evaluateを共有してしまっているのが原因です。
簡単に直すなら、evaluate.clear()evaluate = []に置き換えてください。

実はこれ、よく初心者を悩ませる動作です。

hoge = [0, 0, 0]
fuga = []

fuga.append(hoge)
hoge[0] = 10
fuga.append(hoge)

print(fuga)   # [[10, 0, 0], [10, 0, 0]]

変数はあくまでも参照値に過ぎず、参照先のインプレースな変更が影響してしまうのです。


細かな仕様がよく分かりませんが、こんな風に簡潔に書けそうですね。

def compute_evaluate(point_list):
  """生の得点を『評価値』に変換する"""
    return point_list

raw_result = {
    '001': [1, 1, 1],
    '002': [2, 2, 2],
}
result = {
    k: compute_evaluate(v) for k, v
    in raw_result.items()
}

print(result)

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2018/03/04 00:36

    解説書を読んで理解していたつもりでしたが、気づきませんでした。簡潔な回答ありがとうございます。

    キャンセル

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

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

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

  • Python

    12873questions

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

  • Python 3.x

    10746questions

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