C#: La mejor práctica para la validación de "este" argumento a los métodos de extensión

Question

Digamos que tengo un método de extensión

public static T TakeRandom<T>(this IEnumerable<T> e) 
{ 
    ... 

Para validar el argumento e, debo:

A) si (e == null) throw new NullReferenceException()
B) si (e == null) throw new ArgumentNullException ("e")
C) No comprobar el correo

¿Cuál es el consenso?

Mi primer pensamiento es siempre validar los argumentos, por lo que lanzo una ArgumentNullException. Por otra parte, dado que TakeRandom() se convierte en un método de e, tal vez debería ser una NullReferenceException. Pero si se trata de una NullReferenceException, si trato de usar un miembro de e dentro de TakeRandom(), se lanzará NullReferenceException de todos modos.

Tal vez debería aprovechar el Reflector y averiguar qué hace el framework.

Answer 1

Debería lanzar una ArgumentNullException. Está intentando hacer una validación de argumentos y, por lo tanto, debe arrojar una excepción ajustada a la validación de argumentos. NullReferenceException no es una excepción de validación de argumentos. Es un error de tiempo de ejecución.

No olvide que los métodos de extensión son solo métodos estáticos debajo del capó y se pueden llamar como tales. Aunque en apariencia puede parecer razonable arrojar una NullReferenceException en un método de extensión, no tiene sentido hacerlo para un método estático. No es posible determinar la convención de llamadas en el método y, por lo tanto, ArgumentException es la mejor opción.

Además, nunca deberías lanzar explícitamente una NullReferenceException. Esto solo debe ser lanzado por el CLR. Existen diferencias sutiles que ocurren al lanzar explícitamente excepciones que normalmente solo lanzan los CLR.

Esto también está cerca de una víctima de la siguiente

• ArgumentNullException or NullReferenceException from extension method?

Answer 2

Para coherencia con los operadores Enumerable LINQ, arroje una ArgumentNullException (no una NullReferenceException).

Haría la validación en el método TakeRandom porque la traza de la pila dejará en claro que es TakeRandom que objeta que se le dé un argumento nulo.

Answer 3

Tal vez estoy loco, pero ya que es un argumento, me lanzo una ArgumentNullException:/

general Sin embargo, la regla general es arrojar Excepciones derivadas de System.ApplicationException cuando sea posible. NullReferenceException es algo que lanzaría framework/CLR. existen

http://msdn.microsoft.com/en-us/library/system.exception(VS.71).aspx

dos categorías de excepciones en la excepción clase base:

la lengua común clases de excepción de tiempo de ejecución predefinidos derivados de SystemException.Las clases de excepción de aplicación definidas por el usuario derivaron de ApplicationException.