Cómo evitar que el 'scanf' lea los datos introducidos antes?

[oldesebas]

Hola a todos. Me explico: Llevo varios días intentando depurar un programa que me da infinidad de errores, la mayoría de los cuales he llegado a la conclusión que se producen por que el 'scanf' me lee datos introducidos anteriormente. Ejemplo:

Código C++:

do{
        accion == ' ';
 
        if (accion == 'e' || accion == 'E'){
 
            accion == ' ';
            entrar_vehiculo();
 
        } else if (accion == 's' || accion == 'S'){
 
            accion == ' ';
            salir_vehiculo();
        }
        
        printf("\nIndique acción a realizar (Si entra vehiculo pulse 'E', si sale vehiculo pulse 'S' si desea finalizar el programa pulse 'F'): ");
        scanf("%c", &accion);
 
    } while (accion!= 'f' && accion != 'F');
 

La primera vez el bucle funciona correctamente, pero las siguientes el 'printf' se me repite dos y hasta tres veces porque, creo, el 'scanf' lee datos "residuales" que he introducido para otras funciones del programa en lugar de esperar a que introduzca nuevos datos, de manera que el bucle da una vuelta sin realizar ninguna acción y vuelve a introducir el texto. Creo que es un error básico, pero no he encontrado solución clara. Agradeceré cualquier aportación.

[Fw190]

Ocurre que accion la lees al final del bucle do, mientras que al principio de este bucle haces accion = ' ', con lo cual, introduzcas lo que introduzcas, accion va a valer ' ' de cara a las comparaciones que estás haciendo. La solución es pedir accion al principio del bucle.

[sam90]

Que es lo que ocurre en tu programa? Si imprime varias veces en pantalla lo que esta en el printf??

[oldesebas]

Iniciado por Fw190

Ocurre que accion la lees al final del bucle do, mientras que al principio de este bucle haces accion = ' ', con lo cual, introduzcas lo que introduzcas, accion va a valer ' ' de cara a las comparaciones que estás haciendo. La solución es pedir accion al principio del bucle.

No, ese no es el problema, ya que el 'acción==0' no es una asignación, de hecho está ahí por error, pero no modifica el programa.

El 'scanf' está al final del do por que de tanto marearlo se ha quedado ahí, pero al principio lo tenía arriba y el resultado no cambia, pues el problema es que en 'accion' me carga datos que he introducido en los subprogramas en lugar de los que voy a introducir.

Me he acabado apañando con un rudimentario 'if' en el 'printf' que no me evita que el 'scanf' me lea los datos residuales, pero si evita que se imprima en pantalla el primer bucle, así me ha quedado:

Código C++:

int main (){
 
    char accion = ' ';
 
    do{
 
        if (accion == 'e' || accion == 'E'){
 
            entrar_vehiculo();
 
        } else if (accion == 's' || accion == 'S'){
 
            salir_vehiculo();
 
        } else if(accion == 'o' || accion == 'O'){
 
            printf("\n\n");
            imprimir_parking();
 
        }
 
        if(accion != 'e' && accion != 'E' && accion != 's' && accion != 'S' && accion != 'o' && accion != 'O' && accion != 'F' && accion != 'f' ){
            printf("\nIndique acción a realizar (Si entra vehiculo pulse 'E', si sale vehiculo pulse 'S', si desea observar el estado del parking pulse 'O', si desea finalizar el programa pulse 'F'): ");
        }
 
        scanf("%c", &accion);
 
    } while (accion!= 'f' && accion != 'F');
 
    return 0;
}
 

En otro foro me han dado una solución mejor que directamente limpia los datos de manera que el programa solo escanee una vez:

Código C++:

...
 
flush_stdin()
{
   int ch;
   while( (ch = getchar()) != '\n' && ch != EOF );
}
 
...
 
   scanf("%d", &a );
   flush_stdin();
...
   scanf("%d", &b );
   flush_stdin();
...
 

Gracias.

[oldesebas]

Iniciado por sam90

Que es lo que ocurre en tu programa? Si imprime varias veces en pantalla lo que esta en el printf??

Si, eso es lo que ocurría. La primera vez no, pero las siguientes me hace un ciclo del 'do while' antes de que se pare a esperar que yo introduzca datos, por lo que me imprimía la pregunta dos veces. Con el 'if' que he añadido me sigue haciendo un primer ciclo del bucle antes de parar, pero evito que se imprima la pregunta en ese primer ciclo.

No se si me explico bien, soy noob..

[sam90]

