Warum verursacht scanf() in diesem Code eine Endlosschleife?

Question 1

Ich habe ein kleines C-Programm, das nur Zahlen aus stdin liest, eine bei jedem Schleifenzyklus. Wenn der Benutzer etwas NaN eingibt, sollte ein Fehler auf der Konsole ausgegeben werden und die Eingabeaufforderung sollte erneut zurückkehren. Bei Eingabe von “0” sollte die Schleife enden und die Anzahl der gegebenen positiven/negativen Werte auf der Konsole ausgegeben werden. Hier ist das Programm:

#include <stdio.h>

int main()
{
    int number, p = 0, n = 0;

    while (1) {
        printf("-> ");
        if (scanf("%d", &number) == 0) {
            printf("Err...\n");
            continue;
        }
        
        if (number > 0) p++;
        else if (number < 0) n++;
        else break; /* 0 given */
    }

    printf("Read %d positive and %d negative numbers\n", p, n);
    return 0;
}

Mein Problem ist, dass die Eingabe einer Nichtzahl (wie “a”) zu einer Endlosschleife führt, in der immer wieder “-> Err …” geschrieben wird. Ich denke, es ist ein Problem mit scanf() und ich weiß, dass diese Funktion durch eine sicherere ersetzt werden könnte, aber dieses Beispiel ist für Anfänger, die nur über printf/scanf, if-else und Schleifen Bescheid wissen.

Die Antworten auf die Frage habe ich bereits gelesenscanf() überspringt jeden anderen while Schleife in C und habe andere Fragen überflogen, aber nichts hat dieses spezifische Problem wirklich beantwortet.

Question 2

scanf verbraucht nur die Eingabe, die mit der Formatzeichenfolge übereinstimmt, und gibt die Anzahl der verbrauchten Zeichen zurück. Jedes Zeichen, das nicht mit der Formatzeichenfolge übereinstimmt, führt dazu, dass der Scanvorgang beendet wird, und das ungültige Zeichen bleibt im Puffer. Wie andere bereits sagten, müssen Sie das ungültige Zeichen noch aus dem Puffer löschen, bevor Sie fortfahren. Dies ist eine ziemlich schmutzige Lösung, aber sie entfernt die anstößigen Zeichen aus der Ausgabe.

char c="0";
if (scanf("%d", &number) == 0) {
  printf("Err. . .\n");
  do {
    c = getchar();
  }
  while (!isdigit(c));
  ungetc(c, stdin);
  //consume non-numeric chars from buffer
}

bearbeiten: Der Code wurde korrigiert, um alle nicht numerischen Zeichen auf einmal zu entfernen. Druckt nicht mehr mehrere “Fehler” für jedes nicht-numerische Zeichen mehr aus.

Hier ist ein ziemlich guter Überblick über scanf.

Question 3

Ich denke, Sie müssen nur den Puffer leeren, bevor Sie mit der Schleife fortfahren. So etwas würde wahrscheinlich den Job machen, obwohl ich von hier aus nicht testen kann, was ich schreibe:

int c;
while((c = getchar()) != '\n' && c != EOF);

Question 4

scanf() Verlässt die “a” noch im Eingabepuffer für das nächste Mal. Sie sollten wahrscheinlich verwenden getline() um eine Zeile zu lesen, egal was passiert, und sie dann mit zu analysieren strtol() oder ähnliches stattdessen.

(Ja, getline() ist GNU-spezifisch, nicht POSIX. Na und? Die Frage ist mit “gcc” und “linux” gekennzeichnet. getline() ist auch die einzig sinnvolle Möglichkeit, eine Textzeile zu lesen, es sei denn, Sie möchten alles von Hand machen.)

Question 5

Aufgrund der Probleme mit scanf Wie die anderen Antworten zeigen, sollten Sie wirklich einen anderen Ansatz in Betracht ziehen. Ich habe immer gefunden scanf viel zu begrenzt für ernsthaftes Lesen und Verarbeiten von Eingaben. Es ist eine bessere Idee, einfach ganze Zeilen mit einzulesen fgets und dann mit Funktionen wie bearbeiten strtok und strtol (was BTW Ganzzahlen korrekt analysiert und Ihnen genau sagt, wo die ungültigen Zeichen beginnen).

Question 6

Anstatt zu verwenden scanf() und mit dem Puffer umgehen müssen, der ungültiges Zeichen hat, verwenden fgets() und sscanf().

/* ... */
    printf("0 to quit -> ");
    fflush(stdout);
    while (fgets(buf, sizeof buf, stdin)) {
      if (sscanf(buf, "%d", &number) != 1) {
        fprintf(stderr, "Err...\n");
      } else {
        work(number);
      }
      printf("0 to quit -> ");
      fflush(stdout);
    }
/* ... */

Question 7

Ich hatte ein ähnliches Problem. Ich habe es gelöst, indem ich nur scanf verwendet habe.

Input "abc123<Enter>" um zu sehen, wie es funktioniert.

#include <stdio.h>
int n, num_ok;
char c;
main() {
    while (1) {
        printf("Input Number: ");
        num_ok = scanf("%d", &n);
        if (num_ok != 1) {
            scanf("%c", &c);
            printf("That wasn't a number: %c\n", c);
        } else {
            printf("The number is: %d\n", n);
        }
    }
}

Question 8

Auf einigen Plattformen (insbesondere Windows und Linux) können Sie verwenden fflush(stdin);:

#include <stdio.h>

int main(void)
{
  int number, p = 0, n = 0;

  while (1) {
    printf("-> ");
    if (scanf("%d", &number) == 0) {
        fflush(stdin);
        printf("Err...\n");
        continue;
    }
    fflush(stdin);
    if (number > 0) p++;
    else if (number < 0) n++;
    else break; /* 0 given */
  }

  printf("Read %d positive and %d negative numbers\n", p, n);
  return 0;
}