Por curiosidad: ¿hay un enfoque mejor para hacer este reemplazo de cuerdas?

Question

Tengo el siguiente método donde necesito verificar algunas cadenas determinadas que podrían ser en cualquier caso y luego eliminarlas. Solo me preguntaba si había una mejor manera de realizar?

private void MyMethod(string Filter) 
{ 
    //need to remove <Filter> and </Filter> case in-sensitive 
    var result = Filter.ToLower().Replace("<filter>",""); 
    result = Filter.ToLower().Replace("</filter>,""); 

    ........................... 
} 

Answer 1

Un problema con este enfoque es que convertirá toda la cadena en minúsculas, no solo hará que las mayúsculas y minúsculas se vuelvan insensibles.

Se puede utilizar una expresión regular para hacer un caso partido insensetive:

string result = Regex.Replace(
    Filter, 
    "</?filter>", 
    String.Empty, 
    RegexOptions.IgnoreCase 
); 

Otra alternativa es utilizar el método IndexOf para localizar las cuerdas, ya que puede hacer una búsqueda que insensetive:

string result = Filter; 
int index; 
while ((index = IndexOf("<filter>", StringComparison.OrdinalIgnoreCase)) != -1) { 
    result = result.Remove(index, 8); 
} 
while ((index = IndexOf("</filter>", StringComparison.OrdinalIgnoreCase)) != -1) { 
    result = result.Remove(index, 9); 
} 

Answer 2

Replace llamadas a código no administrado que está implementado en C++ que me imagino que será difícil de superar.

Sin embargo, puedo ver que sigue usando .ToLower() que puede cortar a una llamada y mantener la cadena.

Answer 3

Comprobar esta respuesta: Is there an alternative to string.Replace that is case-insensitive?

Es posible que desee hacer una comparación con un control del funcionamiento. Perfile esto con un perfilador. Es la única forma de saber realmente, qué es más rápido.

Pero, sinceramente, ¿el rendimiento realmente importa? ¿Con qué frecuencia estás haciendo esto? No puedo verte haciendo esto tan a menudo, ese rendimiento se convertirá en un problema ...

Puedes probar Regex.Replace, con una sustitución insensible a mayúsculas y minúsculas. Esto no es mas rapido Pero es insensible a mayúsculas y minúsculas.

Answer 4

En cualquier caso, está colocando aquí la cadena original, lo que podría no ser una buena cosa.

Answer 5

Si el código proporcionado trabaja para usted, que esto será más rápido:

private void MyMethod(string Filter) 
{ 
    //need to remove <Filter> and </Filter> case in-sensitive 
    var result = Filter.ToLower().Replace("</filter>",""); 

    ........................... 
} 

como resultado de la primera instrucción se ignora.

Answer 6

Depende de algunas cosas, cuánto tiempo la cadena de filtro es, etc.
Así que tendrá que medir.

Pero yo esperaría que un (único) RegEx sea más rápido aquí.