.Trim() cuando la cadena está vacía o nula

Question

Recibo algunos datos del cliente en forma de json. estoy escribiendo esto:

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

Si la variable que está siendo analizada desde JSON regresa vacía, hace esto accidente código cuando llamo el método .trim()?

Gracias.

Answer 1

Si el serializador devuelve una cadena vacía, Trim no hará nada.

Si el serializador devuelve null, obtendrá un NullReferenceException en la llamada al Trim.

Su código sería mejor escrito (por lo que la inicialización se refiere) de esta manera:

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

No hay ningún punto en declarar e inicializar la variable y el inmediato asignar a la misma.

El siguiente sería más seguro, si usted no sabe lo que podría devolver el serializador:

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

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

Answer 2

En primer lugar, no, no sería ser mejor para inicializar TheText a "". Lo estás asignando justo después. (Y si alguna vez es necesario utilizar una cadena vacía, utilice string.Empty lugar Por lo general es más clara y es generalmente mejor práctica..)

En segundo lugar, no, no va a estrellarse - Trim() funciona bien en una cadena vacía . Si por "vacío" quiere decir que puede ser null, entonces sí, se bloqueará; usted podría arreglar eso usando el operador nulo se unen:

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

Answer 3

Calling Trim() en una cadena vacía se traducirá en una cadena vacía.Llamando Trim() en null lanzará NullReferenceException

Answer 4

Si usted tiene unos pocos campos que desea recortar, pero sus excepciones conseguir para aquellos registros que tienen valores nulos en algunos campos, a continuación, escribir un método de extensión rápida será el método más sencillo:

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

ejemplo ejemplo de uso:

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

Answer 5

Como se sugiere en algunos de los comentarios, ahora se puede utilizar C# 6 operadores nulos-condicional con esta sintaxis:

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

Documentación: https://msdn.microsoft.com/en-us/library/dn986595.aspx

Answer 6

Puede utilizar Elvis operador:

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

Answer 7

Usted puede utilizar este código como beblow

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