RuntimeError: dictionary changed size during iteration

Question

 for key,value in dct.items():
        if len(value) != 10:
            dct = dct.pop(key)

とコードを書いて、dct辞書のバリューの長さが１０ではないものを削除するコードを書きました。このコードを実行すると、
RuntimeError: dictionary changed size during iteration
とエラーが出ました。

 for key,value in list(dct):
       if len(value) != 10:
　　    dct.pop(key)

とコードを書くと別のエラーが出て、

 for key in list(dct):
　　    dct.pop(key)

だとエラーは出ませんが、バリューの長さが１０であるものを検出できません。

どのように修正すればいいでしょうか？

Answer 1

ループ中に要素を増減させるのが良くないのです。
新しく条件を満たす辞書を作り直し、再代入すれば良いです。

dct = {k: v for k, v in dct.items() if len(v) != 10}

内包表記を使いたくないなら、こんな感じで書けます。

tmp = {}
for key, value in dct.items():
    if len(value) != 10:
        tmp[key] = value

dct = tmp

Answer 2

dct.items()をリストにしておくとできます。

>>> dct = {10:"a"*10, 20:"a"*20}
>>> for k,v in list(dct.items()):
...     if len(v) != 10:
...         dct.pop(k)
... 
'aaaaaaaaaaaaaaaaaaaa'
>>> dct
{10: 'aaaaaaaaaa'}

Comments

[k0908]

ありがとうございます。わかりやすかったのでベストアンサーにさせていただきます！

[unknown_user]

このままでも十分よいですが、`list(dct.items())`で別のオブジェクトを生成してるのが肝だと思うので`{**dct}.items()`の方が直感的かなと思いました。

[hayataka2049]

等価ですが、可読性の観点からすると、dct.copy().items()の方が好ましいかもしれません。またそうする場合、ハッシュテーブルをまるごとコピーする非効率さも気にならなくはありません。

[unknown_user]

なるほど勉強になります。