Discussion :

[fa911662]

Bonjour à tous,
Voilà mon problème je débute en C et je bug sur un exercice..
Je dois rédiger un petit programme qui permet de calculer le périmètre d'un cercle voici le code que j'ai rédigé :

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
#include <stdio.h>
#include <stdlib.h>
 
int main()
{
    float pi = 3.14;
    float rayon = 0,perimetre = 0, aire = 0;
   char reponse;
    printf("bonjour\n");
    printf("entrez la valeur du valeur du rayon svp\n");
    scanf("%f",& rayon);
    printf( "souhaiter vous obtenir le perimetre(O/N)?");
   scanf(" %c",& reponse);
    if(reponse == O)
   {
       perimetre = 2*pi*rayon;
       printf(" la valeur du perimetre du cercle de rayon %f est %f\n",rayon,perimetre);
   }
 
 
   printf("souhaitez vous obtenir l air du cercle (O/N)?");
 
   scanf("%c",&reponse);
 
    if(reponse == O)
    {
        aire = pi*rayon*rayon ;
 
        printf("la valeur de l air du cercle est %f\n",aire);
 
 
    }
    printf(" merci d avoir utiliser ce programme bonne journee");
}

Tout se passe bien au départ mais après la première instruction le programme s'enchaîne sans me demander si je veux calculer l'air ( if(reponse == O) ) en gros cette instruction est ignorer si je peux dire ça ainsi et je ne vois pas trop pourquoi..
Merci d'avance...

[DrDarko]

Bonjour,

Vu que ta variable reponse est un caractère il faut soit que tu l'as compare à un nombre en te reportant sur le tableau ASCII, soit que tu l'as compare à 'O'.

[Bktero]

Ce code ne peut pas compiler sans erreur : /home/pierre/Documents/TestsC/main.c|14|error: ‘O’ undeclared (first use in this function)|En modifiant pour avoir 'O' en rajoutant un petit printf, tu comprendras le problème :

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
#include <stdio.h>
#include <stdlib.h>
 
int main(void)
{
    float pi = 3.14;
    float rayon = 0,perimetre = 0, aire = 0;
    char reponse;
    printf("bonjour\n");
    printf("entrez la valeur du valeur du rayon svp\n");
    scanf("%f",& rayon);
    printf( "souhaiter vous obtenir le perimetre(O/N)?\n");
 
    scanf(" %c",& reponse);
    printf("Caractere lu : [%c]", reponse); // BKTERO
 
 
    if(reponse == 'O')
    {
        perimetre = 2*pi*rayon;
        printf(" la valeur du perimetre du cercle de rayon %f est %f\n",rayon,perimetre);
    }
 
 
    printf("souhaitez vous obtenir l air du cercle (O/N)?\n");
 
    scanf("%c",&reponse);
    printf("Caractere lu : [%c]", reponse); // BKTERO
 
    if(reponse == 'O')
    {
        aire = pi*rayon*rayon ;
        printf("la valeur de l air du cercle est %f\n",aire);
    }
 
    printf("\n\nmerci d avoir utiliser ce programme bonne journee");
 
    return 0;
}

Quand tu entres 'O' suivi de ENTREE, le buffer clavier contient 'O' et '\n' mais le premier scanf (pour les caractères, donc le 2e du programme) ne le consomme pas, cet '\n'. Le second le lit et s'arrête là. Scanf est une fonction délicate, il y a un tutoriel bien fait dans la rubrique cours C, lis-le.

Si tu souhaites récupérer un seul caractère du clavier, il faut mieux utiliser getchar().

Attention aussi à scanf pour récupérer l'entier, il reste aussi un '\n' dans le buffer clavier (qui pose soucis pour utiliser getchar() juste derrière).

[diogene]

Lorsque tu fais un scanf(), l'entrée est lue de la façon suivante :
- les caractères <blancs> (au sens de isspace() donc également le '\n') sont lus et ignorés, sauf pour les formats "%c", "%n" et "%[...".
- la valeur est lue et s'arrête dès que le caractère ne correspond plus au format spécifié
Le reste de la ligne reste en attente de traitement.

Par exemple :

Code :

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

1
2
scanf("%f",&a);
scanf("%f",&b);

avec une entrée 12345 6789 <return> ou une entrée 12345 <return> 6789 <return> donnera a==12345 et b==6789

Code :

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

1
2
scanf("%f",&a);
scanf("%c",&b);

avec une entrée 12345x <return> donnera a==12345 et b=='x' sans besoin d'une entrée supplémentaire
avec une entrée 12345 x <return> donnera a==12345 et b==' ' sans besoin d'une entrée supplémentaire
et l'entrée 12345<return> donnera a==12345 et b=='\n' sans besoin d'une entrée supplémentaire

