辞書間でオブジェクトを移動するにはどうすればよいですか？

Question

私は一定の基準が他への移行を満たしている場合、私は1つの辞書にオブジェクトをチェックする必要があり、簡単なタスクを持っています。私はそれを達成するために言語機能を使うことができる良いパターンがあるかどうか尋ねています。まっすぐアプローチは簡単です - 一時的なコレクションを使用して、最初のステップはcanditatesを決定し、2番目のステップは実際の動きを行います。それは大丈夫ですが、それはクールではありません。

現在のコード

class Order 
{ 
    public int ID; 
    public bool IsReady; 
} 

Dictionary<int, Order> ActiveDictionary; 
Dictionary<int, Order> ProcessedDictionary; 

public Update() 
{  
// temporary list, uncool 
List<Order> processed = new List<Order>(); 


// fist step 
foreach(Order ord in ActiveDictionary) 
{ 
    if(ord.IsReady) 
    { 
    processed.Add(ord); 
    } 
} 

// ok now lets move 
foreach(Order ord in processed) 
{ 
    ActiveDictionary.Remove(ord.ID); 
    ProcessedDictionary.Add(ord.ID, ord); 
} 
} 

Answer 1

あなたが持っているコードで、本当に間違っては何もありません。代替案では練習として

レックスMの答えが適切でない場合は、あなたが何かのような...

ProcessedDictionary = ProcessedDictionary 
    .Concat(
     ActiveDictionary.Where(kvp => kvp.Value.Ready) 
    ) 
    .ToDictionary(kvp => kvp.Key, kvp => kvp.Value); 

ActiveDictionary = ActiveDictionary.Where(kvp => !kvp.Value.Ready) 
    .ToDictionary(kvp => kvp.Key, kvp => kvp.Value); 

Answer 2

を行うことができます（多分あなたは、各iteartionに辞書を再構築する必要はありません） 、その後、小さな改善と私は

var processed = ActiveDictionary.Where(x=>x.Value.Ready).ToArray(); 

に

// temporary list, uncool 
List<Order> processed = new List<Order>(); 

// fist step 
foreach(Order ord in ActiveDictionary) 
{ 
    if(ord.IsReady) 
    { 
    processed.Add(ord); 
    } 
} 

を交換することをお勧めすることができます

と

foreach(var item in processed) 
{ 
    ActiveDictionary.Remove(item.Key); 
    ProcessedDictionary.Add(item.Key, item.Value); 
} 

UPDあなたのコードの残りの部分は次のようになります。

var processed = ActiveDictionary.Values.Where(x=>x.Ready).ToArray(); 

foreach(var item in processed) 
{ 
    ActiveDictionary.Remove(item); 
    ProcessedDictionary.Add(item.Id, item); 
} 

Answer 3

は、私が持っているこの

var keys = ActiveDictionary 
    .Where(kv => kv.Value.Ready) 
    .Select(kv => kv.Key).ToList(); 
keys.ForEach(k => 
     { 
      ProcessedDictionary.Add(k, ActiveDictionary[k]); 
      ActiveDictionary.Remove(k); 
     });