リストに追加すると、なぜforループが壊れるのですか？ （Python）

Question

このコードでは、家族のリストを繰り返し、18歳以上の男性を特定し、家族から削除し、自分の家族を始めようとしています。ループの最後にappendをコメントアウトすると、（家族の数を増やすことなく）うまく動作します。これは私が追記行う場合のようにそれが見えるものです：

Ticking Disan 
There were 5 members of family Evnes 
Now there are 4 members of family Evnes 
Gregor needs a wife 
There were 6 families 
Now there are 7 families 
There were 7 members of family Bhworth 
Now there are 6 members of family Bhworth 
Ewan needs a wife 
There were 7 families 
Now there are 8 families 

debugger.run(setup['file'], None, None) 
    File "C:\Users\Mark\Desktop\eclipse-SDK-3.7.2-win32\eclipse\plugins\org.python.pydev.debug_2.5.0.2012040618\pysrc\pydevd.py", line 1060, in run 

pydev_imports.execfile(file, globals, locals) #execute the script 
    File "C:\Users\Mark\workspace\WorldPop\WorldPop.py", line 610, in <module> 
    main() 
    File "C:\Users\Mark\workspace\WorldPop\WorldPop.py", line 74, in main 
    done = menu() 
    File "C:\Users\Mark\workspace\WorldPop\WorldPop.py", line 77, in menu 
    genWorld() 
    File "C:\Users\Mark\workspace\WorldPop\WorldPop.py", line 116, in genWorld 
    dispWorld(oneWorld) 
    File "C:\Users\Mark\workspace\WorldPop\WorldPop.py", line 135, in dispWorld 
    displayTick(world) 
    File "C:\Users\Mark\workspace\WorldPop\WorldPop.py", line 317, in displayTick 
    calcMarriage(city) 
    File "C:\Users\Mark\workspace\WorldPop\WorldPop.py", line 359, in calcMarriage 
    for member in family.membersList: 
TypeError: iteration over non-sequence 

を、私は戻って最初にループサイクルのための新たなメンバーを検索するときに問題が来ることを認識し、私は、なぜ行って表示されていませんappendはループを壊しています。どんな洞察力も大変ありがとうございます。

Answer 1

一般的には、反復処理中のコレクションを変更することは本当に悪いことです（どちらもfamily.membersListから削除し、city.familyListに追加しています）。

反復処理中のデータ構造や反復アルゴリズムによっては、反復処理されたコレクションを変更すると、項目がスキップされるか、複数回表示されることがあります。私は、ドキュメントが明白にそうでないと言わない限り、これはほとんどの場合、安全ではないと思います。

Answer 2

追加する新しいFamilyインスタンスにはmembersList属性がありますが、反復（/ in）はサポートされていません。だからあなたはこの例外を受け取ります。

それを修正した後でも、シーケンスを反復処理している間にシーケンスを変更することで、驚きを自分で設定しています。これを試してみてください：それは、ループの開始時に存在しているよう

for family in city.familyList[:]: 
    for member in family.membersList[:]: 
     # ... 

[:]スライス構文は、各リストのコピーを作成します。元のリストに加えられた変更はコピーに影響しないので、ループ実行中に驚きはありません。 ループ中に見つかった/作成された新しいアイテムを実際にループに含める必要がある場合は、Queue.Queueオブジェクトを繰り返し処理し、各オブジェクトをそのキューに挿入することをお勧めします。探索するために何も新しいことがなければ、キューは空になり、ループは停止します。