sustituir los elementos de cadena múltiple en C#

Question

¿Hay una mejor manera de hacer esto ...

MyString.Trim().Replace("&", "and").Replace(",", "").Replace(" ", " ") 
     .Replace(" ", "-").Replace("'", "").Replace("/", "").ToLower(); 

He ampliado la clase cadena para mantenerla baja a un puesto de trabajo, pero hay una manera más rápida?

public static class StringExtension 
{ 
    public static string clean(this string s) 
    { 
     return s.Replace("&", "and").Replace(",", "").Replace(" ", " ") 
       .Replace(" ", "-").Replace("'", "").Replace(".", "") 
       .Replace("eacute;", "é").ToLower(); 
    } 
} 

---

Sólo por diversión (y para detener los argumentos en los comentarios) He empujó una esencia a la evaluación comparativa de los diversos ejemplos a continuación.

https://gist.github.com/ChrisMcKee/5937656

La opción de expresiones regulares puntuaciones terriblemente; la opción del diccionario aparece más rápido; la versión larga de la sustitución de stringbuilder es ligeramente más rápida que la mano corta.

Answer 1

Más rápido - no. Más efectivo: sí, si usará la clase StringBuilder. Con su implementación, cada operación genera una copia de una cadena que bajo ciertas circunstancias puede afectar el rendimiento. Las cadenas son objetos inmutables por lo que cada operación solo devuelve una copia modificada.

Si espera que este método sea llamado activamente en múltiples Strings de longitud significativa, podría ser mejor "migrar" su implementación a la clase StringBuilder. Con él, cualquier modificación se realiza directamente en esa instancia, por lo que ahorra operaciones de copia innecesarias.

public static class StringExtention 
{ 
    public static string clean(this string s) 
    { 
     StringBuilder sb = new StringBuilder (s); 

     sb.Replace("&", "and"); 
     sb.Replace(",", ""); 
     sb.Replace(" ", " "); 
     sb.Replace(" ", "-"); 
     sb.Replace("'", ""); 
     sb.Replace(".", ""); 
     sb.Replace("eacute;", "é"); 

     return sb.ToString().ToLower(); 
    } 
} 

Answer 2

esto será más eficiente:

public static class StringExtension 
{ 
    public static string clean(this string s) 
    { 
     return new StringBuilder(s) 
       .Replace("&", "and") 
       .Replace(",", "") 
       .Replace(" ", " ") 
       .Replace(" ", "-") 
       .Replace("'", "") 
       .Replace(".", "") 
       .Replace("eacute;", "é") 
       .ToString() 
       .ToLower(); 
    } 
} 

Answer 3

Tal vez un poco más legible?

public static class StringExtension { 

     private static Dictionary<string, string> _replacements = new Dictionary<string, string>(); 

     static StringExtension() { 
      _replacements["&"] = "and"; 
      _replacements[","] = ""; 
      _replacements[" "] = " "; 
      // etc... 
     } 

     public static string clean(this string s) { 
      foreach (string to_replace in _replacements.Keys) { 
       s = s.Replace(to_replace, _replacements[to_replace]); 
      } 
      return s; 
     } 
    } 

También añadir nuevo en la sugerencia de la ciudad sobre StringBuilder ...

Answer 4

que estoy haciendo algo similar, pero en mi caso estoy haciendo serialización/De-serialización así que tengo que ser capaz de ir ambas direcciones. Creo que el uso de una cadena [] [] funciona de forma casi idéntica al diccionario, incluida la inicialización, pero también puede ir en la otra dirección, devolviendo los sustitutos a sus valores originales, algo que el diccionario realmente no está configurado para hacer.

Editar: Puede utilizar Dictionary<Key,List<Values>> el fin de obtener mismo resultado que la cadena [] []

Answer 5

Si usted es simplemente después de una solución bastante y no es necesario para ahorrar unos cuantos nanosegundos, ¿qué tal un poco de azúcar LINQ ?

var input = "test1test2test3"; 
var replacements = new Dictionary<string, string> { { "1", "*" }, { "2", "_" }, { "3", "&" } }; 

var output = replacements.Aggregate(input, (current, replacement) => current.Replace(replacement.Key, replacement.Value)); 

Answer 6

Hay una cosa que se puede optimizar en las soluciones sugeridas. Tener muchas llamadas al Replace() hace que el código haga múltiples pasadas sobre la misma cadena. Con cadenas muy largas, las soluciones pueden ser lentas debido a fallas en la capacidad de caché de la CPU. Puede ser uno debería considerar replacing multiple strings in a single pass.

Answer 7

string input = "it's worth a lot of money, if you can find a buyer."; 
for (dynamic i = 0, repl = new string[,] { { "'", "''" }, { "money", "$" }, { "find", "locate" } }; i < repl.Length/2; i++) { 
    input = input.Replace(repl[i, 0], repl[i, 1]); 
} 

Answer 8

Otra opción es el uso de LINQ

[TestMethod] 
public void Test() 
{ 
    var input = "it's worth a lot of money, if you can find a buyer."; 
    var expected = "its worth a lot of money if you can find a buyer"; 
    var removeList = new string[] { ".", ",", "'" }; 
    var result = input; 

    removeList.ToList().ForEach(o => result = result.Replace(o, string.Empty)); 

    Assert.AreEqual(expected, result); 
}