Devo gettare ArgumentNullException se una stringa è vuota?

Question

Io lavoro su un metodo che fa qualcosa dato un parametro stringa. Un valore valido per il parametro stringa è diverso da null o string.Empty. Quindi il mio codice sembra così.

 
private void SomeMethod(string someArgument) 
{ 
    if(string.IsNullOrEmpty(someArgument)) 
     throw new ArgumentNullException("someArgument"); 

    // do some work 
} 

Niente di troppo eccitante. La mia domanda è, va bene lanciare un ArgumentNullException anche se la stringa è uguale a string.Empty? Perché tecnicamente non è nulla. Se ritieni che non dovrebbe gettare ArgumentNullException quale eccezione dovrebbe essere generata?

Answer 1

ArgumentException deve essere generato per il caso String.Empty. Ciò indicherebbe un problema diverso da null. Per evitare un NullReferenceException prima ho verificato il null, quindi ho tagliato e verificato il caso vuoto per impedire il passaggio di spazi bianchi.

private void SomeMethod(string someArgument) 
{ 
    if(someArgument == null) 
     throw new ArgumentNullException("someArgument"); 

    if (someArgument.Trim() == String.Empty) 
     throw new ArgumentException("Input cannot be empty", "someArgument"); 

    // do some work 
} 

Come di .NET 4.0 è possibile utilizzare il metodo String.IsNullOrWhiteSpace per eseguire questi controlli in un colpo solo. In questo modo si rinuncia alla possibilità di specificare un tipo di eccezione granulare, quindi opterei per lo ArgumentException e aggiornerei il messaggio di conseguenza.

Answer 2

Si dovrebbe gettare un ArgumentException se una stringa vuota non è un input accettato per il metodo. Può essere molto confuso per i clienti se si lancia un ArgumentNullException mentre non hanno fornito un argomento null.

È semplicemente un altro caso d'uso. Potresti anche avere metodi che non accettano valori di input nulli ma che accettano stringhe vuote. È importante essere coerenti nell'intera applicazione.

Answer 3

ArgumentNullException viene talvolta utilizzato in .NET Framework per il caso String.IsNullOrEmpty - un esempio è System.Windows.Forms.Clipboard.SetText.

Quindi penso che sia ragionevole fare lo stesso nel codice, a meno che non ci sia un valore reale nel distinguere i due casi.

Si noti che questa e altre eccezioni derivate da ArgumentException indicano generalmente un errore di programmazione e pertanto devono fornire le informazioni necessarie per aiutare uno sviluppatore a diagnosticare il problema. Personalmente ritengo improbabile che uno sviluppatore possa trovarlo confuso se si utilizza ArgumentNullException per un argomento stringa vuoto, specialmente se si documenta questo comportamento come nell'esempio seguente.

Answer 4

Considerando tutto ciò che è stato detto (Joe/Ahmad Mageed), avrei creato un'eccezione per quel caso.

class ArgumentNullOrEmptyException : ArgumentNullException 

Answer 5

Questo dipende davvero dalle circostanze.

La domanda si riduce a, è davvero un errore? Con questo voglio dire che hai sempre un valore ? Se lo fai, allora probabilmente la soluzione migliore qui sta creando il proprio Exception, forse in questo modo:

class StringEmptyOrNullException : Exception 
{ 
} 

Dove si può anche aggiungere i propri costruttori e ha aggiunto informazioni ecc

Se, tuttavia, non è un "eccezionale" sta accadendo nel tuo programma, se fosse probabilmente un'idea migliore per restituire null dal metodo e gestirlo da lì.Basta ricordare, Exception sono per condizioni eccezionali.

Spero che questo aiuti,

Kyle

Answer 6

Perché non utilizzare questo codice?

private void SomeMethod(string someArgument) 
{ 
//chek only NULL 
if(ReferenceEquals(someArgument,null)) 
    throw new ArgumentNullException("someArgument"); 

// and after trim and check 
if (someArgument.Trim() == String.Empty) 
    throw new ArgumentException("Input cannot be empty", "someArgument"); 

// do some work 
}