Discussion :

[xeron33]

Débutant en C : voici une boucle simple while :

Code :

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

1
2
3
4
5
6
7
 
while (printf("voulez-vous-continuer ?O/N\n"),scanf("%c",&rep),rep=='O')
 
	 {
	 printf("while%c\n",rep);
 
	 }

mon problème est que à la première exécution si je rentre "O" il exécute bien la boucle et au second passage il affiche "voulez-vous-continuer ?O/N" puis s'arrête sans exécuter la boucle.Là j'avoue je ne comprends pas ! Si quelqu'un peut m'aiguiller
Merci à tous A +

[sidahmed]

Code :

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

1
2
3
4
do
 printf("Voulez-vous continuer ? O/N \n");
 scanf("%c", &rep);
while (rep == 'O');

Si rep différent de 'O', on sort de la boucle.
J'espère avoir répondu à la question.
Au revoir.

[Emmanuel Delahaye]

Débutant en C : voiçi une boucle simple while :

Code :

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

1
2
3
4
5
6
7
 
while (printf("voulez-vous-continuer ?O/N\n"),scanf("%c",&rep),rep=='O')
 
	 {
	 printf("while%c\n",rep);
 
	 }

mon problème est que à la première exécution si je rentre "O" il exécute bien la boucle et au second passage il affiche "voulez-vous-continuer ?O/N" puis s'arrête sans exécuter la boucle.Là j'avoue je ne comprends pas ! Si quelqu'un peut m'aiguiller

Le problème de scanf() avec "%c", c'est qu'il ne lit qu'un caractère à la fois alors que tu en as entré 2 ("O\n"). Au tour suivant, il lit le \n sans s'arréter...

http://emmanuel-delahaye.developpez.com/notes.htm#fgetc

Il est déconseillé d'utiliser scanf() si on ne le maitrise pas. Il est conseillé d'utiliser fgets().

http://emmanuel-delahaye.developpez....tes.htm#saisie
http://emmanuel-delahaye.developpez....s.htm#fichiers

[Emmanuel Delahaye]

Code :

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

1
2
3
4
do
 printf("Voulez-vous continuer ? O/N \n");
 scanf("%c", &rep);
while (rep == 'O');

Si rep différent de 'O', on sort de la boucle.

Ca cache le problème, mais ça ne le résout pas...

http://emmanuel-delahaye.developpez.com/notes.htm#fgetc

[Freed0]

Ca manque d'accolades aussi...

[xeron33]

Le problème de scanf() avec "%c", c'est qu'il ne lit qu'un caractère à la fois alors que tu en as entré 2 ("O\n"). Au tour suivant, il lit le \n sans s'arréter...

http://emmanuel-delahaye.developpez.com/notes.htm#fgetc

Il est déconseillé d'utiliser scanf() si on ne le maitrise pas. Il est conseillé d'utiliser fgets().

http://emmanuel-delahaye.developpez....tes.htm#saisie
http://emmanuel-delahaye.developpez....s.htm#fichiers

J'ai suivi ton conseil et j'ai aussi compris le problème du scanf(entrée de "O" puis de "\n") et suis allé sur ton site parlant de fgets mais je n'arrive pas à mes fins voiçi le code :

Code :

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

1
2
3
4
5
6
7
8
 
char rep[1];
 while (printf("voulez-vous-continuer ?O/N %c\n",rep),fgets(rep,sizeof rep,stdin),rep=='O')
 
	 {
	 printf("while%c\n",rep);
 
	 }

à la compil j'ai l'erreur suivante :comparaison between pointer and integer
Si tu peux m'aiguiller encore merçi à toi et à tous les autres qui particpent au forum A +

[gl]

à la compil j'ai l'erreur suivante :comparaison between pointer and integer

Le test a effectuer n'est pas rep == 'O' mais rep[0] == 'O'.

En outre, ton code serait certainement beaucoup plus clair en utilisant une boucle do{} while(); et en sortant l'invite et la saisie du test.
Il serait aussi preferable de tester le retour de fgets() et de vider le buffer d'entree suite au fgets() (afin de gerer les saisies de plus d'un caractere qui laisseraient de chose dans ce buffer).

