0

Necesito hacer una validación de un string con expresiones regulares para que se repita un loop que tiene la condición de que hasta que el usuario ingrese una identificación alfanumérica, no pueda seguir adelante con el ingreso de datos. La expresión regular ya está armada correctamente.

Lo que me falla es la lógica para que este loop se repita hasta que la identificación sea una alfanumérica de 4 dígitos y ahí sí, se podría continuar ingresando datos.

Código:

public static void validaralfanum(string unNum)
{
    bool bandera = false;
    Regex regex = new Regex(@"^[a-zA-Z0-9\s,]*$");

    do
    {
        if (!regex.IsMatch(unNum))
        {
            Console.WriteLine("El ID debe ser alfanumérico");
        }
        else if (unNum.ToString().Length > 4 || unNum.ToString().Length < 4)
        {
            Console.WriteLine("El ID debe ser de 4 digitos");
        }
        else if (unNum == null || unNum == "")
        {
            Console.WriteLine("Ingrese un ID");
        }
        else
        {
            bandera = true;
        }
    }
    while (!bandera);
}

2 respuestas 2

1

El problema está en que unNum nunca se modifica dentro de esta función. Se acepta por parámetro, y por lo tanto las condiciones para abandonar el bucle nunca cambian. Deberías hacer el bucle conteniendo la entrada de datos del usuario, de forma que cuando lo ingrese correctamente, salga del bucle.

Resumiendo: Saca el bucle de la función de validación y ponlo en la superior.

1
  • Deberías mostrar cómo queda el código final, luego de hacer esos pasos.
    – user13558
    el 21 may. 2017 a las 22:57
0

Hay que actualizar el valor, unNum, o de lo contrario pasará lo siguiente:

El ID debe ser de 4 dígitos
El ID debe ser de 4 dígitos
El ID debe ser de 4 dígitos
...

También hay que poner string.IsNullOrEmpty en lugar de s == "" || s == null y una verificación de longitud en regex.

Regex regex = new Regex(@"^[a-zA-Z0-9\s,]{4}$");
do
{
    if (string.IsNullOrEmpty(unNum))
    {
        Console.WriteLine("Ingrese un ID");
    }
    else if (!regex.IsMatch(unNum))
    {
        Console.WriteLine("El ID debe ser alfanumérico");
    }
    else
    {
        bandera = true;
    }
    unNum = Console.ReadLine();
}
while (!bandera);

Ejemplo:

$ /usr/bin/mono Test.exe
123
El ID debe ser alfanumérico
*&^(^*
El ID debe ser alfanumérico
abcd
$
14
  • Pero en esta modificación que le hiciste a mi código tampoco actualizaste el valor, sigo teniendo el loop infinito.
    – Gonzalo
    el 21 may. 2017 a las 19:24
  • var unNum = Console.ReadLine(); es donde yo actualizo. Hay un ejemplo ahora :) el 21 may. 2017 a las 19:25
  • el problema es que me da un error, ya que yo ya defino alpha como parámetro de mi método y no me deja declararlo de nuevo.
    – Gonzalo
    el 21 may. 2017 a las 19:27
  • Está bien, var unNum = Console.ReadLine(); -> unNum = Console.ReadLine(); :) el 21 may. 2017 a las 19:28
  • 1
    gist.github.com/lol768/8dc865499c13c63b7be7423aec680c1c esta bien para mi el 21 may. 2017 a las 19:57

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.