ArgumentException与ArgumentNullException？

Question

我正在重构一些代码，并添加一个方法来替换即将被弃用的方法。新方法具有以下签名：

FooResult Foo(FooArgs args) { ... }

不推荐使用的方法包含越来越多的参数。这些参数现在是FooArgs类的属性。不推荐使用的方法有几个保护条件，它们使用以下结构检查空值：

if (parameter1 == null)
    throw new ArgumentNullException(“parameter1”);
if (parameter... == null)
    throw new ArgumentNullException(“parameter...”);
if (parameterN == null)
    throw new ArgumentNullException(“parameterN”);

现在参数已经折叠到FooArgs类中，我应该为FooArgs参数的各个属性抛出一个ArgumentNullException：

if (args.Property1 == null)
    throw new ArgumentNullException(“args.Property1”);
if (args.Property... == null)
    throw new ArgumentNullException(“args.Property...”);
if (args.PropertyN == null)
    throw new ArgumentNullException(“args.PropertyN”);

或者为整个 FooArgs参数抛出一个更通用的ArgumentException：

if (args.Property1 == null)
    throw new ArgumentException(“Property1 cannot be null.”, “args”);
if (args.Property... == null)
    throw new ArgumentException(“Property... cannot be null.”, “args”);
if (args.PropertyN == null)
    throw new ArgumentException(“Property2 cannot be null.”, “args”);

谢谢!

Answer 1

您需要添加一个检查，以使args本身不为空。ANE不适用于单个组件，因此您需要使用更通用的AE，如下所示：

if (args == null)
    throw new ArgumentNullException(“args”);
if (args.Property1 == null)
    throw new ArgumentException(“Property1 cannot be null.”, “args”);
if (args.Property... == null)
    throw new ArgumentException(“Property... cannot be null.”, “args”);
if (args.PropertyN == null)
    throw new ArgumentException(“Property2 cannot be null.”, “args”);

Answer 2

虽然我完全同意dasblinkenlight的回答，但您可能还想考虑将FooArgs的验证转移到FooArgs类本身中，如果这个类是专门用来移动参数的，那么它很可能不具有null属性，在这种情况下，我会允许它的构造函数进行它的验证。

Answer 3

在这种情况下，最好检查该方法内部的FooArgs参数是否为空引用，如果传入了空引用，则抛出ArgumentNullException。然后，如果其他方法或代码段使用args类中包含的参数，则它们应该对此进行检查并在必要时抛出异常。但是，如果接受args类的方法是使用所有参数的方法，那么最好按照您的建议检查该方法中的有效参数。

此外，仅对空引用的参数使用ArgumentNullException。如果它只是一个无效值(例如，一个空字符串)，那么您应该使用更通用的ArgumentException。