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

发布时间：2020-12-15 18:28:16 所属栏目：百科来源：网络整理

导读：假设我有一个扩展方法 public static T TakeRandomT(this IEnumerableT e){ ... 要验证参数e,我应该： A)if(e == null)抛出新的NullReferenceException() B)if(e == null)抛出新的ArgumentNullException(“e”) C)不检查e 什么是共识？我的第一个想法是始终

假设我有一个扩展方法

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?

（编辑：李大同）

【声明】本站内容均来自网络，其相关言论仅代表作者个人观点，不代表本站立场。若无意侵犯到您的权利，请及时与联系站长删除相关内容!