IndexNotFoundException versus NullReferenceException

Question

Ho il seguente codice che tenta di catturare un riferimento null. Quindi genera un'eccezione con un motivo più chiaro per l'errore specificato nella proprietà del messaggio.

Che tipo di eccezione deve inviare? Un IndexOutOfRangeException?

var existing = this.GetByItemId(entity.ItemId); // int or long 
if (existing == null) 
{ 
    throw new IndexOutOfRangeException("The specified item does not exist."); 
} 

var price = existing.Price; 

o uno NullReferenceException?

var existing = this.GetByItemId(entity.ItemId); 
if (existing == null) 
{ 
    throw new NullReferenceException("The specified item does not exist."); 
} 

var price = existing.Price; 

oppure, dovremmo lasciare che l'eccezione faccia il suo corso?

var existing = this.GetByItemId(entity.ItemId); 
var price = existing.Price; // NullReferenceException coming your way 

Il motivo si tende a non farlo ultima opzione, è che il NullReferenceException default è luce dettaglio ed appena afferma

riferimento non impostato a un'istanza di un oggetto.

Quale, a dire il vero, potrebbe benissimo essere il messaggio di errore più inutile in C#.

Answer 1

userei un'eccezione personalizzato per questo (qualche cosa come ItemNotFoundException).

A NullReferenceException o IndexOutOfRangeException potrebbero essere lanciati da qualcos'altro all'interno di this.GetByItemId() o nel Framework da qualche parte.

Il chiamante potrebbe voler eseguire un'azione di follow-up se l'elemento non compare nella raccolta (ad esempio aggiungendolo). L'utilizzo della propria eccezione consente al chiamante di chiamare l'catch in modo specifico e reagire di conseguenza.

Answer 2

Un'eccezione personalizzata con la descrizione della vostra scelta dovrebbe farlo:

if (existing == null) 
    { 
     throw new EntityMissingException("'existing' does not exist (ironic, isn't it?)."); 
    } 

Answer 3

Il NullReferenceException indica che si desidera accedere a un membro di riferimento null. Normalmente non si dovrebbe mai lanciarlo (a meno che non si implementi un interprete o qualcosa di simile).

Se il entity è un parametro, quindi direi che questo è un ArgumentException. Se dici che questo non dovrebbe mai accadere in circostanze normali, allora questo è piuttosto un InvalidOperationException.