¿Puedo probar si una expresión regular es válida en C# sin arrojar la excepción?

Question

Permito a los usuarios ingresar una expresión regular para que coincida con las direcciones IP, para hacer una filtración IP en un sistema relacionado. Me gustaría validar si las expresiones regulares ingresadas son válidas ya que muchos usuarios se meterán en problemas, aunque con buenas intenciones.

Por supuesto, puedo hacer un Regex.IsMatch() dentro de un try/catch y ver si explota de esa manera, pero ¿hay alguna forma más inteligente de hacerlo? La velocidad no es un problema, simplemente prefiero evitar excepciones sin ningún motivo.

Answer 1

Siempre y cuando capture excepciones muy específicas, simplemente haga la prueba/captura.

Las excepciones no son malas si se usan correctamente.

Answer 2

Tengo un método para probar si un RegEx es válido, pero simplemente ajusta la expresión regular en un Try/Catch. No estoy seguro de si hay una mejor manera de hacerlo, pero no pude encontrar uno.

Answer 3

Una expresión regular mal formada no es la peor de las razones para una excepción.

A menos que renuncie a un subconjunto muy limitado de la sintaxis de expresiones regulares - y luego escribir una expresión regular (o un analizador) para que - creo que no tienen otra manera de probar si es válido, pero para tratar de construir una establecer la máquina y hacer que coincida con algo.

Answer 4

No sin mucho trabajo. El análisis Regex puede ser bastante complicado, y no hay nada público en el Marco para validar una expresión.

System.Text.RegularExpressions.RegexNode.ScanRegex() parece ser la función principal responsable de analizar una expresión, pero es interna (y arroja excepciones para cualquier sintaxis no válida de todos modos). Por lo tanto, se le solicitará que vuelva a implementar la funcionalidad de análisis, que sin dudas fallará en casos límite o actualizaciones de Framework.

Creo que simplemente capturar la ArgumentException es una buena idea, como es probable que tenga en esta situación.

Answer 5

En .NET, a menos que escriba su propio analizador de expresiones regulares (que desaconsejaría encarecidamente), casi con seguridad tendrá que ajustar la creación del nuevo objeto Regex con un try/catch.

Answer 6

Dependiendo de quién sea el objetivo para esto, sería muy cuidadoso. No es difícil construir expresiones regulares que puedan retroceder sobre sí mismas y consumir mucha CPU y memoria: pueden ser un vector eficaz de denegación de servicio.

Answer 7

Creo que las excepciones son correctas en este caso.

Esto es lo que arme:

private static bool IsValidRegex(string pattern) 
{ 
    if (string.IsNullOrEmpty(pattern)) return false; 

    try 
    { 
     Regex.Match("", pattern); 
    } 
    catch (ArgumentException) 
    { 
     return false; 
    } 

    return true; 
} 

Answer 8

Al utilizar el método siguiente puede comprobado si su expresión reguler es válida o no. aquí testPattern es el patrón que debes verificar.

public static bool VerifyRegEx(string testPattern) 
{ 
    bool isValid = true; 
    if ((testPattern != null) && (testPattern.Trim().Length > 0)) 
    { 
     try 
     { 
      Regex.Match("", testPattern); 
     } 
     catch (ArgumentException) 
     { 
      // BAD PATTERN: Syntax error 
      isValid = false; 
     } 
    } 
    else 
    { 
     //BAD PATTERN: Pattern is null or blank 
     isValid = false; 
    } 
    return (isValid); 
}