ArgumentException与ArgumentNullException?
我正在重构一些代码并添加一个方法来替换(即将推出)已弃用的方法。 新方法具有以下签名:
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”);
谢谢!
您需要将args本身的检查添加为非null。 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”);
虽然我完全同意dasblinkenlight的回答,但您可能还想考虑将FooArgs
的validationFooArgs
到FooArgs
类本身。 如果这个类专门设计用于移动参数,那么它可能无效,因为它具有null proeprties,在这种情况下,我会允许它的构造函数进行validation。
在这种情况下,最好在该方法中检查FooArgs
参数的空引用,如果FooArgs
了空引用,则抛出ArgumentNullException
。然后,如果其他方法或代码段使用args中包含的参数class,它们应该是检查它并在必要时抛出exception的那些。 但是,如果您接受args类的方法是使用所有参数的方法,那么最好按照您的建议检查该方法中的有效参数。
此外,仅对空引用的参数使用ArgumentNullException
。 如果它只是一个无效值(例如一个空字符串),那么你应该使用更通用的ArgumentException
。
这取决于您的工具以及您对工具的感觉(resharper,fxcops等)。 一些静态代码分析工具接受这个:
throw new ArgumentNullException(“args.Property...”,"args");
并拒绝这一点
throw new ArgumentNullException(“args.Property...”,"args.Property");
因此,如果您想使用工具,那么针对参数属性的null-hood断言必须抛出ArgumentException
随身携带它也是有效的。 什么向维护开发人员传达正确的消息以帮助他正确传递参数是正确的信息。