prüfen, ob Eingabe eine Zahl ist und wenn nicht zurück zum Eingang

Kritik

#include <stdio.h>
#include <conio.h>

So, Sie arbeiten unter Windows.

#include <ctype.h>
void main(void)

Den Rückgabetyp main() sollte int, obwohl Sie finden viele Beispiele für das Gegenteil.

{
    int val_a;
    int val_b;
    int result;

    printf("Write a number A=");
    scanf("%d", &val_a);

Sollten Sie testen den Rückgabewert von scanf() um zu sehen, ob es in der Lage war, zu Lesen, eine Zahl oder nicht.

    if (isdigit(val_a))

Diese test 'funktioniert', aber es wird überprüft, ob die eingegebene Zahl ein Wert im Bereich 48..57 (ASCII, CP1252, Unicode, ISO 8859-1, etc-codes für die Ziffern '0' zu '9'). Dies war wahrscheinlich nicht das, was Sie im Sinn hatte.

    {
        printf("\nWrite a number B=");
        scanf("%d", &val_b);

Eingabe ein Zeilenumbruch, so dass der Zeilenumbruch vor ot die printf() format-string ist nicht notwendig, obwohl es auch nicht schädlich. Den gleichen Kommentar über die überprüfung scanf() gilt auch hier.

    }
    else
        printf("\nI said number.\n");

Es ist gut zu tun, Fehler überprüfen. Jedoch, Sie weiter ab auch wenn man den Fehler erkennt. Sie müssen wirklich aufhören.

    result = val_a + val_b;
    printf("%d + %d = %d", val_a, val_b, result);

Auf die ganze, am besten ist es zu Ende Zeilen mit einem Zeilenumbruch. Es spült die Daten in die Datei oder Bildschirm.

    getch();
    printf("\033[2J");   \\ to clear the screen

Den printf() format-string ist nicht tragbar. Gegeben, dass Sie enthalten <conio.h> ist, sollten Sie wahrscheinlich verwenden clrscr() (was wohl tut, senden Sie die escape-Sequenz oder ein äquivalent, um das terminal). Ich bin nicht davon überzeugt, dass es notwendig ist, um den Bildschirm zu löschen am Ende des Programms, aber ich arbeite hauptsächlich auf Unix-Systemen, nicht mit Windows.

}

Gegeben, dass MSVC noch ein C89 compiler sollten Sie auch return 0; am Ende das Programm zurück zum Erfolg-status.

Umgeschrieben

Hinzufügen alle änderungen, die Sie würde am Ende mit:

#include <stdio.h>
#include <conio.h>

int main(void)
{
    int val_a;
    int val_b;
    int result;

    printf("Write a number A=");
    if (scanf("%d", &val_a) != 1)
    {
        printf("I said number.\n");
        getch();
        clrscr();
        return(1);
    }
    printf("\nWrite a number B=");
    if (scanf("%d", &val_b) != 1)
    {
        printf("I said number.\n");
        getch();
        clrscr();
        return(1);
    }

    result = val_a + val_b;
    printf("%d + %d = %d\n", val_a, val_b, result);
    getch();
    clrscr();
    return(0);
}

Wenn Sie gelernt haben, über Funktionen schreiben, könnte man komprimieren:

#include <stdio.h>
#include <conio.h>

static int get_a_number(const char *prompt)
{
    int value;
    printf(prompt);
    if (scanf("%d", &value) != 1)
    {
        printf("I said number.\n");
        getch();
        clrscr();
        exit(1);
    }
    return value;
}

int main(void)
{
    int val_a  = get_a_number("Write a number A=");
    int val_b  = get_a_number("Write a number B=");
    int result = val_a + val_b;
    printf("%d + %d = %d\n", val_a, val_b, result);
    getch();
    clrscr();
    return(0);
}

Den get_a_number() Funktion ist nicht eine Allgemeine Funktion, aber es ist nützlich, in diesem Zusammenhang, weil es kapselt ein gemeinsames Stück code.

• Aufruf des Unterprogramms erlaubt uns, um einen überblick über die Unregelmäßigkeiten in der argument-Liste.
• Das Unterprogramm selbst fasst die Gesetzmäßigkeiten der code.

Das ist ein Zitat von Kernighan & Plauger, "The Elements of Programming Style".

