Validacién en C

[jam1138]

Saludos!

Necesito hacer una validación a un dato ingresado por el usuario. Las condiciones a cumplir son:
1) Que sea un número.
2) Que dicho número no sea menor que cero ni mayor que nueve (ó que sea un número entre 0 y 9 )

Ya lo he intentado.. y suena fácil... pero no le hayo . ¿Qué hay de malo con esto?

Código:

do {
    scanf("%d", &variable);
} while (!isdigit(variable) || variable<0 || variable>9);

Obviamente antes insertando las librerias correspondientes (ctype.h)...

El resultado que obtengo con esto es que si ingreso una letra me produce un ciclo infinito... lo extraño es que es el mismo comportamiento si no niego isdigit... e incluso si no lo inserto...

Utilizo scanf porque tengo que dar la opción a el usuario de que "corrija" si ha escrito un número que no quiere...

La verdad me apena decir que no le hayo... ¿en qué falla?, ¿qué es lo correcto??....

Gracias de antemano.

[Eternal Idol]

while (!isdigit(variable) || variable<0 || variable>9);

Estas comprobando que alguna de estas tres condiciones se cumpla, estas poniendo || que sinigica or cuando deberias poner && que significa and.

[rir3760]

La funcion isdigit se utiliza para verificar si un determinado caracter es un digito o no, no la puedes utilizar para verificar si la entrada en tu caso es valida.
Lo que necesitas hacer es:

* Revisar el valor que devuelve scanf
* Verificar que los limites (0 y 9) no se sobrepasen.

Por ejemplo:

Código:

while (scanf("%d", &variable) != 1 || variable < 0 || variable > 9)
{
   /* pedir la entrada de nuevo */
}

Un saludo

[Eternal Idol]

Ese codigo sigue estando mal si, como dice el primer post, todas las condiciones deben cumplirse.

[rir3760]

El fragmento de codigo que mencione en el mensaje anterior es correcto ya que la condicion de salida del bucle while es que scanf devuelva un valor de 1 y que el numero almacenado en 'variable' sea igual o mayor que 0 e igual o menor que 9.

Un pequeño programa de prueba es:

Código:

#include <stdio.h>
#include <stdlib.h>

int main(void)
{
   int variable;
   
   printf("Introduce un numero: ");
   fflush(stdout);
   
   while (scanf("%d", &variable) != 1 || variable < 0 || variable > 9){
      puts("El rango valido es solo de 0 a 9.");
      printf("Introduce un numero: ");
      fflush(stdout);
   }
   
   printf("El valor de \'variable\' es %d\n", variable);
   return EXIT_SUCCESS;
}

Un saludo

[Eternal Idol]

Si, pone q y contame que pasa.

[Eternal Idol]

Asi funcionaria, igual me parece horrible ...

Código:

while (scanf("%d", &variable) != 1 || variable < 0 || variable > 9){
      puts("El rango valido es solo de 0 a 9.");
      printf("Introduce un numero: ");
      fflush(stdin);
}

[jam1138]

Iniciado por Eternal Idol

Estas comprobando que alguna de estas tres condiciones se cumpla, estas poniendo || que sinigica or cuando deberias poner && que significa and.

..... .... ... según sé... OR ma dará verdadero si alguna y al menos una condición me devuelve verdadero... y es el valor verdadero que necesito para que me repita lo que esta entre el do y el while() ¿no?

Entonces, yo solo necesito que al menos una condición me devuelva verdadero.. . teniendo lo que tengo ahora

Código:

!isdigit(variable) || variable<0 || variable>9

Debería tener los siguentes resultados:

Ingresando un valor correcto como 5:
- !isdigit -- false (sí lo es)
- variable<0 -- false (es mayor)
- variable >9 -- false (es menor)
Resultado??.. continuo... me quedo con el valor

Ingresando un número fuera de rango como lo es 15:
- !isdigit -- false (sí lo es)
- variable<0 -- false (es mayor)
- variable>9 -- TRUE (en efecto es mayor)
Resultado... teniendo un TRUE y utlizardo OR ... se repite el ciclo

Ingrasando una letra:
- !isdigit -- TRUE (no lo es) <- Y con esto debería bastar y sobrar
- variable<0 -- ¿? ni idea... no es número.. ¿por su valor en ASCII??
- variable>9 -- ¿? lo mismo que lo anterior
Resultado: Por no ser dígito tenemos TRUE.. debería repetir el ciclo.


... es por eso que según yo debe de manejarse OR y no AND ... ya que AND me regresaria verdadero solo sí todas las condicones se cumplen... si esto es cierto el simple hecho de condicionar variable<0 && variable>9 no aplicaría.. prácticamente sería una contradicción

Con un valor 10:
- variable<0 -- false
- variable>9 -- true
Resultado: Continuo.. me quedo con 10 ya que por el AND tener un false y un true me regresa FALSO... pero el 10 no sería un número válido para mi...

Lo he comprobado.. tampoco hablo solo por hablar... temo que el AND no me funciona...

También he intentado ( ya casi adivinando) con while (!isdigit(variable) && (variable<0 || variable>9)) y tampoco he obtenido buenos resultados...

Temo que la estructura que necesito es de do{ } while();... No sé cómo adaptar lo que me sugiere rir3760... básicamente dónde va el fflush(stdin) y qué hace???.... ¿por qué es horrible Eternal_Idol?

... Prácticamente lo que tengo funciona de alguna manera... ingreso una letra y me repite el ciclo... pero ¿por qué un ciclo infinito???, ¿por qué no me respeta el scanf???

Mil gracias por la atención prestada.. Saludos!

[Eternal Idol]

El fflush(stdin) tenes que ponerlo despues del scanf para que limpie el buffer, en este caso por ejemplo cuando te ponen una letra y entonces por eso sigue repitiendo el ciclo, limpiando el buffer ya no pasa.

Diferencias de criterio, yo lo haria asi:

Código:

for (;;)
{
  if ( (scanf("%d", &variable) == 1) && (variable >= 0) && (variable <= 9) ) { break; }
  puts("El rango valido es solo de 0 a 9.");
  printf("Introduce un numero: ");
  fflush(stdin);
}

[MaxExtreme]

Hay otro fallo, y es que si se incluye el scanf en la condición junto con variable>0 && variable<9 puede que ello se comprae antes o después.

Depende del compilador, las comparaciones se empiezan por la primera, la última...

[Spike_ak]

Hola,porque no pones una condicion para validar el numero? y usas char?