C#: convalida Argomento: null/stringhe vuote

Question

non so quante volte innumerevoli ho dovuto scrivere il codice per convalidare argomenti stringa:

public RoomName(string name) 
{ 
    if (string.IsNullOrEmpty(name)) 
    { 
     throw new ArgumentException("Cannot be empty", "name"); 
    } 
} 

Esiste un modo per evitare questo? Esiste un attributo o un meccanismo di progettazione per contratto per evitare questo? Non c'è modo di dire:

public RoomName(NotNullOrEmptyString name) 
{ 

senza dover effettivamente creare quel tipo?

Answer 1

Vedere anche C#: How to Implement and use a NotNull and CanBeNull attribute per ulteriori informazioni sui Contratti di codice, come possono essere implementati oggi in VS2008 e come saranno integrati in VS2010.

Answer 2

È possibile farlo tramite l'iniezione di codice con attributi.

Un'altra opzione per risparmiare un po 'di tempo per la codifica, ma che ti dà ancora molto controllo, sarebbe usare qualcosa come CuttingEdge.Conditions. Questo fornisce un'interfaccia fluida per il controllo argomento, in modo da poter scrivere:

name.Requires().IsNotNull(); 

Answer 3

Anche se la questione è stata risolta qualche tempo fa, ho pensato lo stesso problema ultimamente. Contratti di codice formalizzati (con verifica o assegni automatici) sembrano essere una buona idea, ma in generale, la loro capacità di verifica è abbastanza limitata, e per controlli semplici come il controllo di stringa vuota o vuota, richiedono tanto codice (o più) rispetto agli assegni antiquati.

Ironia della sorte, la migliore risposta a mio parere per la stringa caso è infatti uno o due classi che avvolgono una stringa che è stato controllato per non essere nullo, vuoto o bianco-spazio, e passare questa istanza in giro:

public class NonEmptyString : IComparable<NonEmptyString>, ... 
{ 
    private readonly string _value; 

    public NonEmptyString(string value) 
    { 
     if (value == null) 
     { 
      throw new ArgumentNullException("value"); 
     } 
     if (value.Length == 0) 
     {     
      throw NewStringIsEmptyException("value"); 
     } 
     _value = value; 
    } 

    public string Value 
    { 
     get { return _value; } 
    } 

    ... 
} 

public class NonWhiteSpaceString : NonEmptyString 
{ 
    .... 
} 

Certo, passando intorno a questi casi non vi impedisce di dover controllare se essi stessi sono nulli, ma ha alcuni grandi vantaggi:

• non c'è bisogno di controllare il vuoto o bianco- stringhe spaziali ancora e ancora, che possono essere soggette a errori nelle situazioni in cui la stringa è passato molto.
• Come ho fatto nella mia implementazione, il controllo di null è qualcosa di diverso dal controllo di un valore vuoto (o valore di spazi bianchi), perché si vuole lanciare una specifica ArgumentNullException nel primo caso, e qualche ArgumentException nel secondo.
• Segnala chiaramente il vincolo sul valore della stringa, proprio come ogni classe di wrapping deve fare. In effetti, se si dispone di una stringa che presenta vincoli e viene passata molto, consiglio sempre di racchiuderla in una classe che incapsula il controllo e mantiene il resto del codice fuori dai guai. Un buon esempio di ciò sono stringhe che devono soddisfare una certa espressione regolare. Tuttavia, sto deviando dalla domanda qui ...