La solucion que te dieron en el otro foro es la que yo te hubiese dado... el tema esta que cuando introduces una tecla y presionas enter el enter tambien se envia como un caracter. Entonces primero lees la letra hace la accion luego lees el enter que no coincide con ninguno de eso caracteres. Y ahi recien te da la posibilidad de empezar de nuevo.

La funcion flush_stdin() lo que hace es consumir las entradas por teclado hasta que encuentre un salto de linea (enter).

saludos

[Javieer-G]

Iniciado por sam90

La solucion que te dieron en el otro foro es la que yo te hubiese dado... el tema esta que cuando introduces una tecla y presionas enter el enter tambien se envia como un caracter. Entonces primero lees la letra hace la accion luego lees el enter que no coincide con ninguno de eso caracteres. Y ahi recien te da la posibilidad de empezar de nuevo.

La funcion flush_stdin() lo que hace es consumir las entradas por teclado hasta que encuentre un salto de linea (enter).

saludos

Yo lo suelo agregando un getchar() después de cada scanf y listo. Con eso se come el salto de línea en cuestión y no da problemas.... aunque claro, la función esa supongo que evita que el usurio se ponga a meter datos como loco y luego desajusten todo el programa jaja

[oldesebas]

Iniciado por Javieer-G

aunque claro, la función esa supongo que evita que el usurio se ponga a meter datos como loco y luego desajusten todo el programa jaja

Sí, por lo visto si introduzco 20 caracteres en el 'scanf' anterior me formula la pregunta 20 veces, esto evitará que ocurra, ¿no es así?

Aunque no entiendo el significado de ch != EOF ..

[Javieer-G]

No me he mirado tu programa, pero el EOF es la marca de fin de fichero (End Of File). A nivel práctico, cuando pulsas Ctrl+Z en el terminal es como si metieras la marca EOF.

[L3m0n]

En otro foro te han dado una muy buena solucion.

En ese codigo se crea una funcion que sustituye a la que mucha gente usa en windows, el fflush_stdin() que es un gran error, ya que no es portable.

Básicamente lo que hace la funcion que has puesto es vaciar el buffer de entrada
(lee caracteres hasta encontrarse con un '\n' (un enter) o un EOF(end of file))

Espero haberme explicado bien :)

[delomatias]

tendria que ser accion=' '

[oldesebas]

Va como un tiro con el flush_stdin() :)

Aquí os dejo el programa completo y perfectamente depurado para quien le pueda interesar:

Código C++:

/***********************************************************************************************************************************************
 
PARKING:
 
Realizar un programa para controlar un aparcamiento que dispone de 25 plazas de 2 tamaños: 10 grandes y 15 pequeñas. Las plazas se asignaran
automáticamente cuando entre un vehiculo segun el tamaño:
 
· Cada vehiculo solo ocupara una plaza.
· Un vehiculo pequeño siempre ocupara una plaza pequeña salvo que estén todas ocupadas y haya plazas grandes libres.
· Un vehiculo grande solo podrá aparcar en una plaza grande.
· Cuando entre un vehiculo ocupara siempre la plaza de número menor de todas las libres.
 
El programa tendrá tres opciones básicas:
 
· Entrada: es necesario indicar el tamaño del coche.
· Salida: es necesario indicar la plaza que deja libre.
· Situación del aparcamiento: indicando las plazas libres y las ocupadas.
 
***********************************************************************************************************************************************/
 
#include <stdio.h>
#include <ctype.h>
 
/* Vectores que indicarán la situación de cada aparcamiento */
 
typedef bool Tipo_Vector_ParkingP[15];
typedef bool Tipo_Vector_ParkingG[10];
Tipo_Vector_ParkingP Vector_ParkingP = {false, false, false, false, false, false, false, false, false, false, false, false, false, false, false};
Tipo_Vector_ParkingG Vector_ParkingG = {false, false, false, false, false, false, false, false, false, false};
 
/* Función para limpiar el BUFFER */
 
void flush_stdin(){
 
   int ch;
   while( (ch = getchar()) != '\n' && ch != EOF );
 
}
 
/* Subprograma que se encarga de imprimir las plazas pequeñas del parking una a una, vacias o llenas, según la situación del vector 'Vector_ParkingP' */
 
void imprimir_plazaP(){
 
    for(int k = 1; k <= 15; k++){
 
        if (Vector_ParkingP[k]){
            printf("xx|");
        } else {
            printf("  |");
        }
    }
}
 
/* Subprograma que se encarga de imprimir las plazas grandes del parking una a una, vacias o llenas, según la situación del vector 'Vector_ParkingG' */
 
