コレクションが変更されました。列挙操作が実行されない可能性があります

Question

プレイヤーが生きていない場合にプレイヤーを削除するこのコードがありますが、問題はループforeachであると考えました。私は新しいリストを作るという解決策を見てきましたが、どのように私のコードにそれを適用できるのか分かりません。誰でも光を放つことができますか？

private Dictionary<int, Player> numPlayers = new Dictionary<int, Player>(); 

private void CheckPlayers() 
{ 
    foreach (Player player in numPlayers.Values) 
    { 
     if (!player.isAlive) 
     { 
      canvas.Children.Remove(player.hand); 
      numPlayers.Remove(player.id); // breaks here 
     } 
    } 
} 

Answer 1

クエリ選手は削除するためのコレクション：

var playersToDelete = numPlayers.Values.Where(p => !p.isAlive).ToList(); 

そして、プレイヤーを削除します。

foreach(var player in playersToDelete) { 
    canvas.Children.Remove(player.hand); 
    numPlayers.Remove(player.id); 
} 

Answer 2

反復処理するコレクションをforeachで変更することはできません。あなたがしなければならないことは、削除したいアイテムを新しいリストに追加し、そのリストがビルドされたらそれらを削除することです。

var dead = numPlayers.Values 
    .Where(p => !p.isAlive) 
    .ToList(); 

foreach(var player in dead) 
{ 
    canvas.Children.Remove(player.hand); 
    numPlayer.Remove(player.id); 
} 

Answer 3

あなたがのために逆を使用して、コレクションの要素を削除する必要があります。

for(int i = numPlayers.Values.Count - 1; i <= 0; i--) 
{ 
//Remove it. 
}