c# - 这是空 try catch block 的情况吗？

Question

假设您读入一个大的 XML 文件并且大约 25% 的节点是可选的，所以您真的不关心它们是否存在，但是，如果它们被提供给您，您仍然会读入它们并对它们做一些事情(例如将它们存储在数据库中)。

因为它们是可选的，在这种情况下将它们包装在空 try 中不是可以吗？ . . catch block ，所以如果它们不存在，程序就继续执行？你不关心抛出错误或类似的东西。

请记住，仅仅因为它们是可选的并不意味着您不想检查它们。这只是意味着向您提供 XML 的人不想让您知道某些信息，或者他们确实想让您知道并且由您来处理。

最后，如果这只是几个节点，那没什么大不了的，但是如果您有 100 个节点是可选的，那么检查每个节点是否为 null< 可能会很痛苦 或如果找到 null 则停止执行，因此我问这是否是空 try catch 语句的正当理由。

Answer 1

如果处理节点 X 是可选的，那么您的代码应该类似于:

if(node X exists in file)
{
  do work with X
}

而不是:

try
{
  do work with X
}
catch{}

现在，如果除了尝试使用节点 X 之外无法确定节点 X 是否存在，或者在您检查节点 X 是否存在之后是否有可能将其删除，那么您将被迫使用尝试/捕捉模型。这不是这里的情况。 (相对于说，在读取文件之前检查文件是否存在；在您检查文件是否存在后，其他人可以删除它。)

-------------------------------------------- --------------

编辑:

因为您的问题似乎是在以下 XML 中单独访问节点“grandchild”，其中“Parent”可能不存在。 (请原谅我无法在 SO 中呈现此 XML；知识渊博的读者可以随意以适当的格式进行编辑。)

<root>
  <Parent>
    <Child>
      <GrandChild>
        The data I really want.
      </GrandChild>
    </Child>
  </Parent>
</root>

为此我会做这样的事情:

public static class XMLHelpers
{
public static Node GetChild(this Node parent, string tagName)
{
  if(parent == null) return null;
  return parent.GetNodeByTagName(tagName);
}
}

然后你可以这样做:

var grandbaby = rootNode.GetChild("Parent").GetChild("Child").GetChild("GrandChild");
if(grandbaby != null)
{
  //use grandbaby
}