[Thierry Chappuis]

J'ai suivi ton conseil et j'ai aussi compris le problème du scanf(entrée de "O" puis de "\n") et suis allé sur ton site parlant de fgets mais je n'arrive pas à mes fins voiçi le code :

Code :

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

1
2
3
4
5
6
7
8
 
char rep[1];
 while (printf("voulez-vous-continuer ?O/N %c\n",rep),fgets(rep,sizeof rep,stdin),rep=='O')
 
	 {
	 printf("while%c\n",rep);
 
	 }

à la compil j'ai l'erreur suivante :comparaison between pointer and integer
Si tu peux m'aiguiller encore merçi à toi et à tous les autres qui particpent au forum A +

Je n'ai pas beaucoup de temps, mais si tu désires saisir un unique caractère '0' ou N' avec fgets(), le tableau rep doit contenir au minimum 3 cases: une pour le caractère 'O' ou 'N', une pour '\n' et une pour '\0'.


Thierry

[Thierry Chappuis]

Voici une bonne manière de faire avec fgets():

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
#include <stdio.h>
#include <stdlib.h>
#include <ctype.h>
#include <string.h>
 
void fclean(char *s_buffer, FILE *stream);
 
int main(void)
{
    char answer[3] = "";
    char *p_eof = NULL;
 
    do
    {
        /* On commence par afficher l'invite */
        printf("Voulez-vous continuer? (o/n) ");
        fflush(stdout);
 
        /* Saisie: p_eof reçoit la valeur NULL si la fin de fichier est
           rencontrée sans qu'aucun autre caractère ait été saisi. */
        if ((p_eof = fgets(answer, sizeof answer, stdin)) != NULL)
        {
            /* Après usage de fgets() on vérifie que la saisie n'a pas été
               tonquée et, si nécessaire, on nettoie le flux d'entrée standard.
            */
            fclean(answer, stdin);
            if (tolower((unsigned char) answer[0]) == 'o')
            {
                printf("While %c\n", answer[0]);
            }
        }
 
    } while (tolower((unsigned char) answer[0]) != 'n' && p_eof != NULL);
 
    return EXIT_SUCCESS;
}
 
/* Vérifie que la saisie avec fgets() n'a pas été tronquée. Si tel est le cas.
   le caractère de fin de ligne est remplacé par le caractère nul. Sinon, le
   tampon du flux d'entrée stream est nettoyé.
*/
void fclean(char *s_buffer, FILE *stream)
{
    if (s_buffer != NULL)
    {
        /* On teste la présence du caractère de fin de ligne */
        char *pc = strchr(s_buffer, '\n');
 
        if (pc != NULL)
        {
            /* La saisie est complète: on remplace '\n' par '\0' */
            *pc = 0;
        }
        else /* La saisie a été tronquée: on vide le tampon de stdin */
        {
            int c;
            while ((c = fgetc(stream)) != '\n' && c != EOF)
            {
                /* Cette boucle ne fait rien */
                continue;
            }
        }
    }
}

Une autre solution (à mon avis plus simple dans ce cas) utilisant la fonction fgetc() pour la saisie d'un caractère unique:

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
#include <stdio.h>
#include <stdlib.h>
#include <ctype.h>
 
void fpurge(FILE *stream);
 
int main(void)
{
    int answer;
 
    do
    {
        printf("Voulez-vous continuer? (o/n) ");
        fflush(stdout);
 
        if ((answer = fgetc(stdin)) != '\n' && answer != EOF)
        { /* La saisie n'est pas vide */
 
            /* On teste la réponse de l'utilisateur */
            if (tolower(answer) == 'o')
            {
                printf("While %c\n", answer);
            }
 
            /* On doit encore purger le tampon du flux d'entrée standard */
            fpurge(stdin);
        }
    } while (tolower(answer) != 'n' && answer != EOF);
 
    return EXIT_SUCCESS;
}
 
/* Purge le flux stream */
void fpurge(FILE *stream)
{
    int c;
    while ((c = fgetc(stdin)) != '\n' && c != EOF)
    {
        continue;
    }
}

