Como evitar que se introduzcan valores no válidos

Iniciado por grodomio, 17 Mayo 2013, 12:47 PM

0 Miembros y 3 Visitantes están viendo este tema.

grodomio

Hola, tengo que hacer un método de una clase que compruebe si un valor introducido es válido o no.
Ese valor proviene de un menú que tiene las opciones desde 0 hasta 6 incluidas.

Con el código que pongo a continuación controlo los valores numéricos, pero no se cómo evitar que un usuario introduzca una letra por ejemplo.

bool Help::isvalid(int option) {
if(option < '0' || option > '6') {
return false;
}
return true;
}


Espero vuesta ayuda, gracias.

leosansan

#1
Cita de: grodomio en 17 Mayo 2013, 12:47 PM
Hola, tengo que hacer un método de una clase que compruebe si un valor introducido es válido o no.
Ese valor proviene de un menú que tiene las opciones desde 0 hasta 6 incluidas.

Con el código que pongo a continuación controlo los valores numéricos, pero no se cómo evitar que un usuario introduzca una letra por ejemplo.

bool Help::isvalid(int option) {
if(option < '0' || option > '6') {
return false;
}
return true;
}


Espero vuesta ayuda, gracias.

Cámbiala a:

Citarbool Help::isvalid(char option) {
   if(option >= '0' && option< '7') {
      return true;
   }
   return false;
}

Observa que opcion debe ser char, que en pricipio debía serlo porque tienes en las condiciones '1' y '6' que son caracteres y no enteros, que entonces serían en la condición 1 y 6 sin las comillas.

Saluditos!. ....  

grodomio

#2
Hola leosansan,

no veo la diferencia, lo que has hecho ha sido invertir las condiciones y los returns.

Sigue aceptando caracteres no numéricos como válidas.

leosansan

#3
Cita de: grodomio en 17 Mayo 2013, 13:39 PM
Hola leosansan,

no veo la diferencia, lo que has hecho ha sido invertir las condiciones y los returns.

Sigue aceptando caracteres no numéricos como válidas.


Como te lo he puesto si introduces un numero fuera de 0 y 6, que retorna true es decir lo acepta, o bien introduces un caracter no numérico, al no estar entre 0 y 6 retorna false, es decir, lo rechazaría.

Pero igual estoy "nublado" como el día. :P

Por ejemplo una salida a lo que propones:

Código (cpp) [Seleccionar]
Introduzca un  numero entre 0 y 6 :
3
3 es aceptado
Introduzca un  numero entre 0 y 6 :
6
6 es aceptado
Introduzca un  numero entre 0 y 6 :
8
8 es rechazado
Introduzca un  numero entre 0 y 6 :
a
a es rechazado
Introduzca un  numero entre 0 y 6 :


con el siguiente código de testeo:

Código (cpp) [Seleccionar]
#include <stdio.h>
#include <stdlib.h>

int main(void)
{
    char option;
    do{
        printf("Introduzca un  numero entre 0 y 6 :\n");
        scanf(" %c",&option);
        if (option>='0' && option<'7')
            printf("%c es aceptado\n",option);
        else
        printf("%c es rechazado\n",option);
    }while(1);
    return EXIT_SUCCESS;
}


Saluditos!. ... .  

[/size]

Caster

Yo creo que la solución sería cambiar el tipo de opción, en vez de ponerlo como int, ponlo como char.

Saludos

leosansan

#5
Cita de: Caster en 17 Mayo 2013, 14:19 PM
Yo creo que la solución sería cambiar el tipo de opción, en vez de ponerlo como int, ponlo como char.

Saludos


En la función que le pase estaba como char.

Ya he reeditado el mensaje y le pongo un código de prueba que usa ese sistema y funciona O.K.

Saluditos!. ....

Caster

Cita de: leosansan en 17 Mayo 2013, 14:22 PM

En la función que le pase estaba como char.

No me había fijado en eso, el tampoco se debió de fijar y solo copió las condiciones. Tal y como lo pusiste debería de funcionar.

Saludos

leosansan

Cita de: Caster en 17 Mayo 2013, 14:26 PM
No me había fijado en eso, el tampoco se debió de fijar y solo copió las condiciones. Tal y como lo pusiste debería de funcionar.

Saludos

Eso creo yo, por eso le he reeditado el primer mensaje con un zoom.

¡Hasta lueguito¡ .....

grodomio

Perdón por no especificarlo, pero estoy programando en C++ y cout no entiende de formatos, se lo traga todo, no le importa que sea int, char, ...

Voy a probar a ver si cambiando el tipo del parámetro de entrada de entero a char se soluciona.

Ya os comento.

Y gracias  a todos.

grodomio

#9
Perdón por no especificarlo, pero estoy programando en C++ y cou y cin no entienden de formatos, se lo traga todo, no le importa que sea int, char, ...

Voy a probar lo que dices  a ver si cambiando el tipo del parámetro de entrada de entero a char se soluciona.

Ya os comento.

Y gracias  a todos