Noob a un pobleme avec While .

**Sobuga** · 03/08/2006, 22h30

Bonjour,
Mon probleme n'est pas grand mais me barbe assez car je ne sit pas ou se situe mon erreur et y a rien de plus horrible que ça ...

Voici mon programme :

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
#include <stdio.h>
 
main () {
 
	    char reponse='n';
    	while ( reponse == 'n' )
 
			       {
 
				 printf ("Est-ce terminé‚ ? ");
				 scanf ("%c", &reponse);
 
			       }
 
    	printf ("D'accord, je m'arrete.");
 
    	}

Lorsque j'éxécute le programme voici ce que j'obtient :

Est-ce terminé ? n
Est-ce terminé ? D'accord je m'arrete.

Donc je pense que vous voyez ou se situe mon probleme ...
Il me stop le programe alors que la variable reponse pend bien pour valeur 'n'.
Et encore pire, avant de me stopper le programme il me demande " Est-ce terminé ?" sans me laisser la possibilité de repondre en enchainant tout de suite avec le " D'accord, je m'arrete."

Help me Je sait pas ou est l'erreur dans mon programme ( qui entre nous est tres rudimentaire ).
Merci d'avance.

**millie** · 03/08/2006, 22h51

Plusieurs faute :

Code :

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

int main()

Et :

Code :

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

1
2
	char reponse[]="n"; // char reponse = 'n', fonctionne aussi
    	while ( *reponse == 'n' )

[edit] j'ai dit une connerie

**Sobuga** · 03/08/2006, 23h01

Envoyé par millie

Plusieurs faute :

Code :

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

int main()

Et :

Code :

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

1
2
	char* reponse='n'; // char reponse = 'n', fonctionne aussi
    	while ( *reponse == 'n' )

[edit] j'ai dit une connerie

Merci pour ta réponse mais lorsque je met " * " au edroit ou tu me la dit lorsque je compile le compilateur m'indique une erreur ...

De plus pour le int main ben dans mon cour de C y a pas marqué ça mais bon j'ai essayer quand meme et sa ne chande rien.

Au fait je me suis trompé sur l'erreur. Quand j'éxécute le progamme j'ai ceci :

Est-ce terminé ? n
Est-ce terminé ? D'accord je m'arrete.

C'est encore plus bizzare ...

**hegros** · 03/08/2006, 23h01

C'est un probléme lié au buffer de ton clavier et aux fonctions d'affichage vers l'écran.

Il faut consultez la Faq ou faire une recherche sur le site c'est un probléme assez récurent.Mais rassure toi au bout d'un moment on commence à s'y faire

**millie** · 03/08/2006, 23h03

Oui, en fait, fallait mettre :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
	    char reponse[]= "n"; //chaine de caractère (tableau de taille 2)
    	while ( *reponse == 'n' )
 
			       {
 
				 printf ("Est-ce terminé‚ ? \n");
				 scanf ("%s", reponse); //on prend toute une chaine
 
			       }
 
    	printf ("D'accord, je m'arrete.");

Excusez pour le premier post, je n'avais pas tester, et donc pas rendu compte de la connerie.

Le problème là étant que si tu tappes une ligne trop importante, cela va écrit dans la zone mémoire ou il y a le pointeur reponse, mais il n'y a que 2 caractères reservées, donc, écriture pas bonne en mémoire. A toi de trouver comment régler le problème !

**Sobuga** · 03/08/2006, 23h09

Envoyé par millie

Oui, en fait, fallait mettre :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
	    char reponse[]= "n"; //chaine de caractère (tableau de taille 2)
    	while ( *reponse == 'n' )
 
			       {
 
				 printf ("Est-ce terminé‚ ? \n");
				 scanf ("%s", reponse); //on prend toute une chaine
 
			       }
 
    	printf ("D'accord, je m'arrete.");

Excusez pour le premier post, je n'avais pas tester, et donc pas rendu compte de la connerie.

Le problème là étant que si tu tappes une ligne trop importante, cela va écrit dans là ou ya le pointeur reponse, mais il n'y a que 2 caractères reservées, donc, écriture pas bonne en mémoire. A toi de trouver comment régler le problème !

Merci

Pour le

Code :

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

1
2
char reponse[]= "n"; //chaine de caractère (tableau de taille 2)
    	while ( *reponse == 'n' )

ça n'a pas marché

.

Mais peut importe car le probleme venait en fait de scanf

il fallait mettre %s car %c. Pourquoi je sait pas mais ça marche

