SoloCodigo

Lo primero de todo es que este programa no hace nada. Sólo te da para que escribas por pantalla lo que quieras y si da la casualidad de que es una de esas letras pues ésta se incrementa en uno. Luego no muestras el valor ni haces nada con ellas asi que el programa no tiene mucho sentido.

Si que veo un fallo claro en las condiciones. Dentro del "if" no se pone "break;", esto se pone solo en los switches. De todas formas, el programa que quieres hacer, que no hace nada se quedaría bien así.

Código: C

1. #include <stdio.h>
2. #include <conio.h>
3.  
4. int main()
5. {
6.     int calif, aa = 0,bb = 0,cc = 0,dd = 0,ee = 0,ff = 0;
7.  
8.     while ( ( calif = getchar()) != EOF )
9.     {
10.         scanf("%d", &calif);
11.         if (calif == 'a')
12.         {
13.             aa++;
14.         }
15.             if (calif == 'b')
16.             {
17.                 bb++;
18.             }
19.  
20.                 if (calif == 'c')
21.                 {
22.                     cc++;
23.                 }
24.                     if (calif == 'd')
25.                     {
26.                         dd++;
27.                     }
28.                         if (calif == 'e')
29.                         {
30.                             ee++;
31.                         }
32.                             if (calif == 'f')
33.                             {
34.                                 ff++;
35.                             }
36.                                 if (calif == ' ' || calif == 'n')
37.                                 {
38.                                 }
39.     }
40.     getch();
41.     return 0;
42. }
43.  

Pero te repito que este programa no hace nada.