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达到峰值并找出框架的作用.
解决方法
不要忘记,扩展方法只是引擎盖下的静态方法,可以这样调用.虽然表面上看起来似乎有理由在扩展方法上抛出NullReferenceException,但对静态方法这样做是没有意义的.无法确定方法中的调用约定,因此ArgumentException是更好的选择.
此外,您不应该显式抛出NullReferenceException.这应该只由CLR抛出.当明确抛出通常仅由CLR抛出的异常时,会发生细微差别.
这也接近于以下的欺骗
> ArgumentNullException or NullReferenceException from extension method?
原文地址:https://www.jb51.cc/csharp/98594.html
版权声明:本文内容由互联网用户自发贡献,该文观点与技术仅代表作者本人。本站仅提供信息存储空间服务,不拥有所有权,不承担相关法律责任。如发现本站有涉嫌侵权/违法违规的内容, 请发送邮件至 dio@foxmail.com 举报,一经查实,本站将立刻删除。