c# - 异常或 If/Else 语句

Question

我对何时使用 Exception 或 If/Else 语句有疑问。 在我的情况下，我想检查 DocumentNode 是否是 TestNode。当它是一个TestNode时，我想获取该节点。

我在下面为此写了两个可能的解决方案。第一个解决方案认为它是一个 TestNode，否则它会给出一个异常。第二种解决方案 检查它是否是一个 TestNode，然后它执行大致相同的函数来获取该节点。谁能告诉我什么是最好的解决方案？或者有没有 更好的解决方案？谢谢，彼得。

*对不起我的英语不好..

private DocumentNode GetTestNode(IEnumerable<DocumentNode> nodes)
{
    foreach (DocumentNode node in nodes)
    {
        if (node.GetValueProperty().ToString() == "TestValue")
        {
            return node;
        }
    }
    throw new TestNodeNotFoundException("This is not a TestNode");
}

OR: 

private DocumentNode IsTestNode(IEnumerable<DocumentNode> nodes) 
{
    foreach (DocumentNode node in nodes)
    {
        if (node.GetValueProperty().ToString() == "TestValue")
        {
            return true;
        }
    }
    return false;
}

private DocumentNode GetTestNode(IEnumerable<DocumentNode> nodes)
{
    foreach (DocumentNode node in nodes)
    {
        if (node.GetValueProperty().ToString() == "TestValue")
        {
            return node;
        }
    }
}

Answer 1

如果可以的话，更好的解决方案是使用 LINQ。如果你想要一个异常(exception)，你可以使用:

var node = nodes.Where(n => node.GetPropertyValue().ToString() == "TestValue")
                .First();

(如果恰好有一个这样的节点，您也可以使用Single。)

如果您不想出现异常，请改用 FirstOrDefault:

var node = nodes.Where(n => node.GetPropertyValue().ToString() == "TestValue")
                .FirstOrDefault();

如果没有 TestValue 节点，node 将为 null。

要测试测试节点是否存在，请使用Any

var isTest = nodes.Where(n => node.GetPropertyValue().ToString() == "TestValue")
                  .Any();

(您的 final 方法当前不会编译，因为它可以在不返回的情况下到达结尾。)

哪个合适实际上取决于您要做什么。如果缺少测试节点表明存在错误，则抛出异常是合适的。如果不是，使用 null 来表示这是非常合理的，或者您可以使用 TryXXX 模式显式返回一个 bool 值，同时保存在输出参数中找到节点(如果有)。