Discussion :

[benjamin_malaussene]

Bonjour je dois écrire un programme de résolution d'une équation du second degré. Les coefficients (réels) sont saisis au clavier. Les erreurs éventuelles doivent être traitées. Est-ce que je peux améliorer mon programme ? Par ailleurs, comment est-ce que je peux vider le buffer sans avoir à taper une deuxième fois sur entrée ?

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
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
 
#include <stdio.h>
#include <math.h>
#include <ctype.h>
 
void saisie(double* coeff,char c)
{
    int ok;
    char buffer[1024];
 
    printf("Entrez le coefficient %c : ",c);
    ok=scanf("%lf",coeff);
 
    if(getchar()!='\n' || ok==0)
    {
        printf("La saisie n'est pas valide.");
        exit (-1);
    }
    //fgets(buffer,1023,stdin);
    //while(getchar()!='\n');
}
 
void racines_polynome (double a,double b,double c,int* nbracines,double* x1,double* x2)
{
    double delta;
    delta=b*b-4*a*c;
 
    if(a==0)
        if (b==0)
            if(c==1)
                *nbracines=-2;
            else
                *nbracines=-1;
        else
        {
            *nbracines=1;
            *x1=-c/b;
        }
 
    else
    {
        if(delta<0)
            *nbracines=0;
        else
        {
            *nbracines=2;
            *x1=(-b+sqrt(delta))/(2*a);
            *x2=(-b-sqrt(delta))/(2*a);
        }
    }
}
 
int main()
{
    double a,b,c,x1,x2;
    int nbracines,ok;
    char rep;
 
    do
    {
        printf("Entrez les coefficients a,b et c du polynome ax2+bx+c : \n");
 
        saisie(&a,'a');
        saisie(&b,'b');
        saisie(&c,'c');
 
        racines_polynome(a,b,c,&nbracines,&x1,&x2);
 
        switch (nbracines)
        {
            case -2 : printf("Le polynome est constant et n'a pas de racines.\n");break;
 
            case -1 : printf("Le polynome est nul et a une infinite de racines.\n");break;
 
            case 0 : printf("Le polynome n'a pas de racines dans R.\n");break;
 
            case 1 : printf("Le polynome est du premier degre; racine=%lf.\n",x1);break;
 
            case 2 : {  if ( x1 == x2 )
                            printf( "Le polynome a une racine double; racine = %lf\n" , x1 );
                        else
                            printf( "Le polynome a 2 racines;x1 = %lg; x2 = %lf\n" , x1 , x2 );
                     }
        }
 
        printf("Voulez-vous recommencer ? o/n");
        scanf("%c",&rep);
        while(getchar()!='\n');
 
    } while (tolower(rep)!='n');
 
    return 0;
}

[Médinoc]

Par ailleurs, comment est-ce que je peux vider le buffer sans avoir à taper une deuxième fois sur entrée ?

À ma connaissance, ce problème n'est pas censé se poser après un scanf(), mais se pose après un fgets().
Le principe est de ne tenter de vider le buffer que si fgets() n'a pas retourné de \n.

Regarde du côté de ce code: http://www.developpez.net/forums/d14...e/#post8140473

[Sve@r]

Bonjour

Est-ce que je peux améliorer mon programme ?

Ne calculer delta que dans le cas où tu l'utilises !!!

Par ailleurs je ne comprends pas trop pourquoi tu testes si "c" vaut 1. Peut-être que l'utilisateur cherchera à résoudre 0x²+0x+18=0 !!!

Autre chose: tu as aussi un cas particulier où a, b et c sont différents de 0 mais où il n'y a quand-même qu'une seule solution: quand delta vaut 0 (ton programme fonctionne dans ce cas mais trouve deux solutions égales ce qui peut-être vu par ton prof comme une erreur).

Sinon on ne sort jamais par exit() d'une fonction autre que main(). On entre dans le programme par le main(), on en ressort par le main(). Si tu écris une fonction qui détecte et qui renvoie les erreurs alors tu dois ensuite faire l'effort (du coté appelant) de gérer ces erreurs récupérées (y compris renvoyer le problème à l'appelant du dessus jusqu'à arriver au main).

Par ailleurs, comment est-ce que je peux vider le buffer sans avoir à taper une deuxième fois sur entrée ?

