在c#中处理一个不抛出异常的类

Question

我有一些如下所示的 UI 代码：

try
{
    SomeClass classInstance = new SomeClass(someId);
}
catch (Exception exception)
{
    // Content wasn't created, show a message, stop processing
    return;
}

似乎添加了 try catch，因为如果 SomeClass 的构造函数接收到的 someId 无效，并且无法在 DB 中找到数据，那么它的构造函数就会崩溃。

最近通过 FXCop 运行此代码，它警告不要使用一般异常，但 SomeClass 所做的只是抛出一个新异常，并显示一条消息说它无法初始化。

我想问题是类构造函数应该有它自己的自定义异常，然后我可以在我的 UI 中处理它，但我想知道我还能对上面的代码做些什么来处理满足 FXCop 要求的异常？

Answer 1

FxCop 的规则之所以存在，是因为上面的 catch (Exception) 块捕获了所有可能的异常，包括像 StackOverflowException 这样的低级异常，您可能无法以有用的方式捕获这些异常。

正确的方法肯定是抛出一个更具体的类型：要么是你发明的，要么是与你的情况非常匹配的现有 .NET 框架异常类型。 （如果有疑问，我通常会选择InvalidOperationException。）

或者，您可以在捕获时检查确切的异常类型。这不会阻止 FxCop 警告，但它应该解决根本问题：

catch (Exception exception)
{
    if (exception.GetType() == typeof(Exception))
    {
        // Content wasn't created, show a message, stop processing
        return;
    }
    else
    {
        // Some other exception type that wasn't thrown from our code -
        //    delegate to a higher-level exception handler
        throw;
    }
}

Answer 2

你不需要自定义异常；在给定的情况下，只需使用框架中已经存在的几十个之一。如果someId 不好，请抛出ArgumentException——这就是它的用途。如果某些不应该为 null 的内容，则会出现 NullReferenceException；让它被扔掉。等等。抛出一个普通的Exception 有点像说“出了点问题 - 阅读消息以了解详细信息”而不是“这出了问题”。

FxCop 抱怨catch (Exception)，因为它被滥用太普遍了，无法吞下所有异常，而不是让它们传播并由知道如何这样做的代码处理。您应该能够说出正在抛出的异常类型并捕获这些异常，同时让那些您不认识的异常进入调用堆栈。

Answer 3

您应该修复类构造函数。抛出 Exception 是永远一个好主意。

但是，要临时解决您遇到的问题（因为这是一种可怕的、不可靠的黑客攻击），您可以将异常消息与您期望的消息进行对比：

catch (Exception exception)
{
    if (exception.Message == "whatever your class sets the message to")
        // Content wasn't created, show a message, stop processing
        return;
    else
        // Any other exception should bubble
        throw;
}

Answer 4

使用InvalidOperationException 代替投掷Exception 听起来可能是明智的。

Answer 5

如果 FXCop 不喜欢处理一般异常（我倾向于同意），那么也许您可以访问 SomeClass 的源代码。修改构造函数以抛出更具体的异常，例如ArgumentOutOfRangeException 或一些自定义异常。

在这种情况下，您的代码将如下所示：

try
{
   SomeClass classInstance = new SomeClass(someId);
}
catch(ArgumentOutOfRangeException exception)
{
   // Content wasn't created, show a message, stop processing
   return;
}

Answer 6

正如许多其他人所说，构造函数不应该抛出一个赤裸裸的Exception。看到构造函数从DB取数据并根据结果抛出，最好的解决办法就是自己创建异常类。

在 Visual Studio 中创建异常非常简单。只需输入Exception 并按TAB。然后它将使用所需的构造函数（全部四个）创建异常类。不要害怕创建不做太多事情的类，这就是它们的设计目的。

这就是我编写这个类的方式：

public class SomeClass {
    public SomeClass(int someId) {
        if (someId < 0) //validation on the ID, can it be negative?
            throw new ArgumentException("someId", "ID cannot be negative");

        //Perform DB operation
        if (/*DB error - not found*/)
            throw new DataNotFoundException("Cannot find record with ID " + someId);
    }
}

[Serializable]
public class DataNotFoundException : Exception {
    public DataNotFoundException() { }
    public DataNotFoundException(string message) : base(message) { }
    public DataNotFoundException(string message, Exception inner) : base(message, inner) { }
    protected DataNotFoundException(
      System.Runtime.Serialization.SerializationInfo info,
      System.Runtime.Serialization.StreamingContext context)
        : base(info, context) { }
}