Merci encore

Par contre pour le uffer du clavier je sait pas ce que c'est et je vais aller regarder.

**Skyounet** · 03/08/2006, 23h22

Envoyé par Sobuga

Bonjour,
Mon probleme n'est pas grand mais me barbe assez car je ne sit pas ou se situe mon erreur et y a rien de plus horrible que ça ...

Voici mon programme :

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
#include <stdio.h>
 
main () {
 
	    char reponse='n';
    	while ( reponse == 'n' )
 
			       {
 
				 printf ("Est-ce terminé‚ ? ");
				 scanf ("%c", &reponse);
 
			       }
 
    	printf ("D'accord, je m'arrete.");
 
    	}

Lorsque j'éxécute le programme voici ce que j'obtient :

Est-ce terminé ? n
Est-ce terminé ? D'accord je m'arrete.

Donc je pense que vous voyez ou se situe mon probleme ...
Il me stop le programe alors que la variable reponse pend bien pour valeur 'n'.
Et encore pire, avant de me stopper le programme il me demande " Est-ce terminé ?" sans me laisser la possibilité de repondre en enchainant tout de suite avec le " D'accord, je m'arrete."

Help me Je sait pas ou est l'erreur dans mon programme ( qui entre nous est tres rudimentaire ).
Merci d'avance.

La réponse que je vais donner fonctionne, mais juste dans ce cas, dans de nombreux autres aussi, mais les membres de ce forum preconisent l'utilisation d'un buffer, puis ensuite de le vider, lui ainsi que le buffer d'entrée.

Mais bref, passons

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
 
#include <stdio.h>
 
int main(void)
{
    char reponse='n';
    while ( reponse == 'n' )
    {
        printf ("Est-ce terminé‚ ? \n");
        fflush(stdout);
        scanf ("%c%*c", &reponse);
    }
 
    printf ("D'accord, je m'arrete.\n");
 }

**granquet** · 03/08/2006, 23h26

si Emmanuel lit vos reponses, y vas vous tapper sur les doigts les gars

.

pour les debutants, on recommande de ne PAS utiliser scanf, qui est une fonction compliquée a mettre en oeuvre. (bien que ça paraisse simple)
une solution simple et a toute epreuve: fgets. (oubien fgetc)

http://emmanuel-delahaye.developpez.com/inputs.htm