Ca ça t'a déjà été expliqué: un fgets() pour vider le buffer et récupérer tout son contenu sous forme de chaine puis traiter la chaine par sscanf pour récupérer ses valeurs (et détecter les erreurs éventuelles).

À ma connaissance, ce problème n'est pas censé se poser après un scanf(), mais se pose après un fgets().

Ben si (ou alors j'ai mal compris ce que tu veux dire).
Si tu fais un scanf("%d"), comme tu es obligé de valider ta saisie par <return> tu stockes dans stdin un caractère '\n' qui n'est pas récupéré par scanf("%d") (ou scanf("%f")).
A la saisie suivante (surtout si tu fais saisir une chaine), tu récupères alors ce '\n' à la place de ce que tu aurais aimé taper...

[benjamin_malaussene]

Super merci beaucoup Svear, j'ai l'impression que je fais tout le temps des erreurs... J'ai modifié mon programme du coup :

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
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
 
#include <stdio.h>
#include <math.h>
#include <string.h>
 
int saisie(double* coeff,char c)
{
    int ok;
    char buffer[1024];
    char other[1024];
    char *c;
 
    printf("Entrez le coefficient %c : ",c);
    fgets(buffer,1023,stdin);
 
    if (c=strchr(buf, '\n')) *c='\0';
    if (sscanf(buffer, "%lf", coeff) == 0 || (sprintf(other, "%lf", *coeff), strcmp(buffer, other) != 0))
    {
        printf("La saisie n'est pas valide.");
        return (-1);
    }
    return 0;
}
 
void racines_polynome (double a,double b,double c,int* nbracines,double* x1,double* x2)
{
    double delta;
 
    if(a==0)
        if (b==0)
            if(c!=0)
                *nbracines=-2;
            else
                *nbracines=-1;
        else
        {
            *nbracines=1;
            *x1=-c/b;
        }
 
    else
    {
        delta=b*b-4*a*c;
 
        if(delta<0)
            *nbracines=0;
        else
        {
            *nbracines=2;
            *x1=(-b+sqrt(delta))/(2*a);
            *x2=(-b-sqrt(delta))/(2*a);
        }
    }
}
 
int main()
{
    double a,b,c,x1,x2;
    int nbracines,ok;
    char rep;
 
    do
    {
        printf("Entrez les coefficients a,b et c du polynome ax2+bx+c : \n");
 
        if(saisie(&a,'a')==-1 || saisie(&b,'b')==-1 || saisie(&c,'c')==-1)
            exit(-1);
 
        racines_polynome(a,b,c,&nbracines,&x1,&x2);
 
        switch (nbracines)
        {
            case -2 : printf("Le polynome est constant et n'a pas de racines.\n");break;
 
            case -1 : printf("Le polynome est nul et a une infinite de racines.\n");break;
 
            case 0 : printf("Le polynome n'a pas de racines dans R.\n");break;
 
            case 1 : printf("Le polynome est du premier degre; racine=%lf.\n",x1);break;
 
            case 2 : {  if ( x1 == x2 )
                            printf( "Le polynome a une racine double; racine = %lf\n" , x1 );
                        else
                            printf( "Le polynome a 2 racines;x1 = %lg; x2 = %lf\n" , x1 , x2 );
                     }
        }
 
        printf("Voulez-vous recommencer ? o/n");
        scanf("%c",&rep);
        while(getchar()!='\n');
 
    } while (tolower(rep)!='n');
 
    return 0;
}

[Sve@r]

Code :

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

1
2
3
4
if(c!=0)
    *nbracines=-2;
else
    *nbracines=-1;

Code :

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

*nbracines=(c != 0) ?-2 :-1;

...

[Médinoc]

Si tu fais un scanf("%d"), comme tu es obligé de valider ta saisie par <return> tu stockes dans stdin un caractère '\n' qui n'est pas récupéré par scanf("%d") (ou scanf("%f")).
A la saisie suivante (surtout si tu fais saisir une chaine), tu récupères alors ce '\n' à la place de ce que tu aurais aimé taper...

Ce que je veux dire, c'est qu'après un scanf("%d") on a toujours un \n restant dans le buffer clavier, et donc on n'est jamais confronté au problème de devoir appuyer une seconde fois sur Entrée.