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

前端之家收集整理的这篇文章主要介绍了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.f2er.com/csharp/98594.html

猜你在找的C#相关文章