si tu veux absolument utiliser scanf (moi j'ai renoncé); faut lire, comprendre, et assimiler ça: http://xrenault.developpez.com/tutoriels/c/scanf/

**Sobuga** · 03/08/2006, 23h31

Effectivement ce que tu me propose marche

Cependant je ne peut repondre " n " que une seul fois apres quoi si je retappe " n " une fois de plus la boule prend fin come si j'avais tapé " o ".

vraiment tres bizarre

En tout cas merci a tous pour votre aide.

**Emmanuel Delahaye** · 04/08/2006, 01h07

Envoyé par Sobuga

Voici mon programme :

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
#include <stdio.h>
 
main () {
 
	    char reponse='n';
    	while ( reponse == 'n' )
 
			       {
 
				 printf ("Est-ce terminé‚ ? ");
				 scanf ("%c", &reponse);
 
			       }
 
    	printf ("D'accord, je m'arrete.");
 
    	}

On t'a obligé à présenter ton code aussi mal ou c'est juste par esprit rebelle ?

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
#include <stdio.h>
 
main ()
{
   char reponse = 'n';
   while (reponse == 'n')
   {
      printf ("Est-ce terminé‚ ? ");
      scanf ("%c", &reponse);
   }
 
   printf ("D'accord, je m'arrete.");
}

**Emmanuel Delahaye** · 04/08/2006, 01h13

Envoyé par Dark_Ebola

si Emmanuel lit vos reponses, y vas vous tapper sur les doigts les gars

.

Ca va chier !

**David.Schris** · 04/08/2006, 01h58

http://c.developpez.com/faq/c/?page=...EN_scanf_fgets
http://c.developpez.com/faq/c/?page=...buffer_clavier

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
#include <stdio.h>
#include <stdlib.h> /* exit() / EXIT_SUCESS / EXIT_FAILURE */
 
int main(void) {
	int caractereLu;
	char reponse;
	int codeDeRetour=EXIT_FAILURE;
 
	do {
 
		printf( "Est-ce terminé ('n' pour continuer, une autre touche pour arrêter) ? " );
 
		caractereLu = getc( stdin ); /* lit un caractere */
 
		/* verifie si la lecture a reussi */
		if ( caractereLu == EOF ) {
 
			if ( 0 != ferror( stdin ) ) {
				fprintf( stderr, "Erreur lors de la lecture d'un caractère\n" );
				goto exit_main;
			} else {
				fprintf( stderr, "Fin de fichier rencontrée lors de la lecture d'un caractère\n" );
				goto exit_main;
			}
 
		}
 
		/* si oui, on recupere le caractere lu dans la reponse */
		reponse=(char)caractereLu;
 
		/* on vide le tampon */
		while ( (caractereLu = getc( stdin ) ) != '\n' && caractereLu != EOF);
 
	} while ( reponse == 'n' ); /* on recommence tant que la reponse est 'n' */
 
	codeDeRetour=EXIT_SUCCESS; /* si on arrive là, c'est que tout s'est bien passé */
 
exit_main:
	exit(codeDeRetour); /* on quitte */
}

Pour provoquer une fin de fichier, presser CTRL+D sous linux (je crois que c'est CTRL+Z sous Windows).

**granquet** · 04/08/2006, 02h22

le goto çaymal !

**Emmanuel Delahaye** · 04/08/2006, 02h33

Envoyé par Dark_Ebola

le goto çaymal !

Pour gérer proprement les erreurs c'est acceptable, mais attention à bien libérer les ressources...
Mais ici, pas besoin de goto...

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>             /* exit() / EXIT_SUCESS / EXIT_FAILURE */
 
int main (void)
{
   int codeDeRetour = EXIT_SUCCESS;
   int caractereLu;
 
   do
   {
      printf ("Est-ce terminé ('n' pour continuer, une autre touche pour arrêter) ? ");
      caractereLu = getc (stdin);  /* lit un caractere */
      /* verifie si la lecture a reussi */
      if (caractereLu == EOF)
      {
         if (0 != ferror (stdin))
         {
            fprintf (stderr, "Erreur lors de la lecture d'un caractère\n");
         }
         else
         {
            fprintf (stderr, "Fin de fichier rencontrée lors de la lecture d'un caractère\n");
         }
         codeDeRetour = EXIT_FAILURE;
      }
      else
      {
         /* on vide le tampon */
         int c;
         while ((c = getc (stdin)) != '\n' && c != EOF)
         {
         }
      }
   }
   while (caractereLu == 'n');
   /* on recommence tant que la reponse est 'n' */
 
   return codeDeRetour;         /* on quitte */
}

**David.Schris** · 04/08/2006, 08h50

Envoyé par Dark_Ebola

le goto çaymal !

Envoyé par Emmanuel Delahaye

Pour gérer proprement les erreurs c'est acceptable, mais attention à bien libérer les ressources...

C'est le but de cette façon de faire, même si ce n'est pas flagrant ici (pas de malloc() & Co).
En fait, je procède comme avec les parenthèses et les accolades : quand j'en ouvre une, je la ferme aussitôt et c'est seulement ensuite que je remplis entre les deux.
Idem pour un malloc() : j'écris le free() aussitôt (précédé d'une étiquette) et c'est seulement ensuite que je mets du code entre les deux.
Cela évite d'oublier de libérer des ressources et de se retrouver avec du code parfois moins lisible comme :

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
tentative reservation ressource1
si echec
    quitter
fin si
 
tentative reservation ressource2
si echec
    libération ressource1
    quitter
fin si
 
tentative reservation ressource3
si echec
    libération ressource2
    libération ressource1
    quitter
fin si
 
libération ressource3
libération ressource2
libération ressource1
quitter

Sans parler du cas où la libération de ressource elle-même peut échouer...

Une petite remarque pour terminer, en modifiant mon code tu as écrit :

Code :

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

int codeDeRetour = EXIT_SUCCESS;

C'est contre un principe que je me suis fixé et qui peut être rapproché du précédent : par défaut, et "jusqu'à preuve du contraire", je considère qu'une fonction va échouer. Dans pas mal de cas, cela permet d'éviter d'avoir plusieurs "codeDeRetour=EXIT_FAILURE;" (tout en ayant souvent qu'un seul "codeDeRetour=EXIT_SUCCESS;") et , surtout, cela évite d'oublier de renvoyer une erreur et de se rendre compte plus tard (dans une des fonctions appelantes) qu'il y a un problème (=> signaler l'erreur au plus tôt).

Voilà-voilà...

Bonne journée

Noob a un pobleme avec While .

C

Discussions similaires

Partager

Partager