---

Mit atexit()

Den code oben wurde aktualisiert, um getch() und clrscr() auf alle exit-Pfade (und zu entfernen #include <ctype.h> da keine der Funktionen von es waren mehr). Es ist ein ärgernis zu schreiben, die die beiden Funktionsaufrufe wiederholt. Ein anderer Weg, dieses problem zu vermeiden, ist eine Funktion schreiben, die aufgerufen wird, wenn das Programm beendet; Sie registrieren es mit der atexit() Funktion, erklärt in <stdlib.h>:

#include <stdio.h>
#include <conio.h>
#include <stdlib.h>

static void wait_at_exit(void)
{
    getch();
    clrscr();
}

static int get_a_number(const char *prompt)
{
    int value;
    printf(prompt);
    if (scanf("%d", &value) != 1)
    {
        printf("I said number.\n");
        exit(1);
    }
    return value;
}

int main(void)
{
    atexit(wait_at_exit);
    int val_a  = get_a_number("Write a number A=");
    int val_b  = get_a_number("Write a number B=");
    int result = val_a + val_b;
    printf("%d + %d = %d\n", val_a, val_b, result);
    return(0);
}

Code in main() davon ausgegangen, dass Sie können mix-Anweisungen (wie atexit()) und Erklärungen (wie int val_a) beliebig. Das ist zulässig, sowohl den aktuellen (2011) und alten (1999) - Versionen der Standard-C, aber war nicht erlaubt, die ursprüngliche (1989) version der Standard-C. ich glaube, MSVC hält sich an die ursprünglichen standard. Wenn ja, werden Sie wahrscheinlich benötigen, um zu schreiben, dass main():

int main(void)
{
    atexit(wait_at_exit);
    {
    int val_a  = get_a_number("Write a number A=");
    int val_b  = get_a_number("Write a number B=");
    int result = val_a + val_b;
    printf("%d + %d = %d\n", val_a, val_b, result);
    }
    return(0);
}

Ist es eine feine Diskussion Stelle, ob der code im inneren block (die inneren Klammern) eingerückt werden soll eine extra-Ebene oder nicht. Sie sind immer erlaubt, um Variablen zu deklarieren, die zu Beginn eines code-block in geschweiften Klammern, so dass ermöglicht mir schreiben Sie den Aufruf atexit() und dann definieren Sie und verwenden Sie die Variablen.

Wenn Sie Lesen, über Agile Programmierung, werden Sie feststellen, dass eines der mantras ist:

• DRY — Don ' T Repeat Yourself

Den schreibt oben zu vermeiden, die wiederholt getscr() fordert, könnte als eine Anwendung des mantra. Ähnlich ist es mit der get_a_number() Funktion; es gilt die TROCKENE mantra.

---

Versuchen wieder

Wenn Sie möchten, dass das Programm zurück zu gehen und die Eingabeaufforderung wieder, wenn der Benutzer einen Fehler macht in Ihrer Eingabe, dann müssen Sie zum ändern der get_a_number() Funktion. Und, da die Funktion wächst erheblich komplexer, Sie definitiv brauchen, um mit einer Funktion für den job. Sie auch ein problem mit scanf(); es erlaubt nicht, Sie zu erzwingen, eine Zahl pro Zeile. Wenn Sie eingegeben haben 1 2 in Reaktion auf die erste Aufforderung, das erste Gespräch zu get_a_number() Lesen würde der einen und die leere, stoppen Parsen (und die blanko-Rückseite für die Wiederverwendung) und kehrt mit dem Wert 1. Der zweite Aufruf get_a_number() würde die Ausgabe der prompt aber wieder mit dem Wert 2, ohne das der Anwender irgendetwas eingeben mehr. Wenn nicht, Verhalten Sie wollen, dann haben Sie zu ordnen, Lesen Sie die ganze erste Zeile, und suchen Sie nach der Nummer, und dann verwerfen Sie den rest. Es ist ein weiterer Grund für sorgen zu. Wenn der Benutzer Arten a statt 1, die scanf() wird nie gelesen, nichts mehr; Sie gehen in eine Schleife, Feststellung, dass die a gilt nicht als eine Ziffer, 0 zurückgeben Umbauten abgeschlossen, und Ihr code wird wahrscheinlich am Ende in einer Endlosschleife, mit der Aufforderung für die Eingabe und das Lesen der a. Diese Art von problem ist, warum viele Menschen (insbesondere mich) vermeiden Sie die Verwendung scanf().

Der einfachste Weg, um zu vermeiden die meisten der genannten Probleme ist mit fgets() und sscanf():

enum { MAX_LINELENGTH = 4096 };
enum { MAX_ATTEMPTS   = 10   };

static int get_a_number(const char *prompt)
{
    int value;
    char line[MAX_LINELENGTH];
    int count = 0;
    while (fputs(prompt, stdout) != EOF &&
           fgets(line, sizeof(line), stdin) != 0)
    {
        if (sscanf(line, "%d", &value) == 1)
            return value;
        if (count++ > MAX_ATTEMPTS)
        {
            printf("I give in; I don't understand what you're typing\n");
            exit(1);
        }
        printf("I said please enter a number.\n");
    }

    printf("Oops: I got EOF or an error; goodbye!\n");
    exit(1);
}

Dass deckt ziemlich viele Probleme. Oft ist es eine gute Idee, um echo zurück der fehlerhaften Daten. Könnten Sie das mit:

        printf("I didn't understand what you meant when you typed:\n%s\n"
               "I said please enter a number.\n", line);

Diese verwendet "benachbarten string-Verkettung', und zeigt, was der Benutzer das Programm empfangen, die können oft helfen, den Benutzer zu verstehen, was falsch ist.

Ich verwendet fputs(prompt, stdout) statt printf(prompt) für die Ausgabe der Eingabeaufforderung. Entweder verwendet werden könnten (printf(prompt) != 0 würde der test sein), aber es gibt einen leichten Vorteil zu fputs() denn wenn jemand es geschafft, Sie zu untergraben, Ihr Programm und erhalten Sie einen % in die Eingabeaufforderung, die printf() möchte versuchen, den Prozess ein argument es nicht bestanden wurde, in der Erwägung, dass fputs() interpretiert nicht das argument Zeichenfolge an alle. Hier, der Unterschied ist zu vernachlässigen; es kann vernünftigerweise argumentiert werden, dass ich über-verkomplizieren. Aber die Sicherheit der Programme ist auch etwas, das Sie am Ende lernen, und Techniken wie "nicht verwenden printf(variable)" kann dazu beitragen, dass Ihre Programme mehr belastbar. (Tatsächlich ist die Komplikation ist die Erklärung; der code ist ganz einfach.)

Ein weiterer Ausführlicher Diskussion Stelle ist " sollte es einen fflush() nach fputs() und vor fgets()?' Wäre es möglich, das extra - 'Zustand' fflush(stdout) == 0 && zum loop control, aber in der Praxis ist es meist unnötig. Es würde nur einen Unterschied machen, wenn standard-Eingabe kam von etwas anderem, als das terminal und standard-Ausgabe geht zu Datei. Also habe ich es weggelassen.

Den Zähler in der Schleife verhindert, dass die Dinge schlecht gehen schief — das Programm beendet, anstatt hängen, um für immer, vereitelt die Benutzer können nicht die Daten richtig eingegeben. Haben Sie jemals ein Programm mit einem Dialogfenster in dem Sie können nur getroffen der 'OK' - Taste (obwohl die nicht OK) und dann kommen Sie zurück mit der gleichen box wieder, immer und immer und immer wieder? Ich hoffe nicht, um deiner selbst Willen. Ich habe; es frustriert mich, wenn es passiert.

Und Sie können sehen, warum es vorteilhaft ist, den code zu schreiben in get_a_number() nur einmal, statt zweimal, das wäre viel Wiederholung.

InformationsquelleAutor Jonathan Leffler

Hast du die isdigit Funktion alle falsch. Es ist dazu gedacht, um zu erhalten eine char-argument, aber in Ihrem Fall die int umgewandelt wird, herabgestuft werden Sie vielleicht sagen.

Die Funktion basiert auf der ASCII-codes der Buchstaben, hier der link. Sie können versuchen, diese für sich selbst zu sehen.

char c = '1';
if(isdigit(c))
   printf("digit");

Außerdem, was Sie brauchen, ist ein do{ /* something */} while(boolean);

InformationsquelleAutor alegen