Si on veut s'affranchir de ce comportement, il faut s'assurer que la ligne est entièrement consommée avant la lecture en "%c". Par exemple,

Code :

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

scanf("%f%*[^\n]%*c", &a);

le %*[^\n] lit tous les caractères différents (à cause du ^) de '\n' et les ignorent (à cause de *)
le %*c lit le caractère '\n' et l'ignore.

[Sve@r]

Salut
Bienvenue dans ce monde. A la fois simple et complexe.
Donc comme d'autres te l'ont écrit, le vrai problème vient de scanf et de sa façon de gérer le buffer stdin qui donc n'est pas du tout destiné à gérer les saisies humaines mais qu'on est bien obligé d'utiliser quand on débute car on n'a rien d'autre.

Maintenant voici le même exercice mais utilisant une autre méthode plus robuste pour faire saisir des valeurs

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
#include <stdio.h>
#include <stdlib.h>
 
int main()
{
    float pi = 3.14;
    float rayon = 0;
    char saisie[101];
    char reponse;
 
    printf("bonjour\n");
    while(1)
    {
        printf("entrez la valeur du rayon svp ? "); fflush(stdout);
        fgets(saisie, 101, stdin);
        if (sscanf(buffer, "%f", &rayon) == 1 && rayon >= 0) break;
        printf("saisie incorrecte - recommencez\n");
    }
 
    printf("souhaitez vous obtenir le périmètre (O/N)? "); fflush(stdout);
    fgets(saisie, 101, stdin);
    sscanf(buffer, "%c", &reponse);
    if(reponse == 'O' || reponse == 'o')
        printf("la valeur du périmètre du cercle de rayon %f est %f\n", rayon, 2*pi*rayon);
 
    printf("souhaitez vous obtenir l'aire (O/N)? "); fflush(stdout);
    fgets(saisie, 101, stdin);
    sscanf(buffer, "%c", &reponse);
    if(reponse == 'O' || reponse == 'o')
        printf("la valeur de l'aire du cercle de rayon %f est %f\n", rayon, pi*rayon*rayon);
    printf("merci d'avoir utilisé ce programme et bonne journée\n");
}

Ce programme n'est pas à l'épreuve du couillon qui rentrerait 102 caractères lors de la saisie mais peut gérer les cas les plus courants. Evite cependant de le présenter à ton prof comme étant le tien car il ne sera pas dupe. Je ne l'ai pas écrit pour te permettre de briller temporairement par une tricherie mais pour que tu puisses mieux comprendre les subtilités de la saisie et t'améliorer de façon à ce que tu puisses briller naturellement tout au long de ta vie. Et pour ce faire, je te suggère aussi de te documenter sur fgets() et sur sscanf().

PS: le fflush(stdout) est là pour que printf() affiche son texte immédiatement sans attendre le vidage automatique. Mais on éviterait ce souci en remplaçant les printf("question") par des fputs("question", stdout)...

[Kirilenko]

PS: le fflush(stdout) est là pour que printf() affiche son texte immédiatement sans attendre le vidage automatique. Mais on éviterait ce souci en remplaçant les printf("question") par des fputs("question", stdout)...

stdin étant bufferisé par ligne, utiliser fputs ne changera pas grand chose par rapport à printf, car il n'y a pas de '\n' ajoutés automatiquement, comme c'est le cas avec puts.

[Sve@r]

stdin étant bufferisé par ligne, utiliser fputs ne changera pas grand chose par rapport à printf, car il n'y a pas de '\n' ajoutés automatiquement, comme c'est le cas avec puts.

Exact, erreur de ma part, fputs() n'imprime pas non plus de façon immédiate et donc nécessite lui-aussi un fflush(stdout) si on veut voir son action se faire immédiatement (et on parle bien de stdout et non stdin)...

[Bktero]

Si on met un \n à la fin de la chaine de printf(), est-ce que cela suffit (si c'est juste une histoire de retour à la ligne) ?

D'ailleurs, je ne met jamais de fflush(stdout) dans mes programmes. Dans quels cas pourrais-je avoir des soucis ?

[Pouet_forever]

L'utilisation d'un '\n' force le vidage du tampon, donc ça suffit.
Pour l'utilisation du ffflush(stdout), je n'ai pas d'exemples en tête, mais j'ai déjà été obligé de l'utiliser (je ne me souvient plus du contexte).

[Sve@r]

D'ailleurs, je ne met jamais de fflush(stdout) dans mes programmes. Dans quels cas pourrais-je avoir des soucis ?

Uniquement (si tu n'utilises pas QtCreator) dans un cas similaire à celui-ci

Code c :

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

1
2
3
4
5
6
int main()
{
    printf("Hello...");
    sleep(15);
    printf("...world\n");
}

Ton "Hello" ne sortira réellement à l'écran qu'après les 15 secondes (et donc tu auras "Hello......world" d'un coup...)