N'hésite pas à poser des questions si tu ne comprends pas.

Thierry

[xeron33]

Voici une bonne manière de faire avec fgets():

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
#include <stdio.h>
#include <stdlib.h>
#include <ctype.h>
#include <string.h>
 
void fclean(char *s_buffer, FILE *stream);
 
int main(void)
{
    char answer[3] = "";
    char *p_eof = NULL;
 
    do
    {
        /* On commence par afficher l'invite */
        printf("Voulez-vous continuer? (o/n) ");
        fflush(stdin);
 
        /* Saisie: p_eof reçoit la valeur NULL si la fin de fichier est
           rencontrée sans qu'aucun autre caractère ait été saisi. */
        if ((p_eof = fgets(answer, sizeof answer, stdin)) != NULL)
        {
            /* Après usage de fgets() on vérifie que la saisie n'a pas été
               tonquée et, si nécessaire, on nettoie le flux d'entrée standard.
            */
            fclean(answer, stdin);
            if (tolower((unsigned char) answer[0]) == 'o')
            {
                printf("While %c\n", answer[0]);
            }
        }
 
    } while (tolower((unsigned char) answer[0]) != 'n' && p_eof != NULL);
 
    return EXIT_SUCCESS;
}
 
/* Vérifie que la saisie avec fgets() n'a pas été tronquée. Si tel est le cas.
   le caractère de fin de ligne est remplacé par le caractère nul. Sinon, le
   tampon du flux d'entrée stream est nettoyé.
*/
void fclean(char *s_buffer, FILE *stream)
{
    if (s_buffer != NULL)
    {
        /* On teste la présence du caractère de fin de ligne */
        char *pc = strchr(s_buffer, '\n');
 
        if (pc != NULL)
        {
            /* La saisie est complète: on remplace '\n' par '\0' */
            *pc = 0;
        }
        else /* La saisie a été tronquée: on vide le tampon de stdin */
        {
            int c;
            while ((c = fgetc(stream)) != '\n' && c != EOF)
            {
                /* Cette boucle ne fait rien */
                continue;
            }
        }
    }
}

Une autre solution (à mon avis plus simple dans ce cas) utilisant la fonction fgetc() pour la saisie d'un caractère unique:

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
#include <stdio.h>
#include <stdlib.h>
#include <ctype.h>
 
void fpurge(FILE *stream);
 
int main(void)
{
    int answer;
 
    do
    {
        printf("Voulez-vous continuer? (o/n) ");
        fflush(stdin);
 
        if ((answer = fgetc(stdin)) != '\n' && answer != EOF)
        { /* La saisie n'est pas vide */
 
            /* On teste la réponse de l'utilisateur */
            if (tolower(answer) == 'o')
            {
                printf("While %c\n", answer);
            }
 
            /* On doit encore purger le tampon du flux d'entrée standard */
            fpurge(stdin);
        }
    } while (tolower(answer) != 'n' && answer != EOF);
 
    return EXIT_SUCCESS;
}
 
/* Purge le flux stream */
void fpurge(FILE *stream)
{
    int c;
    while ((c = fgetc(stdin)) != '\n' && c != EOF)
    {
        continue;
    }
}

N'hésite pas à poser des questions si tu ne comprends pas.

Thierry

Bonjour Thierry et merçi pour tes tuyaux. J'ai essayé pour l'instant ton code pour fgets mais tu utilises l'instruction

Code :

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

 return EXIT_SUCCESS

et le compilateur n'est pas d'accord car "undeclared".Je suppose que EXIT_SUCCESS est une étiquette mais je la vois pas dans ton code. Je connais la fonction EXIT mais pas EXIT_SUCCESS.
D'autre part l'instruction suivante :

Code :

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

1
2
 
if (tolower((unsigned char) answer[0]) == 'o')

m'interpèle... c'est quoi "tolower" ?
Merçi de m'aiclairer à +

[xeron33]

Bonjour Thierry et merçi pour tes tuyaux. J'ai essayé pour l'instant ton code pour fgets mais tu utilises l'instruction

