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

Question

前提・実現したいこと

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を使用

Answer 1

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)

Comments

[T.Tom]

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