リスト内の一致するノードをすべて削除します

Question

私のスクリプトには2つの問題があり、理解を助ける必要があります。

ifステートメントは、元のリストに一致する値を探します。次にそれを削除します。これは何らかの理由で機能しません。 items.Remove(item)は実際にアイテムを削除しません。

else文が機能します。しかし、私はこれをすべて1行で行うことができると感じています。

思考？可能であれば、私はそれをリストにすることを避けたいと思います。その元はObservableCollectionです。

var items = TreeViewCollection.ToList(); 

if (items.Contains(SelectedTreeViewItem)) 
    items.Remove(SelectedTreeViewItem); 
else 
    items.ForEach(e=>e.Nodes.Remove(SelectedTreeViewItem)); 

更新：Iveはもう少し意味をなさせるようにコードを更新しました。

私の目標は、これを1行にすることです。

if (TreeViewCollection.Contains(SelectedTreeViewItem)) 
    TreeViewCollection.Remove(SelectedTreeViewItem); 
else 
    TreeViewCollection.ToList().ForEach(e=>e.Nodes.Remove(SelectedTreeViewItem)); 

Answer 1

新しいリストを作成することは逆効果です。 SelectedTreeViewItemはリストから削除されますが、元のコレクションからは削除されません。

ForEachは、IEnumerableの拡張メソッドではなく、List<T>クラスの一般的なメソッドです。それはvoidを返すので、Linqのメソッドチェーンの最後にしか存在できません。 foreachを使用すると、コードが簡単になり、意図が明示されます。

bool del = TreeViewCollection.Remove(SelectedTreeViewItem); 
if (false == del) 
    foreach(var t in TreeViewCollection) 
     t.Nodes.Remove(SelectedTreeViewItem);