Discussion :

[benjamin_malaussene]

Bonjour je veux faire une table de multiplication mais je veux empecher la saisie d'un réel ou d'un entier suivi de caractères :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
 
#include <stdio.h>
 
int main ()
{
    int entier,i,ok;
 
    printf("Entrer un entier positif : ");
    ok=scanf("%d",&entier);
    while (getchar()!='\n');
 
    if(entier>=0 && ok!=0)
    {
        for(i=1;i<=10;i++)
            printf("%d x %d = %d\n",entier,i,entier*i);
    }
    if(ok==0)
        printf("Le nombre saisi n'est pas un entier.");
    if(entier<0)
        printf("Le nombre saisi est negatif");
 
    return 0;
}

[benjamin_malaussene]

J'ai réussi à bidouiller qqc, dites moi si je peux l'améliorer :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
 
#include <stdio.h>
 
int main ()
{
    int entier,i,ok;
 
    char c=0;
 
    printf("Entrer un entier positif : ");
    ok=scanf("%d",&entier);
    if(getchar()!='\n')
        c=getchar();
 
    if(entier>=0 && ok==1 && c==0)
    {
        for(i=1;i<=10;i++)
            printf("%d x %d = %d\n",entier,i,entier*i);
    }
    if(c!=0)
        printf("Le nombre saisi n'est pas un entier ou contient des caracteres.\n");
    else if(ok==0)
        printf("Le nombre saisi n'est pas un entier.\n");
    if(entier<0)
        printf("Le nombre saisi est negatif");
 
    return 0;
}

[Médinoc]

Traditionnellement, tu pourrais faire une boucle qui se poursuit jusqu'à une saisie conforme ou jusqu'à ce que scanf() retourne une valeur négative.

[Sve@r]

J'ai réussi à bidouiller qqc, dites moi si je peux l'améliorer :

Bonjour

Bien sûr

Code c :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
#include <stdio.h>
int saisie()
{
    char buff[1024];
    int n;
 
    while (1)
    {
        fputs("Entrer un entier positif : ", stdout);
        fflush(stdout);
        fgets(buf, 1024, stdin);
        if (sscanf(buf, "%d", &n) == 0)
        {
            printf("La valeur (%s] n'est pas un nombre entier - Recommencez\n", buf);
            continue;
        }
        if (n < 0)
        {
            printf("Le nombre (%d] est négatif - Recommencez\n", n);
            continue;
        }
        break;
    }
    return n;
}
 
int main ()
{
    int entier, i;
    entier=saisie();
    for(i=1;i<=10;i++)
        printf("%d x %d = %d\n",entier,i,entier*i);
    }
    return 0;
}

Toujours essayer de séparer les groupes d'actions. Tu as d'une part une action dédiée à la saisie (et au contrôle de celle-ci) et d'autre part une action dédiée au traitement de ce qui a été saisi. Et utiliser fgets() te permet de ne pas laisser des saletés dans stdin ce qui t'évite ce getchar() à répétition (sauf si un abruti s'amuse à rentrer plus de 1024 caractères mais je pense que c'est assez rare pour ne pas avoir à blinder aussi contre ce genre de truc...)

[benjamin_malaussene]

Bonjour SVear, je te remercie pour ta réponse, effectivement l'utilisation de fgets est très intéressante. Par contre j'évite d'utiliser les break et les continue pour que le code soit plus lisible. Mais dans ton code tu ne vérifies pas si un utilisateur tape un chiffre suivi d'un caractère par exemple.

[Sve@r]

Par contre j'évite d'utiliser les break et les continue pour que le code soit plus lisible.

Tu as bien tort. Certains profs coincéspuristes n'aiment pas en effet les break et les continue parce que cela nuit à l'harmonie de l'univers (ou bien ça dérange les voix qu'ils ont dans leur tête, j'en sais rien) mais tu verras que parfois, un bon break/continue au bon endroit rend justement un code bien plus lisible que tout autre chose...

Mais dans ton code tu ne vérifies pas si un utilisateur tape un chiffre suivi d'un caractère par exemple.

Exact - J'avais tapé vite fait

Code rectifié (et tu verras, j'ai enlevé le break pour te faire plaisir

Code c :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
#include <stdio.h>
#include <string.h>
int saisie()
{
    char buf[1024];
    char other[1024];
    int n;
    char *c;
 
    while (1)
    {
        fputs("Entrer un entier positif : ", stdout);
        fflush(stdout);
        fgets(buf, 1024, stdin);
        if (c=strchr(buf, '\n')) *c='\0';
 
        if (sscanf(buf, "%d", &n) == 0 || (sprintf(other, "%d", n), strcmp(buf, other) != 0))
        {
            printf("La valeur [%s] n'est pas un nombre entier - Recommencez\n", buf);
            continue;
        }
        if (n < 0)
        {
            printf("Le nombre [%d] est négatif - Recommencez\n", n);
            continue;
        }
        return n;
    }
}
 
int main ()
{
    int entier, i;
    entier=saisie();
    for(i=1;i<=10;i++)
        printf("%d x %d = %d\n",entier,i,entier*i);
    return 0;
}

C'est ça aussi l'avantage de découper: quand une partie de l'action est incomplète, elle est facile à rectifier...

[benjamin_malaussene]

Salut svear, merci pour ta réponse, c'est super car tu me fais découvrir des fonctions sur les chaines de caractères que je ne connaissais pas. Par contre est ce que tu pourrais m'expliquer cette ligne ?

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
if (c=strchr(buf, '\n')) *c='\0';

Merci beaucoup