我正在重构一些代码,并添加一个方法来替换即将被弃用的方法。新方法具有以下签名:
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”);
谢谢!
发布于 2011-12-11 04:52:19
您需要添加一个检查,以使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”);
发布于 2011-12-11 05:12:21
虽然我完全同意dasblinkenlight的回答,但您可能还想考虑将FooArgs
的验证转移到FooArgs
类本身中,如果这个类是专门用来移动参数的,那么它很可能不具有null属性,在这种情况下,我会允许它的构造函数进行它的验证。
发布于 2011-12-11 04:53:46
在这种情况下,最好检查该方法内部的FooArgs
参数是否为空引用,如果传入了空引用,则抛出ArgumentNullException
。然后,如果其他方法或代码段使用args类中包含的参数,则它们应该对此进行检查并在必要时抛出异常。但是,如果接受args类的方法是使用所有参数的方法,那么最好按照您的建议检查该方法中的有效参数。
此外,仅对空引用的参数使用ArgumentNullException
。如果它只是一个无效值(例如,一个空字符串),那么您应该使用更通用的ArgumentException
。
https://stackoverflow.com/questions/8459755
复制相似问题