vérifiez si l'entrée est un nombre et si pas de retour à l'entrée

Critique

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

Si vous travaillez sur Windows.

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

Le type de retour de main() devrait être int, même si vous trouverez beaucoup d'exemples du contraire.

{
    int val_a;
    int val_b;
    int result;

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

Vous devriez être en testant la valeur de retour de scanf() pour voir si elle est capable de lire un nombre ou pas.

    if (isdigit(val_a))

Ce test 'marche', mais il vérifie si le nombre entré est une valeur dans la plage de 48..57 (qui sont en ASCII, CP1252, Unicode, ISO 8859-1, etc codes pour les chiffres '0' à '9'). Ce n'était probablement pas ce que vous aviez à l'esprit.

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

L'utilisateur a entré un retour à la ligne, de sorte que le saut de ligne en face de l'ot de la printf() chaîne de format n'est pas nécessaire, même si elle n'est pas nuisible. Le même commentaire à propos de la vérification scanf() s'applique ici aussi.

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

Il est bon de faire la vérification d'erreur. Cependant, vous continuerez même si vous détecter l'erreur. Vous avez vraiment besoin de s'arrêter.

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

Dans l'ensemble, il est le meilleur pour la fin des lignes avec un saut de ligne. Il vide les données dans le fichier ou de l'écran.

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

La printf() chaîne de format n'est pas portable. Étant donné que vous avez inclus <conio.h>, vous devriez probablement utiliser clrscr() (ce qui, probablement, n'envoyer que la séquence d'échappement, ou un équivalent de la terminale). Je ne suis pas convaincu qu'il est nécessaire pour effacer l'écran à la fin du programme, mais je travaille principalement sur les systèmes Unix, pas avec Windows.

}

Étant donné que MSVC est encore un C89 compilateur, vous devez inclure return 0; à la fin du programme de retour d'un état de réussite.

Réécrit

L'ajout de tous les changements, vous finirais avec:

#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);
}

Lorsque vous avez appris à propos de l'écriture de fonctions, vous pouvez le compresser pour:

#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);
}

La get_a_number() fonction n'est pas d'un usage général, fonction, mais il est utile dans ce contexte, puisqu'il encapsule un bout de code commun.

• Un sous-programme appel nous permet de résumer les irrégularités dans la liste d'arguments.
• La sous-routine résume à lui seul la régularités du code.

C'est une citation de Kernighan & Plauger, "Les Éléments de Style de Programmation".

---

À l'aide de atexit()

Le code ci-dessus a été mis à jour pour inclure getch() et clrscr() sur toutes les trajectoires de sortie (et supprimer #include <ctype.h>, puisque aucune de ces fonctions étaient plus utilisés). C'est une nuisance pour écrire les deux appels de fonction à plusieurs reprises. Une autre façon d'éviter ce problème est d'écrire une fonction qui est appelée lorsque le programme s'arrête; vous enregistrer avec le atexit() fonction, qui est déclarée dans <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);
}

Ce code dans main() suppose que vous pouvez mélanger des états (comme atexit()) et les déclarations (comme int val_a) de façon arbitraire. Qui est permise par l'actuel (2011) et les anciens (1999) versions de la Norme C, mais n'a pas été autorisée par l'original (1989) version de la Norme C. je crois que MSVC adhère à la norme d'origine. Si oui, vous avez probablement besoin d'écrire que main() comme:

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);
}

