C - fgets wird einmal übersprungen

  • C - fgets wird einmal übersprungen

    Hi,

    ich habe folgendes kleine C Programm:

    C-Quellcode

    1. #include <stdio.h>
    2. #include <strings.h>
    4. //Fakultätsfunktionsdeklaration
    5. int factorial(int i) {
    7. //Fakultät initialisieren
    8. int factorial = 1;
    10. //Nur berechnen wenn index gr√sser als 1, da !1 = 1 und !0 = 1
    11. while (i > 1) {
    13. //Fakultät erhöhren und index um eins verringern
    14. factorial *= i--;
    15. }
    17. return factorial;
    18. }
    20. int main(void) {
    23. char choice;
    24. char input[2] = {0};
    26. printf("Dieses Programm berechnet die Fakultäten von 1 bis 10 und gibt sie aus.\n");
    28. do {
    30. printf("Wollen Sie fortfahren? (y/n)\n");
    31. //Benutzereingabe einlesen
    32. fgets(input, sizeof(input), stdin);
    33. sscanf(input, "%c", &choice);
    35. //Der Nutzer möchste fortfahren
    36. if (choice == 'y') {
    38. //Array of factorials
    39. int factorials[10];
    41. //Anzahl der Einträge dynamisch berechnen
    42. int numberOfElements = sizeof(factorials) / sizeof(factorials[0]);
    43. //Array durchlaufen
    44. for (int i = 0; i < numberOfElements; i++) {
    46. //Fakultät berechnen und entsprechendem Feld zuweisen
    47. factorials[i] = factorial((i + 1));
    48. }
    50. //Einträge durchlaufen
    51. for (int i = 0; i < numberOfElements; i++) {
    53. //Index is the Zahl und das Element des Array the Fakultät zu der Zahl
    54. printf("Die Fakultät der Zahl %d ist %d\n", (i + 1), factorials[i]);
    55. }
    57. } else if (choice != 'y' && choice != 'n') {
    59. printf("Bitte tätigen Sie eine gültige Eingabe.\n");
    60. choice = '\0';
    61. memset(input, 0, sizeof(input));
    62. }
    64. } while (choice != 'y' && choice != 'n');
    66. printf("Das Programm wird beendet. Auf Wiedersehen!\n");
    68. return 0;
    69. }
    Alles anzeigen


    Wenn ich das Programm ausführe und etwas eingebe, das ungleich "y" oder "n" ist, dann macht das Programm das:

    Quellcode

    1. Dieses Programm berechnet die Fakultäten von 1 bis 10 und gibt sie aus.
    2. Wollen Sie fortfahren? (yes/no)
    3. h
    4. Bitte tätigen Sie eine gültige Eingabe.
    5. Wollen Sie fortfahren? (yes/no)
    6. Bitte tätigen Sie eine gültige Eingabe.
    7. Wollen Sie fortfahren? (yes/no)


    Warum? Ich bin schon mit dem Debugger durchgesteppt, konnte den Fehler aber nicht ausfindig machen. Ich verstehe nicht, warum fgets einfach einmal übergangen wird.
    Weiß jemand Rat?

  • Das Konstrukt mit fgets() und sscanf() sieht komisch aus und funktioniert wohl nicht wie gewollt! Des weiteren ist das else if so nicht gut...

    Jedenfalls wenn Du das Input-Array größer anlegst, dann hat auch fgets() genug Raum um eine Zeile einzulesen. Um das erste Zeichen zu bekommen, brauchst Du nicht unbedingt sscanf().

    Mit dem Debugger hätte dir auffallen können, dass choice die Werte durchgeht, die Du eingegeben hast. Also wenn Du ein Zeichen + Enter eintippst, dann nimmt choice ein Zeichen + Enter an. Wenn Du mehr Zeichen + Enter eintippst, dann halt mehr Zeichen und zum Schluss Enter.

  • Hm,

    durch den sizeof() wartet fgets nicht auf ein Return sondern auf die Eingabe von 2 Zeichen (sizeof(Input)==3-1). Du kannst also z.B. 'y' + 'Return' eingeben oder aber auch 'fg' und es wird der fgets verlassen.

    Gruß

    Claus
    2 Stunden Try & Error erspart 10 Minuten Handbuchlesen.

    Pre-Kaffee-Posts sind mit Vorsicht zu geniessen :)