Quale eccezione devo lanciare per un valore nullo imprevisto?

Question

Sono stato uno sviluppatore .NET per oltre un decennio quindi ecco una domanda vergognosa a cui non ho mai saputo la risposta. Capisco - se un argomento è nullo, posso lanciare un ArgumentNullException. Se cerco di dereferenziare un valore nullo, verrà generata una eccezione NullReferenceException.

Ma cosa succede se ho il codice come il seguente:

var someVitalObject = someServiceReference.GetVitalObject(); 

if (someVitalObject == null) 
{ 
    throw new IDontKnowWhatException(); // what exception should I throw here? 
} 

Ora, questo non è necessariamente un problema con il servizio per il quale un'eccezione avrebbe dovuto essere gettato in precedenza.

Answer 1

Vorrei buttare un System.NullReferenceException. Oppure, se non sei soddisfatto, potresti derivarne da un tipo più specializzato di eccezioni di riferimento null per i tuoi scopi.

Answer 2

È difficile dire senza vedere più contesto, ma forse System.InvalidOperationException?

L'eccezione che viene generata quando una chiamata al metodo non è valida per lo stato corrente dell'oggetto.

Answer 3

Considerare per creare la propria classe Exception ereditato dal ApplicationException o dalla Exception:

public sealed class MyException : Exception 
{ 
    ... 
} 

questo modo si sarà in grado di controllare il tipo di informazioni per memorizzare in esso.

Answer 4

Utilizzare solo System.ArgumentNullException solo quando si verifica direttamente un parametro del metodo, non quando si convalida il risultato di alcune chiamate.

Il tipo di eccezione che lancio dipende molto dal contesto. Ma in questo caso mi sarebbe probabilmente andare a fare un'eccezione personalizzato come:

public class VitalObjectNotAcquiredException : Exception { ... } 

Answer 5

Io di solito uso ArgumentNullException per gli oggetti passati in una funzione. Tutto il resto nullo relativo Io uso InvalidOperationException. In casi specifici creerò un'eccezione personalizzata se ha senso.

Answer 6

Persino, sceglierei in base al contratto del metodo GetVitalObject. Se questo metodo dovesse restituire un oggetto non nullo, lo cambierei in modo da generare un'eccezione in questo caso.

Se non si ha il controllo sul metodo o se restituire un valore nullo è corretto (ma non nel proprio contesto) andrei con un'eccezione personalizzata come già detto Mark e Dmitry.