Foro de elhacker.net

Hola amigos que tal,

estoy intentando de intercambiar la sentencia switch por un if-else anidado y mi problema viene acarreado a la hora de que cada vez se ejecuta una sentencia se ejecuta igualmente el último else mostrando:

else{
                printf( "**Introdujo una letra incorrecta**" );
                printf( "\nIntroduzca una nueva calificacion: " );
        }
 

Donde supuestamente es el default de la antigua estructura switch por la que estoy intercambiando por este if-else anidado. Les adjunto el código.

/* Cuenta las calificaciones con letras pero se usa if-else anidados en lugar de condicional switch */
#include <stdio.h>
 
int main()
{
   int calificacion,cuentaA = 0,cuentaB = 0,cuentaC = 0,cuentaD = 0,cuentaF = 0;
 
   printf(  "**Introduzca el caracter EOF para finalizar la entrada de datos**\n"  );
   printf(  "\nIntroduzca la letra de la calificacion: "  );
 
   while ( ( calificacion = getchar() ) != EOF ) {
 
        if( calificacion =='a' || calificacion == 'A' )
                ++cuentaA;
        else if( calificacion =='b' || calificacion == 'B' )
                ++cuentaB;
        else if( calificacion =='c' || calificacion == 'C' )
                ++cuentaC;
        else if( calificacion =='d' || calificacion == 'D' )
                ++cuentaD;
        else if( calificacion =='f' || calificacion == 'F' )
                ++cuentaF;
        else{
                printf( "**Introdujo una letra incorrecta**" );
                printf( "\nIntroduzca una nueva calificacion: " );
        }
   }
   printf( "\nLos totales por calificacion son:\n" );
   printf( "A: %d\n", cuentaA ); printf( "B: %d\n", cuentaB );printf( "C: %d\n", cuentaC );printf( "D: %d\n", cuentaD ); printf( "F: %d\n", cuentaF );
   return 0;
}
 

Y claro me gustaría como tratar este supuesto caso default pero adaptado a if-else.. sin que me arroje en cada petición de introducción de carácter lo que les indico..

Muchas gracias

int calificacion;

Lo primero que veo es que declaras calificación como un entero, y luego lo comparas como un carácter  :silbar:

---

Si no quieres cambiar esa declaración, puedes dejarlo así

  while ( ( calificacion = getchar() ) != EOF ) {
 
       getchar(); #Este getchar esta para que se coma el intro
 
       if( (char)calificacion =='a' || (char)calificacion == 'A' )
               ++cuentaA;
       else if( (char)calificacion =='b' || (char)calificacion == 'B' )
               ++cuentaB;
       else if( (char)calificacion =='c' || (char)calificacion == 'C' )
               ++cuentaC;
       else if(  (char)calificacion =='d' ||  (char)calificacion == 'D' )
               ++cuentaD;
       else if(  (char)calificacion =='f' ||  (char)calificacion == 'F' )
               ++cuentaF;
       else{
               printf( "**Introdujo una letra incorrecta**" );
               printf( "\nIntroduzca una nueva calificacion: " );
       }
  }

Si te fijas he añadido un getchar() para que se coma el intro, tal y como lo tienes puesto, cuando introduces un caracter, por ejemplo la 'a', en el buffer se queda algo como 'a\n', ya que \n es el salto de linea, por lo que incrementaria cuentaA y luego verias el error del caracter incorrecto, al poner el getchar, lo que haces es calificacion = (int)a; y el siguiente getchar, se traga el \n  ;-)

Saludos

Como ya te indico MeCraniDOS el problema que tienes es un error logico: al introducir la calificacion seguida de un avance de linea este lo procesas y por supuesto se considerara una entrada no valida. Soluciones hay varias: descartar el resto de la linea mediante un bucle, leer el caracter con scanf utilizando el especificador " %c", etc..

No hay problema con ello ya que el retorno de getchar, la macro EOF y los caracteres literales como 'a', 'b', etc. son todos de tipo int.

Un saludo

Hola gracias por los sabios consejos, he aprendido con sus explicaciones varias formas para solventar este problema.

No caía en ese salto de línea que me producía el que se tragara el último else...

Me gustaría conocer para futuros problemas como o que medios usan para corregir o detectar errores, he visto uso de debugger o imprimir por pantalla lo que deseo ver si se ejecuta adecuadamente... no se si voy bien encaminado....

Me despido agradeciendo de nuevo su ayuda.

Saludos