假设我有一个扩展方法
public static T TakeRandom<T>(this IEnumerable<T> e) { ...
要验证参数e,我应该:
A)if(e == null)抛出新的NullReferenceException()
B)if(e == null)抛出新的ArgumentNullException(“e”)
C)不检查e
什么是共识?
我的第一个想法是始终验证参数,因此抛出ArgumentNullException.然后,由于TakeRandom()成为e的方法,也许它应该是NullReferenceException.但是如果它是NullReferenceException,如果我尝试在TakeRandom()中使用e的成员,那么无论如何都会抛出NullReferenceException.
也许我应该使用Reflector达到峰值并找出框架的作用.
解决方法
你应该抛出一个ArgumentNullException.您正在尝试进行参数验证,因此应该抛出一个针对参数验证的异常. NullReferenceException不是参数验证异常.这是一个运行时错误.
不要忘记,扩展方法只是引擎盖下的静态方法,可以这样调用.虽然表面上看起来似乎有理由在扩展方法上抛出NullReferenceException,但对静态方法这样做是没有意义的.无法确定方法中的调用约定,因此ArgumentException是更好的选择.
此外,您不应该显式抛出NullReferenceException.这应该只由CLR抛出.当明确抛出通常仅由CLR抛出的异常时,会发生细微差别.
这也接近于以下的欺骗
> ArgumentNullException or NullReferenceException from extension method?