微信公众号搜"智元新知"关注
微信扫一扫可直接关注哦!

C#:在扩展方法中验证“this”参数的最佳实践

假设我有一个扩展方法
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?

原文地址:https://www.jb51.cc/csharp/98594.html

版权声明:本文内容由互联网用户自发贡献,该文观点与技术仅代表作者本人。本站仅提供信息存储空间服务,不拥有所有权,不承担相关法律责任。如发现本站有涉嫌侵权/违法违规的内容, 请发送邮件至 dio@foxmail.com 举报,一经查实,本站将立刻删除。

相关推荐