C:并非所有代码路径都返回值

实际上我认为你的算法是错误的。如果只检查第一个值,那么为什么要使用foreach语句呢?另外,返回之后的所有其他代码都是多余的,这使得代码更难执行。

我怀疑你想做的事情更像这样:

private bool IsDestinationNodeAChildOfDraggingNode(TreeNode draggingNode, TreeNode destinationNode) {
  // special case, no children
  if (draggingNode.Nodes.Count == 0) 
    return false;

  // check if the target is one of my children
  if (draggingNode.Nodes.Contains(destinationNode)) 
     return true;

  // recursively check each of my children to see if one of their descendants is the target
  foreach (TreeNode node in draggingNode.Nodes) 
    if (IsDestinationNodeAChildOfDraggingNode(node, destinationNode)) 
        return true;

  // didn't find anything
  return false;
}

有些人会坚持认为提前归还是邪恶的,这将导致这个版本的功能:

private bool IsDestinationNodeAChildOfDraggingNode(TreeNode draggingNode, TreeNode destinationNode) {
  bool retVal = false;

  if (draggingNode.Nodes.Count != 0) {

    // check if the target is one of my children
    if (draggingNode.Nodes.Contains(destinationNode)) {
      retVal = true;
    } else {

      // recursively check each of my children to see if one of their descendants is the target
      foreach (TreeNode node in draggingNode.Nodes) 
        if (IsDestinationNodeAChildOfDraggingNode(node, destinationNode)) {
          retVal = true;
          break;
        }
    }
  }

  return retVal;
}

它可能是指这样一个事实:如果draggingnode.nodes中没有任何项,那么它将退出else,退出函数而不返回任何内容。

也许可以这样做:

foreach (TreeNode node in draggingNode.Nodes) 
     return IsDestinationNodeAChildOfDraggingNode(node, destinationNode);

return false

编译器认为您需要添加 return 后 foreach 循环,因为可能没有节点 draggingNode.Nodes 所以循环实际上不会执行。

private bool IsDestinationNodeAChildOfDraggingNode(TreeNode draggingNode, TreeNode destinationNode) {
    if (draggingNode.Nodes.Count == 0) 
        return false;
    else {
        if (draggingNode.Nodes.Contains(destinationNode)) 
            return true;
        else {
            foreach (TreeNode node in draggingNode.Nodes) 
            {
                return IsDestinationNodeAChildOfDraggingNode(node, destinationNode);
            }
            return false; // <-- here for example
        }
    }
}

现在,这可能不是您的算法的正确位置,但是编译器认为 返回 这里失踪。按照Jherico建议的方式重新编写代码将使情况更加清楚。

如果draggingnodes.nodes中没有项目怎么办?则不会返回值。

我的期望是,如果foreach中的所有元素都未能返回任何内容,或者没有节点,则foreach循环不一定返回值。在最后的if/else/foreach之后,添加一个默认返回值。

我猜C编译器假定foreach循环正常终止,而不是返回到大约三个控制级别。

在外面,否则你 必须 归还一些东西。

例如:

private bool IsDestinationNodeAChildOfDraggingNode(TreeNode draggingNode, TreeNode destinationNode) {
if (draggingNode.Nodes.Count == 0) 
        return false;
else {
       if (draggingNode.Nodes.Contains(destinationNode)) 
            return true;
       else {
            foreach (TreeNode node in draggingNode.Nodes) 
            return IsDestinationNodeAChildOfDraggingNode(node, destinationNode);
            }
    }
//Return something here.
}