Noob a un pobleme avec While .

Version imprimable

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:

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.

Plusieurs faute :

Code:

int main()

Et :
Code:

1 2 char reponse[]="n"; // char reponse = 'n', fonctionne aussi while ( *reponse == 'n' )
[edit] j'ai dit une connerie

03/08/2006, 22h01
Sobuga
Citation:
Envoyé par millie

Plusieurs faute :

Code:

int main()

Et :

Code:

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 ...
03/08/2006, 22h01
hegros

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 ;)

Oui, en fait, fallait mettre :
Code:

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 !

Citation:
Envoyé par millie
Oui, en fait, fallait mettre :
Code:

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 :D
Pour le
Code:

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 :D
il fallait mettre %s car %c. Pourquoi je sait pas mais ça marche :D

Merci encore

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

Citation:
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:

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:

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"); }

03/08/2006, 22h26
granquet

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

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/
03/08/2006, 22h31
Sobuga

Effectivement ce que tu me propose marche :D
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 ".

:D vraiment tres bizarre :D

En tout cas merci a tous pour votre aide.

Citation:

Envoyé par Sobuga

Voici mon programme :

Code:

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:

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.");
}

04/08/2006, 00h13
Emmanuel Delahaye

Citation:

Envoyé par Dark_Ebola

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

http://www.sptimes.com/2003/06/23/photos/xprs-hulk.jpg
Ca va chier !

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

Code:

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).

04/08/2006, 01h22
granquet

le goto çaymal ! :langue:

Citation:

Envoyé par Dark_Ebola

le goto çaymal ! :langue:

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

Code:

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 */
}

Citation:

Envoyé par Dark_Ebola

le goto çaymal ! :langue:

:langue:

Citation:

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:

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:

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 :)