LINQ：RemoveAllとget要素を削除しました

Question

いくつかの条件に一致する項目をリストから削除して、その項目を取得する最も簡単な方法はどれですか。

私はいくつかの方法で考えることができ、私は最高の一つであるかわからない：

var subList = list.Where(x => x.Condition); 
list.RemoveAll(x => x.Condition); 

または

var subList = list.Where(x => x.Condition); 
list.RemoveAll(x => subList.Contains(x)); 

最善の方法のこの1の任意のですか？あれが、どちらですか？そうでない場合、どうすればいいですか？

Answer 1

私はあなたが最初のリストをマテリアライズする必要があり、またはあなたが次の行に選択しようとしている非常にアイテムを失うノートで、読みやすさのために、最初のオプションとなるだろう：

var sublist = list.Where(x => x.Condition).ToArray(); 
list.RemoveAll(x => x.Condition); 

2番目の例は、理由がないためO（n^2）ですが、最後は完全に問題ありませんが、読みにくいです。

編集：あなたの最後の例を読んだので、今すぐ書かれているように、他のすべての項目が取り出されることに注意してください。削除後にi+1番目の要素がi番目の要素になり、iを増やすと、その要素をスキップしているため、条件チェックがなくなり、削除行が実際にlist.RemoveAt(i--);になるはずです。

Answer 2

私は機能プログラミングアプローチ（新しいものを作るだけで、既存のものは変更しないでください）を使うのが好きです。 ToLookupのメリットの1つは、2つ以上のアイテムを分割して処理できることです。

ILookup<bool, Customer> lookup = list.ToLookup(x => x.Condition); 
List<Customer> sublist = lookup[true].ToList(); 
list = lookup[false].ToList(); 

それとも、元のインスタンスを変更する必要があれば...

list.Clear(); 
list.AddRange(lookup[false]);