C#: Le migliori pratiche per la convalida di "questo" argomento in metodi di estensione

Question

Diciamo che ho un metodo di estensione

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

Per convalidare l'argomento e, devo:

A) se (e == null) throw new NullReferenceException()
B) se (e == null) throw new ArgumentNullException ("e")
C) Non controllare la posta

Qual è il consenso?

Il mio primo pensiero è quello di convalidare sempre gli argomenti, quindi lanciare un'eccezione ArgumentNullException. Poi di nuovo, dato che TakeRandom() diventa un metodo di e, forse dovrebbe essere una NullReferenceException. Ma se si tratta di una NullReferenceException, se provo a utilizzare un membro di e all'interno di TakeRandom(), verrà comunque generata NullReferenceException.

Forse dovrei fare un picco usando Reflector e scoprire cosa fa il framework.

Answer 1

Si dovrebbe generare una ArgumentNullException. Stai tentando di eseguire la convalida degli argomenti e quindi dovresti generare un'eccezione sintonizzata sulla convalida degli argomenti. NullReferenceException non è un'eccezione di convalida argomento. È un errore di runtime.

Non dimenticare, i metodi di estensione sono solo metodi statici sotto il cofano e possono essere chiamati come tali. Mentre può sembrare in superficie avere un senso per lanciare una NullReferenceException su un metodo di estensione, non ha senso farlo per un metodo statico. Non è possibile determinare la convenzione di chiamata nel metodo e quindi ArgumentException è la scelta migliore.

Inoltre, non si dovrebbe mai lanciare esplicitamente una NullReferenceException. Questo dovrebbe essere lanciato solo dal CLR. Ci sono sottili differenze che si verificano quando si lanciano esplicitamente eccezioni che vengono normalmente lanciate solo dal CLR.

Questo è anche vicino a una vittima dei seguenti

• ArgumentNullException or NullReferenceException from extension method?

Answer 2

Per coerenza con gli operatori LINQ enumerabili, generare un'eccezione ArgumentNullException (non una NullReferenceException).

Effettuerò la convalida nel metodo TakeRandom perché la traccia dello stack renderà quindi chiaro che è TakeRandom a cui viene assegnato un argomento nullo.

Answer 3

Forse sono pazzo, ma dal momento che è un argomento, vorrei gettare un ArgumentNullException:/

generale la regola generale è di lanciare eccezioni che derivano da System.ApplicationException quando possibile. NullReferenceException è qualcosa che il framework/CLR genererebbe.

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

due categorie di eccezioni esistono sotto l'eccezione classe base:

il linguaggio comune runtime classi di eccezioni predefinite derivati ​​da SystemException.Le classi di eccezione dell'applicazione definite dall'utente derivano da ApplicationException.