如何在c＃/ .net中记录抛出的异常[关闭]

Question

我目前正在编写一个小框架，将由公司内部的其他开发人员在内部使用。

我想提供良好的Intellisense信息，但我不知道如何记录抛出的异常。

在以下示例中：

public void MyMethod1()
{
    MyMethod2();

    // also may throw InvalidOperationException
}

public void MyMethod2()
{
    System.IO.File.Open(somepath...); // this may throw FileNotFoundException

    // also may throw DivideByZeroException
}

我知道记录异常的标记是：

/// <exception cref="SomeException">when things go wrong.</exception>

我不明白的是如何记录由MyMethod1()调用的代码引发的异常？

• 我应该记录MyMethod2()抛出的异常
• 我应该记录File.Open()抛出的异常吗？

记录可能的异常的最佳方法是什么？

Answer 1

您应该记录代码可能抛出的每个异常，包括您可能调用的任何方法中的异常。

如果列表有点大，您可能想要创建自己的异常类型。捕获您在方法中可能遇到的所有内容，将它们包装在异常中，并抛出它。

你可能想要这样做的另一个地方是你的方法是在你的API面前。就像外观将多个接口简化为单个接口一样，您的API应将多个异常简化为单个异常。使呼叫者更容易使用您的代码。

---

为了回答安德鲁的一些顾虑（来自评论），有三种类型的例外：你不了解的，你知道的和不能做的事，以及你所知道的并且可以做些什么。

那些你不了解的人想要放手。它是快速失败的原则 - 更好地使您的应用程序崩溃，而不是进入可能最终破坏您的数据的状态。崩溃将告诉你发生了什么以及为什么，这可能有助于将这个异常移出“你不知道的”列表。

你知道但无法做任何事情的是像OutOfMemoryExceptions这样的例外。在极端情况下，您可能希望处理这样的异常，但除非您有一些非常出色的要求，否则您将它们视为第一类 - 让我们去吧。你必须记录这些例外吗？对于新建对象的每一种方法，你都会看起来非常愚蠢地记录OOM。

你知道并可以做些什么的是你应该记录和包装的。

你可以找到更多的guidelines on exception handling here.

Answer 2

你应该使用standard xml documentation。

/// <exception cref="InvalidOperationException">Why it's thrown.</exception>
/// <exception cref="FileNotFoundException">Why it's thrown.</exception>
/// <exception cref="DivideByZeroException">Why it's thrown.</exception>
public void MyMethod1()
{
    MyMethod2();
    // ... other stuff here
}

/// <exception cref="FileNotFoundException">Why it's thrown.</exception>
/// <exception cref="DivideByZeroException">Why it's thrown.</exception>
public void MyMethod2()
{
    System.IO.File.Open(somepath...);
}

/// <exception cref="FileNotFoundException">Why it's thrown.</exception>
public void MyMethod3()
{
    try
    {
        MyMethod2();
    }
    catch (DivideByZeroException ex)
    {
        Trace.Warning("We tried to divide by zero, but we can continue.");
    }
}

这样做的价值在于您提供了可能发生的已知异常的文档。如果您使用visual studio，则可以在intellisense中使用此文档，并且可以在以后提醒您（或其他人）您可以预期的异常。

您希望指定特定的异常类型，因为您可能能够处理一种类型的异常，而其他类型是严重问题的结果，无法更正。

Answer 3

您可以使用几个出色的加载项来简化文档编制过程。其中之一是GhostDoc，它是Visual Studio的免费插件，可生成XML-doc注释。此外，如果您使用ReSharper，请查看ReSharper的优秀Agent Johnson Plugin，它为添加的异常生成XML注释。

更新：似乎Agen Johnson不适用于R＃8，结帐Exceptional for ReSharper作为替代......

第1步：GhostDoc生成XML注释（Ctrl-Shift-D），而ReSharper的Agent Johnson插件也建议记录异常：

第2步：使用ReSharper的快捷键（Alt-Enter）添加异常文档：

step 2 http://i41.tinypic.com/osdhm

希望有帮助:)

Answer 4

根据我的理解，使用<exception>元素的意图是在装饰方法时使用它，而不是例外：

/// <summary>Does something!</summary>
/// <exception cref="DidNothingException">Thrown if nothing is actually done.</exception>
public void DoSomething()
{
// There be logic here
}

可以通过其他方法抛出的异常应该在这些方法中捕获，处理和记录。可能会记录.NET可能抛出的异常，或者您自己的代码显式抛出的异常。

至于获得更具体的信息，也许您可​​以捕获并抛出自己的自定义异常？

Answer 5

您的方法的部分合同应该是检查前置条件是否有效，因此：

public void MyMethod2()
{
    System.IO.File.Open(somepath...); // this may throw FileNotFoundException
}

变

/// <exception cref="FileNotFoundException">Thrown when somepath isn't a real file.</exception>
public void MyMethod2()
{
    FileInfo fi = new FileInfo( somepath );
    if( !fi.Exists )
    {
        throw new FileNotFoundException("somepath doesn't exists")
    }
    // Maybe go on to check you have permissions to read from it.

    System.IO.File.Open(somepath...); // this may still throw FileNotFoundException though
}

使用这种方法，更容易记录您明确抛出的所有异常，而不必记录可能抛出OutOfMemoryException等。

Answer 6

您应该记录您的方法可能抛出的所有异常。

为了隐藏实现细节，我会尝试自己处理MyMethod2的一些异常。

如果您无法处理或解决异常，您可以考虑重新启动它们。主要是为调用者打包/包装一个更有意义的例外。

Answer 7

实际上，正如已经回答的那样，记录异常的方法是使用XML注释。

除了插件之外，您还可以使用可与TFS集成的静态分析工具，以确保记录了异常。

在下面的链接中，您可以看到如何为StyleCop构建自定义规则，以验证正在记录方法引发的异常。

http://www.josefcobonnin.com/post/2009/01/11/Xml-Documentation-Comments-Exceptions-I.aspx http://www.josefcobonnin.com/post/2009/01/15/Xml-Documentation-Comments-Exceptions-II.aspx

问候。

Answer 8

在您的方法中记录预期的异常，在您的示例中，我会让用户知道该方法可以抛出未找到的文件异常。

请记住，它是告知调用者期望的内容，以便他们可以选择如何处理它。