すべてのノードをツリービューでステータスをチェックしようとしました - get StackOverflowException

Question

TreeView（trvP）にいくつかのノードが追加されています。ルート要素はTagプロパティがRootに設定されています。

私はルート要素をチェックすると、他のすべてのノードが同じステータスになるようにしています。ただし、以下のコードを実行するとStackOverflowExceptionになります。

private void trvP_AfterCheck(object sender, TreeViewEventArgs e) 
{ 
     if(e.Node.Tag.Equals("Root")) 
     { 
      var nodes = TreeViewExtensions.GetAllNodes(e.Node.TreeView); 
      foreach (TreeNode node in nodes) 
       node.Checked = e.Node.Checked; 
     } 
} 

GetAllNodes関数のコード：それだけforeachループ（ルートノード）の内部に単一のノードを設定して、それは、何度もコードのvar nodes = ...一部を実行することをデバッグから思わ

public static List<TreeNode> GetAllNodes(this TreeView _trv) 
{ 
     List<TreeNode> result = new List<TreeNode>(); 
     foreach (TreeNode child in _trv.Nodes) 
     { 
      result.AddRange(child.GetAllNodes()); 
     } 
     return result; 
} 
public static List<TreeNode> GetAllNodes(this TreeNode _trn) 
    { 
     List<TreeNode> result = new List<TreeNode>(); 
     result.Add(_trn); 
     foreach (TreeNode child in _trn.Nodes) 
     { 
      result.AddRange(child.GetAllNodes()); 
     } 
     return result; 
    } 

。しかし、nodesは正しい値を持つノードの適切なリストです。

なぜこれが何度も繰り返し実行されているかわかりませんが、例外が発生します。

Answer 1

あなたのGetAllNodes()の拡張子は、最初の子ノードから始まり、TreeNodeの拡張子をGetAllNodes()とします。

このGetAllNodes()の拡張子は、この非常にTreeNodeを結果リストに追加します。
最初にTreeNodeがルートノードに再びです。

これは順番に、このルートノードのために再びハンドラtrvP_AfterCheckを呼び出して、あなたのルートノードのCheckedプロパティを設定する行に

node.Checked = e.Node.Checked; 

を意味します。これは無限に繰り返され、スタックにあふれてStackOverflowExceptionを上げます。これを解決するため、ルートノードをフィルタリングするには

：

private void trvP_AfterCheck(object sender, TreeViewEventArgs e) 
{ 
     if(e.Node.Tag.Equals("Root")) 
     { 
      var nodes = e.Node.TreeView.GetAllNodes(); 
      foreach (TreeNode node in nodes) 
      { 
       if (node == e.Node) continue; // don't do it for root again 
       node.Checked = e.Node.Checked; 
      } 
     } 
} 

---

ところで：拡張メソッドの良いところは、彼らがinstancメソッドであるかのようにあなたが構文的にそれらを呼び出すことができるということです。したがって、この

var nodes = TreeViewExtensions.GetAllNodes(e.Node.TreeView) 

は単に

var nodes = e.Node.TreeView.GetAllNodes(); 

Answer 2

あなたは

はすでに_trv.NodesにTREENODEするためにループしているとき、あなたはforeachのに現在のノードを追加する必要がありJST

public static List<TreeNode> GetAllNodes(this TreeView _trv) 
{ 
     List<TreeNode> result = new List<TreeNode>(); 
     foreach (TreeNode child in _trv.Nodes) 
     { 
      result.Add(child); 
     } 
     return result; 
} 

を試してみてください。..無限状態にループを作るどの再帰的GetAllNodesを呼び出しています