.NET TreeView 在尝试检查父节点时导致应用程序崩溃

Question

我有一个带有复选框的 TreeView，当用户检查子节点时，我想在树上检查每个父节点。但是，由于某种原因，每当我触摸父节点时，我的应用程序就会崩溃。对于树中的节点，我扩展了 TreeNode 来创建自己的对象，其中包含一些我需要存储在其中的数据，但在检查/取消检查时我仍然将它们引用为 TreeNodes。我的代码如下所示：

//checkBox checked event handler
    if (node.Parent != null)
    {
        checkAllParents(node.Parent);
    }
//

private void checkAllParents(TreeNode node)
{
    node.Checked = true;
    if (node.Parent != null)
    {
       checkAllParents(node.Parent);
    }
}

Answer 1

好吧，想通了。它不是循环引用，但本质上肯定是循环引用。这对我来说是一个很大的愚蠢错误。在​​事件处理程序中，我也使用递归来检查树的向下...我不久前实现了这个，但并没有真正考虑它，所以当我添加另一块时递归检查树我最终在递归函数和事件处理程序之间出现了无限循环（每次递归函数检查节点之一时都会调用该循环......After_checked 事件）。呸。

Answer 2

如果你确实有一个循环引用，那么你会遇到堆栈溢出，这是无法捕获的。尝试修改您的代码以消除递归：

// checkBox checked event handler
checkAllParents(node);

private void checkAllParents(TreeNode node)
{
  var parent = node.Parent;
  while (parent != null) {
  {
    parent.Checked = true;
    parent = parent.Parent;
  }
}

如果应用程序进入无限循环，那么您确实有一个循环引用。您可以进一步修改代码以捕获循环引用：

private void checkAllParents(TreeNode node)
{
  var parent = node.Parent;
  var visitedNodes = new List<TreeNode>();
  while (parent != null)
  {
    if (visitedNodes.Contains(parent))
      throw new InvalidOperationException("Circular reference!");
    visitedNodes.Add(parent);
    parent.Checked = true;
    parent = parent.Parent;
  }
}

当然，这不是处理循环引用的正确位置。但如果有帮助的话..

Answer 3

肯定是无限递归导致堆栈崩溃（堆栈溢出也不少）。 node.Parent 决不能计算为 null。它不会被 try catch 捕获，因为您的代码没有抛出异常，您只是导致堆栈溢出，并且抛出异常的是 .net 框架。每次设置 node.Checked = true; 时，您还将多次触发递归。您将再次触发事件处理程序，但您还再次调用检查所有父级，我将删除递归调用以检查所有父级，只需将节点设置为选中，事件处理程序将为您执行递归。