Code :

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

 return EXIT_SUCCESS

et le compilateur n'est pas d'accord car "undeclared".Je suppose que EXIT_SUCCESS est une étiquette mais je la vois pas dans ton code. Je connais la fonction EXIT mais pas EXIT_SUCCESS.
D'autre part l'instruction suivante :

Code :

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

1
2
 
if (tolower((unsigned char) answer[0]) == 'o')

m'interpèle... c'est quoi "tolower" ?
Merçi de m'aiclairer à +

Pardon Thierry je viens de m'appercevoir que j'avais oublié d'inclure certaines bibliothèques que tu avais dans ton code désolé, je te tiens au courant de la suite ... Salut A +

[xeron33]

Encore moi malgrès tout avant de quitter le forum je te repose la question car malgrés qu'à la compil ça passe maintenant c'est quoi "tolower" et EXIT_SUCCES ?
Merçi Thierry et à +

[Emmanuel Delahaye]

c'est quoi "tolower" et EXIT_SUCCES ?

C'est écrit dans ton livre de C...

Une bonne référence pour les fonctions : http://man.developpez.com

et pour éviter à l'avenir d'oublier des headers : http://emmanuel-delahaye.developpez....tm#cfg_compilo

[Thierry Chappuis]

Encore moi malgrès tout avant de quitter le forum je te repose la question car malgrés qu'à la compil ça passe maintenant c'est quoi "tolower" et EXIT_SUCCES ?
Merçi Thierry et à +

Ma bibliothèque standard définit la constante EXIT_SUCCESS comme étant égale à 0. Par convention, la valeur de cette constante est celle retournée par main lors d'une exécution avec succès.

L'utilisation de tolower() (déclarée dans le fichier d'en-tête standard ctype.h) permet de convertir les réponses 'O' ou 'N' éventuellement saisies par l'utilisateur en 'o' et 'n', respectivement. C'est un petit sucre ici, car l'invite que lit l'utilisateur est la suivante: "Voulez-vous continuer? (o/n) ". Si on veut que seules les réponses 'o' et 'n' soient acceptées, l'usage de la fonction tolower() est inutile.

Thierry

[Thierry Chappuis]

Pardon Thierry je viens de m'appercevoir que j'avais oublié d'inclure certaines bibliothèques que tu avais dans ton code désolé, je te tiens au courant de la suite ... Salut A +

Comme on peut le lire plusieurs fois par jour sur ce forum... ce ne sont pas des bibliothèques, mais des fichiers d'en-tête.

Thierry

[xeron33]

Merçi à tous pour vos réponses voiçi ou j'en suis niveau code :

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
 
 
#include <stdio.h>
#include <stdlib.h>
#include <ctype.h>
#include <string.h>
#define LGMAX 100
 
