.Trim() quando la stringa è vuota o nulla

Question

Sto ricevendo alcuni dati dal client sotto forma di JSON. Sto scrivendo questo:

string TheText; // or whould it be better string TheText = ""; ? 
TheText = ((serializer.ConvertToType<string>(dictionary["TheText"])).Trim()); 

Se la variabile che viene analizzato da JSON torna vuoto, fa questo incidente codice quando chiamo il metodo .Trim()?

Grazie.

Answer 1

Se il serializzatore restituisce una stringa vuota, Trim non farà nulla.

Se il serializzatore restituisce null, si riceverà NullReferenceException sulla chiamata a Trim.

Il tuo codice sarebbe meglio scritto (per quanto riguarda l'inizializzazione è in questione) in questo modo:

string theText = 
      ((serializer.ConvertToType<string>(dictionary["TheText"])).Trim()); 

v'è alcun punto nel dichiarare e inizializzare la variabile e il immediatamente assegnando ad esso.

Di seguito sarebbe più sicuro, se non si sa cosa potrebbe restituire il serializzatore:

string theText = ((serializer.ConvertToType<string>(dictionary["TheText"]))); 

if(!string.IsNullOrEmpty(theText)) 
{ 
    theText = theText.Trim(); 
} 

Answer 2

Innanzitutto, no, sarebbe non meglio inizializzare TheText a "". Ti stai assegnando subito dopo. (E se hai mai bisogno di usare una stringa vuota, usa invece string.Empty. Di solito è più chiaro ed è solo una pratica generalmente migliore.)

In secondo luogo, no, non si blocca - Trim() funziona perfettamente su una stringa vuota . Se per "vuoto" intendi che può essere null, allora sì, si bloccherà; si potrebbe risolvere il problema che, utilizzando l'operatore null coalesce:

string TheText = (serializer.ConvertToType<string>(dictionary["TheText"]) ?? string.Empty).Trim(); 

Answer 3

Calling Trim() su una stringa vuota si tradurrà in una stringa vuota.Chiamata Trim() su null getterà NullReferenceException

Answer 4

Se si dispone di un paio di campi da tagliare, ma le vostre eccezioni ottenendo per quei record che hanno valori nulli in determinati campi, quindi scrivendo un metodo di estensione rapido sarà il metodo più semplice:

public static class ExtensionMethods 
    { 
     public static string TrimIfNotNull(this string value) 
     { 
      if (value != null) 
      { 
       return value.Trim(); 
      } 
      return null; 
     } 
    } 

esempio esempio di utilizzo:

string concatenated = String.Format("{0} {1} {2}", myObject.fieldOne.TrimIfNotNull(), myObject.fieldTwo.TrimIfNotNull(), myObject.fieldThree.TrimIfNotNull()); 

Answer 5

Come suggerito in alcuni dei commenti, si può ora utilizzare C# 6 operatori Null-condizionale con la seguente sintassi:

string TheText = (serializer.ConvertToType<string>(dictionary["TheText"]))?.Trim(); 

Documentazione: https://msdn.microsoft.com/en-us/library/dn986595.aspx

Answer 6

È possibile utilizzare elvis operatore:

GetNullableString()?.Trim(); // returns NULL or trimmed string 

Answer 7

È possibile utilizzare questo codice come beblow

string theText = (((serializer.ConvertToType<string>(dictionary["TheText"])))+ string.Empty).Trim();