Error usando FOR y Do-While anidados

[RGT]

Código:

int main()
{
    //Declaración de variables
    int liContador, laiNumeros[20];

    //Procesamiento
    for (liContador = 0; liContador < 20; liContador++)
    {
        do
        {
            //Entrada de datos
            printf("Ingrese #%d: ", liContador + 1);
            scanf("%d", &laiNumeros[liContador]);

            if (laiNumeros[liContador] > 10 || laiNumeros[liContador] < 1)
                printf("\n\tError!, debe ingresar un valor entre 1 y 10.\n");
        } while (laiNumeros[liContador] > 10 && laiNumeros[liContador] < 1);
    }

    //Salida de datos

    return 0;
}

Hola, quiero un programa que lea 20 números entre el 1 y el 10.
La unica forma que se me ocurre para validar que el valor que se ingresa no es menor que 1 y no es mayor que 11 es usando el DO WHILE para que vuelva a preguntarle al usuario el dato pero, hay un problema!.

El programa sigue ejecutandose aún saliendo el error.

Screenshot del problema:

[ecfisa]

Hola RGT.

Intenta de este modo:

Código C:

... 
{
  int con=0, num[20];
  int inOneTen;
 
  do {
    do {
        printf("Ingrese #%d: ", con+1 );
        scanf("%d", &num[con]);
        inOneTen = num[con] >= 1 && num[con] <= 10;
        if (!inOneTen) printf("\n\tError!, debe ingresar un valor entre 1 y 10.\n\n");
    } while (!inOneTen);
    con++;
  } while (con < 20);
 
  ...
}
 

Saludos.

[eferion]

Código C++:

} while (laiNumeros[liContador] > 10 && laiNumeros[liContador] < 1)
 

fíjate que ese while se repetirá mientras el valor introducido sea mayor que 10 y menor que 1, algo que, obviamente, nunca va a suceder.

Tienes que cambiar ese and '&&' por un '||' para dejar la comparación exactamente igual a la que usas para sacar el mensaje de error.

Una forma de evitar este tipo de problemas es almacenar el resultado de la comparación en una variable. Así puedes reutilizar dicho resultado de forma sencilla:

Código C++:

int main()
{
    //Declaración de variables
    int liContador, laiNumeros[20];
 
    //Procesamiento
    for (liContador = 0; liContador < 20; liContador++)
    {
        int hayError = 0;
        do
        {
            //Entrada de datos
            printf("Ingrese #%d: ", liContador + 1);
            scanf("%d", &laiNumeros[liContador]);
 
            hayError = (laiNumeros[liContador] > 10 || laiNumeros[liContador] < 1);
            if ( hayError )
                printf("\n\tError!, debe ingresar un valor entre 1 y 10.\n");
        } while ( hayError);
    }
 
    //Salida de datos
 
    return 0;
}
 

Un saludo

[RGT]

Iniciado por eferion

Código C++:

} while (laiNumeros[liContador] > 10 && laiNumeros[liContador] < 1)
 

fíjate que ese while se repetirá mientras el valor introducido sea mayor que 10 y menor que 1, algo que, obviamente, nunca va a suceder.

Tienes que cambiar ese and '&&' por un '||' para dejar la comparación exactamente igual a la que usas para sacar el mensaje de error.

Una forma de evitar este tipo de problemas es almacenar el resultado de la comparación en una variable. Así puedes reutilizar dicho resultado de forma sencilla:

Código C++:

int main()
{
    //Declaración de variables
    int liContador, laiNumeros[20];
 
    //Procesamiento
    for (liContador = 0; liContador < 20; liContador++)
    {
        int hayError = 0;
        do
        {
            //Entrada de datos
            printf("Ingrese #%d: ", liContador + 1);
            scanf("%d", &laiNumeros[liContador]);
 
            hayError = (laiNumeros[liContador] > 10 || laiNumeros[liContador] < 1);
            if ( hayError )
                printf("\n\tError!, debe ingresar un valor entre 1 y 10.\n");
        } while ( hayError);
    }
 
    //Salida de datos
 
    return 0;
}
 

Un saludo

Hola, gracias por la ayuda amigo.
Otra cosa, sabes o alguien sabe como puedo hacer para saber qué número ha aparecido más veces?, no se me ocurre nada!.

[aguml]

si son 10 valores creare un array int entradas[10]=(0); //en vez de parentesis son corchetes. Luego, cada vez que introduzcas un valor y salgas del do while que has solucionado ya, usas un for que recorrerá los 10 miembros comparando dicho valor, y si lo encuentras o el valor es 0 sales del for. Con eso impides duplicados ya que con comprobar en el for si se encuentra y si es asi modificas una variable, al salir del for, dependiendo del valor de la variable ya sabrás que pasó.

[ecfisa]

Hola.

Código C:

...
#define ELEMENTS 20
 
int main()
{
  int num[ELEMENTS], can[ELEMENTS] = {0};
  int i, maximo=0, inOneTen;
 
  for(i=0; i < ELEMENTS; i++) {
    do {
        printf("Ingrese #%d: ", i+1 );
        scanf("%d", &num[i]);
        inOneTen = num[i] >= 1 && num[i] <= 10;
        if (!inOneTen) printf("\n\tError!, debe ingresar un valor entre 1 y 10.\n\n");
    } while (!inOneTen);
    can[num[i]]++;
  }
 
  for(i=0; i<ELEMENTS; i++) if (can[i] > maximo) maximo = i;
 
  printf("\nEl numero mas ingresado es %d con %d apariciones\n", maximo, can[maximo]);
  ...
}
 

Saludos.

[RGT]

Iniciado por ecfisa

Hola.

Código C:

...
#define ELEMENTS 20
 
int main()
{
  int num[ELEMENTS], can[ELEMENTS] = {0};
  int i, maximo=0, inOneTen;
 
  for(i=0; i < ELEMENTS; i++) {
    do {
        printf("Ingrese #%d: ", i+1 );
        scanf("%d", &num[i]);
        inOneTen = num[i] >= 1 && num[i] <= 10;
        if (!inOneTen) printf("\n\tError!, debe ingresar un valor entre 1 y 10.\n\n");
    } while (!inOneTen);
    can[num[i]]++;
  }
 
  for(i=0; i<ELEMENTS; i++) if (can[i] > maximo) maximo = i;
 
  printf("\nEl numero mas ingresado es %d con %d apariciones\n", maximo, can[maximo]);
  ...
}
 

Saludos.

Hola hermano, gracias por ayudarme.
pero, podrías explicarme como lo hiciste?, trato de entender y aprender a hacerlo por mí mismo.

[eferion]

Código C:

can[num[i]]++;
 

La variable "can" tiene 10 elementos, uno por cada cifra. Inicialmente se inicializan todas sus posiciones a 0. Cada vez que introduces un número incrementas el índice de "can" correspondiente ( el 0 si se introduce un 0, el 1 si se introduce un 1, ...). Tras estas iteraciones, en can[0] tendrás un valor equivalente al número de 0s, en can[1] el número total de 1s y así. Finalmente, recorres "can" y te quedas con el índice que tiene el valor más alto y listo.

Por cierto, así a bote pronto:

Código C:

for(i=0; i<ELEMENTS; i++) if (can[i] > maximo) maximo = i;
 

si máximo almacena el índice... el chequeo "can[i]>maximo" no es correcto porque estás comparando un índice con un valor... tendría que ser "can[i] > can[maximo]". No?