void fclean(char *s_buffer, FILE *stream);
int main(void)
{
  struct enr
  {
	 char nom[21];
	 char prenom[16];
	 int age;
	 char numtel[12];
  } ;
  struct enr f_enr[LGMAX];
  int i=0;
  char rep[3]="";
  char *p_eof=NULL;
/*while (printf("voulez-vous-continuer ?O/N %c\n",rep),scanf("%c",&rep),rep=='O')*/
  do 
  {
	 printf("voulez-vous-continuer ?O/N ");
	 fflush(stdin);
	 if ((p_eof = fgets(rep, sizeof rep, stdin)) != NULL)
	         {
	             /* Après usage de fgets() on vérifie que la saisie n'a pas été
	                tonquée et, si nécessaire, on nettoie le flux d'entrée standard.
	             */
	            fclean(rep, stdin);
	             if	 (tolower((unsigned char) rep[0])=='o')
	             {
	                 printf("While %c\n", rep[0]);
	             }
	         }
 
  }
  while (tolower((unsigned char) rep[0]) != 'n' && p_eof != NULL);
 
	     return EXIT_SUCCESS;
 
 
	 /* Vérifie que la saisie avec fgets() n'a pas été tronquée. Si tel est le cas.
	    le caractère de fin de ligne est remplacé par le caractère nul. Sinon, le
	    tampon du flux d'entrée stream est nettoyé.
	 */
void fclean(char *s_buffer, FILE *stream)
	 {
	     if (s_buffer != NULL)
	     {
	         /* On teste la présence du caractère de fin de ligne */
	         char *pc = strchr(s_buffer, '\n');
 
	         if (pc != NULL)
	         {
	             /* La saisie est complète: on remplace '\n' par '\0' */
	             *pc = '\0';
	         }
	         else /* La saisie a été tronquée: on vide le tampon de stdin */
	         {
	             int c;
	  /* fgetc lit le caractère courant du flux int fgetc(FILE * flux)   */      
	             while ((c = fgetc(stream)) != '\n' && c != EOF)
	             {
	                 /* Cette boucle ne fait rien */
	                  continue;
	             }
	         }
	     }
	 }

A la compil pas de problème mais à l'édition de lien j'ai le message suivant :
"undefined reference to fclean"
Oui Thierry ta fonction a pour paramètres 2 pointeurs et

Code :

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

fclean(rep, stdin);

a pour paramètres rep et stdin , j'ai essayé de mettre 2 pointeurs en paramètres mais non ça marche pas là si tu peux m'aiguiller... Merçi encore pour toutes tes explications A+

[gl]

A la compil pas de problème mais à l'édition de lien j'ai le message suivant : "undefined reference to fclean"

Il manque l'accolade fermante a la fin du main.

En outre que vient faire le fflush(stdin); dans ce code ?
fflush() n'est defini que sur les flux sortant, fflush(stdin); est donc un comportement indetermine. De plus ici tu voulais certainement force l'affichage de ton texte, c'est donc plutot fflush(stdout);

[Thierry Chappuis]

Il manque l'accolade fermante a la fin du main.

En outre que vient faire le fflush(stdin); dans ce code ?
fflush() n'est defini que sur les flux sortant, fflush(stdin); est donc un comportement indetermine. De plus ici tu voulais certainement force l'affichage de ton texte, c'est donc plutot fflush(stdout);

Je suis responsable de ce fflush(stdin) qui s'est infiltré. C'était une faute de frappe et j'ai corrigé mon post.

Thierry

[gl]

Je suis responsable de ce fflush(stdin) qui s'est infiltré. C'était une faute de frappe et j'ai corrigé mon post.

Au temps pour moi, je n'avais pas remarque ce probleme dans ton post.

[xeron33]

Je suis responsable de ce fflush(stdin) qui s'est infiltré. C'était une faute de frappe et j'ai corrigé mon post.

Thierry

Bonsoir Thierry, maintenant ça marche j'ai laissé le fflush qui apparement pose pas de problème par contre j'aimerais que tu m'expliques le but du code suivant dans ta fonction fclean mis en italique et en gras en fait si j'ai bien compris tu remplaces "\n" par" \O" dans quel but ?(Qu'elle est l'utilité de "\O" par apport à "\n"? D'ailleurs autre chose dans ton code tu avais :

Code :

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

*pc = '0';

que j'ai remplaçé par

Code :

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

*pc = '\0';

et l'une ou l'autre solution marche.
Merçi si tu peux me préciser tout ça.J'aurai d'autres questions sur cette fonction mais plus tard... Et merçi encore à toi et à tous ceux qui suivent ce site trés utile !!!

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
void fclean(char *s_buffer, FILE *stream)
	 {
	     if (s_buffer != NULL)
	     {
	         /* On teste la présence du caractère de fin de ligne */
	         char *pc = strchr(s_buffer, '\n');
	  
	         if (pc != NULL)
	         {
	             /* La saisie est complète: on remplace '\n' par '\0' */
	             *pc = '\0';
	         }	         else /* La saisie a été tronquée: on vide le tampon de stdin */
	         {
	             int c;
	  /* fgetc lit le caractère courant du flux int fgetc(FILE * flux)   */      
	             while ((c = fgetc(stream)) != '\n' && c != EOF)
	             {
	                 /* Cette boucle ne fait rien */
	                  continue;
	             }
	         }
	     }
	 }