C'est une belle discussion point de savoir si le code à l'intérieur du bloc interne (à l'intérieur des accolades) devrait être mis en retrait d'un niveau supplémentaire ou pas. Vous êtes toujours autorisé à déclarer les variables en début de bloc de code entre accolades, alors, ce qui me permet d'écrire l'appel à atexit(), puis de définir et d'utiliser ces variables.

Si vous lisez à propos de la programmation Agile, vous verrez que l'un des mantras est:

• À SEC — Ne pas Répéter

Le réécrit ci-dessus pour éviter la répétition de l' getscr() appels pourrait être considéré comme une application de ce mantra. De la même façon avec les get_a_number() fonction; il s'applique à la SEC mantra.

---

D'essayer de nouveau

Si vous souhaitez que le programme allez retour et invite de nouveau si l'utilisateur fait une erreur en leur tapant, alors vous avez besoin de modifier la get_a_number() fonction. Et, parce que la fonction se développe considérablement plus complexe, vous certainement besoin à l'aide d'une fonction pour le travail. Vous aussi vous rencontrez un problème avec scanf(); il ne permet pas d'appliquer un nombre par ligne. Si vous avez tapé 1 2 en réponse à la première invite, le premier appel à get_a_number() de lire le un et le vide, arrêtez d'analyse (et de mettre le dos de blanc pour la réutilisation), et de revenir avec la valeur 1. Le deuxième appel à get_a_number() serait sortie de l'invite de commandes, mais de retour avec la valeur 2 sans que l'utilisateur en tapant quelque chose de plus. Si ce n'est pas le comportement que vous voulez, alors vous avez à prendre des dispositions pour lire l'intégralité de première ligne et d'analyse pour le numéro, puis jetez le reste. Il y a une autre raison pour inquiéter trop. Si l'utilisateur tape a au lieu de 1, le scanf() ne sera jamais rien lu de plus; il va entrer dans une boucle, en constatant que la a n'est pas valable qu'un chiffre, en renvoyant 0 conversions complète, et votre code sera probablement jusqu'à la fin dans une boucle infinie, d'une demande d'entrée et de lecture de la a. Ce genre de problème, c'est pourquoi beaucoup de gens (surtout moi) éviter d'utiliser scanf().

La façon la plus simple d'éviter la plupart des problèmes ci-dessus est avec fgets() et 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);
}

Qui couvre beaucoup de questions. Souvent, c'est une bonne idée de récupérer les données erronées. Vous pouvez le faire avec:

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

Il utilise 'près de concaténation de chaîne", et indique à l'utilisateur que le programme a reçu, ce qui peut souvent aider l'utilisateur à comprendre quel est le problème.

J'ai utilisé fputs(prompt, stdout) au lieu de printf(prompt) à la sortie de l'invite de commandes. Soit pourraient être utilisés (printf(prompt) != 0 serait le test), mais il y a un léger avantage à fputs() parce que si quelqu'un a réussi à contourner votre programme et d'obtenir un % dans l'invite, le printf() voudrais essayer de traiter un argument a pas été adopté, alors que fputs() ne pas interpréter sa chaîne d'argument à tous. Ici, la différence est négligeable; il peut être raisonnablement soutenu que je suis au-dessus-de compliquer les choses. Mais la sécurité des programmes est aussi quelque chose que vous de fin d'apprentissage, et des techniques telles que "ne pas utiliser printf(variable)" peut aider à rendre vos programmes plus résilient. (En fait, la complication est l'explication; le code est assez simple.)

Une autre point de discussion est " faut-il être un fflush() après fputs() et avant fgets()?' Il serait possible d'ajouter un petit "condition" fflush(stdout) == 0 && à la boucle de contrôle, mais dans la pratique, il est généralement inutile. Il ne ferait qu'une différence si l'entrée standard est à venir à partir d'autre chose que le terminal et la sortie standard allait à un fichier. Donc, je l'ai laissée.

Le compteur dans la boucle empêche les choses vont mal tourné — le programme des sorties plutôt que de traîner éternellement, frustrant l'utilisateur qui ne peuvent pas obtenir les données saisies correctement. Avez-vous déjà eu un programme ouvre une boîte de dialogue dans laquelle vous pouvez seulement frapper le bouton 'OK' (même si sa n'est pas OK) et puis revenir avec la même boîte de nouveau, maintes et maintes et maintes fois? Je ne l'espère pas pour votre bien. J'ai, il me frustre quand il arrive.

Et vous pouvez voir pourquoi il est avantageux d'écrire le code dans get_a_number() juste une fois, plutôt que deux fois; ce serait beaucoup de répétition.

Vous avez obtenu le isdigit fonction de tout mal. Il est destiné à recevoir un char argument, mais dans votre cas, l'int est converti, déclassé, pourrait-on dire.

La fonction est basée sur les codes ASCII des lettres, lien ici. Vous pouvez essayer ce pour voir par vous-même.

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

En outre, ce que vous avez besoin est un do{ /* something */} while(boolean);