void imprimir_plazaG(){
 
    for(int k = 1; k <= 10; k++){
 
        if (Vector_ParkingG[k]){
            printf("xxx|");
        } else {
            printf("   |");
        }
    }
}
 
/* Subprograma que imprime la gráfica del parking indicando su situación */
 
void imprimir_parking(){
 
    printf("   _01_02_03_04_05_06_07_08_09_10_11_12_13_14_15_\n");
    printf("P >|");
    imprimir_plazaP();
    printf("\n   |");
    imprimir_plazaP();
    printf("\n   |                                            |\n");
    printf("   |- - - - - - - - - - - - - - - - - - - -     |\n");
    printf("   |                                            |\n");
    printf("   |");
    imprimir_plazaG();
    printf("    |\nG >|");
    imprimir_plazaG();
    printf("in/ |\n   |___|___|___|___|___|___|___|___|___|___| out|\n");
    printf("    01  02  03  04  05  06  07  08  09  10 |    |\n");
 
}
 
/* Subprograma que gestiona las entradas de vehiculos. */
 
void entrar_vehiculo(){
 
    char tecla = ' ';
 
    while (tecla != 'P' && tecla != 'G'){
 
        printf("\nIndique si el vehiculo es grande 'G' o pequeño 'P': ");
 
        scanf("%c", &tecla);
        flush_stdin();
        tecla = toupper (tecla);
 
        if(tecla != 'P' && tecla != 'G'){
            printf("\nCOMANDO INCORRECTO\n");
        }
    }
 
    if (tecla == 'P'){
 
        for(int k = 1; k <= 15; k++){
 
            tecla = 'G';
 
            if (!Vector_ParkingP[k]){
                Vector_ParkingP[k] = true;
                tecla = ' ';
                k = 16;
            }
        }
    }
 
    if (tecla == 'G'){
 
        for(int k = 1; k <= 10; k++){
 
            if (!Vector_ParkingG[k]){
                Vector_ParkingG[k] = true;
                k = 11;
                tecla = ' ';
            }
        }
    }
 
    if (tecla != ' '){
 
        printf("\n\nParking lleno, no pueden entrar más vehiculos:\n\n");
 
    } else {
 
        printf("\n\nEl vehiculo ha ocupado su plaza:\n\n");
        tecla = ' ';
    }
 
    imprimir_parking();
}
 
/* Subprograma que gestiona las salidas de vehiculos. */
 
void salir_vehiculo(){
 
    int plaza = 0;
    char teclo = ' ';
 
    while (teclo != 'P' && teclo != 'G'){
 
        printf("\nIndique la zona (G/P)): ");
 
        scanf("%c", &teclo);
        flush_stdin();
        teclo = toupper (teclo);
 
        if(teclo != 'P' && teclo != 'G'){
            printf("\nCOMANDO INCORRECTO\n");
        }
 
        if (teclo == 'P'){
 
            printf("\nIndique la plaza (1-15)): ");
 
            scanf("%d", &plaza);
            flush_stdin();
 
            if (Vector_ParkingP[plaza]){
                Vector_ParkingP[plaza] = false;
                printf("\n\nEl vehiculo ha abandonado la plaza:\n\n");
 
            } else {
                printf("\n\nERROR: No hay vehiculo en esa plaza:\n\n");
            }
        }
 
        if (teclo == 'G'){
 
            printf("\nIndique la plaza (1-10)): ");
 
            scanf("%d", &plaza);
            flush_stdin();
 
            if (Vector_ParkingG[plaza]){
                Vector_ParkingG[plaza] = false;
                printf("\n\nEl vehiculo ha abandonado la plaza:\n\n");
 
 
            } else {
                printf("\n\nERROR: No hay vehiculo en esa plaza:\n\n");
            }
        }
    }
    teclo = ' ';
    imprimir_parking();
}
 
/* Programa principal. */
 
int main (){
 
    char accion = ' ';
 
    do{
 
        printf("\nIndique acción a realizar (Si entra vehiculo pulse 'E', si sale vehiculo pulse 'S', si desea observar el estado del parking pulse 'O', si desea finalizar el programa pulse 'F'): ");
        scanf("%c", &accion);
        flush_stdin();
        accion = toupper (accion);
 
        switch (accion) {
 
            case 'E':
            entrar_vehiculo();
            break;
 
            case 'S':
            salir_vehiculo();
            break;
 
            case 'O':
            printf("\n\n");
            imprimir_parking();
            break;
 
            default:
            printf("\nCOMANDO INCORRECTO\n");
        }
 
    } while (accion != 'F');
 
    return 0;
}
 

Gracias a todos por vuestra ayuda ;)