Discussion :

[Angelina007]

Bonsoir tout le monde
après des heures de recherches à devenir fou
je décide de m'en réferer à vous car sinon ...

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
 
#include <stdio.h>
#include <stdlib.h>
 
void initialiserPile(Pile** p);
void affichePile(Pile* p);
void push (Pile **p, char a);
 
int main(){
	Pile* p;
 
	char car;
 
	initialiserPile(&p);
	affichePile(p);
 
	printf("Entrer un mot se finissant par '.'\n");
	printf("Entrer la valeur de c : \n");
 
 
	do{
		car=getchar();
		if(car!='.'){
			push (&p, car);
			printf("Valeur rajouté !!\n");
		}
		else printf("Valeur non rajouté !!");
	}while(car!='.');
 
	push (&p, 'z');
	push (&p, 'u');
 
	affichePile(p);
 
	return(0);
}
 
 
 
 
 
void initialiserPile(Pile** p){
	*p=NULL;
}
 
void affichePile(Pile* p){
	int n=0;	
	while(p!=NULL){
		printf("val de a : %c\n",p->valeur);
		p=p->prec;
		n++;
	}	
	printf("TAILLE DE LA PILE : %d\n",n);
}
 
void push (Pile **p, char a){
	Pile *tmp;
	tmp=(Pile*) malloc(sizeof(Pile));
	tmp->valeur=a;
	tmp->prec=*p;
	*p=tmp;
}

Le problème, est que la boucle while, à chaque
passage, est éxécutée deux fois pour je ne sais quelle raison;
et exactement le même comportement si j'utilise un scanf à la place de getchar()



Vous ne le voyez pas, mais quand j'affiche la taille de la pile suite à cette
exemple, elle est également doublé, donc à chaque fois qu'il y a eu :
"Entrer la valeur de a :"
il a rajouté "RIEN?" à la pile


J'ai alors essayé de me passer du while, et utilisé ceci à la place pour
essayer de deviner d'où vient ce fameux 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
 
....
 
	printf("===>Entrer la valeur de c : \n");
	scanf_s("%c",&car);
	if(car!='.') push (&p, car);
 
	printf("===>Entrer la valeur de c : \n");
	scanf_s("%c",&car);
	if(car!='.') push (&p, car);
 
	push (&p, 'z');
	push (&p, 'u');
....

et cette fois-ci, le premier scanf marche parfaitement, mais le 2è
n'a pas du tout l'air d'avoir été pris en comte, et le caractère espace,
ou NULL, semble avoir été mis dedans ...
Dans les deux cas, les caractères 'z', et 'u', sont bien rajoutés à la pile




J'aimerais bien connaître mes erreurs dans les deux cas si possible

[Skyounet]

Simplement quand tu tapes ta lettre et que tu appuis sur la touche Entrée, ton buffer d'entrée contient la lettre tapée et le \n du Entrée.

Ton getchar prend la lettre tapée et lors du tour de boucle suivant getchar prend directement le \n sans te laisser le temps de taper quoi que ce soit.

Solution : utiliser fclean que tu trouveras sur ce forum (fais une recherche sur fclean).

fclean sera a utiliser après chaque getchar.

Voilà.

Au fait je n'ai pas regardé le reste du code.

[Thierry Chappuis]

Simplement quand tu tapes ta lettre et que tu appuis sur la touche Entrée, ton buffer d'entrée contient la lettre tapée et le \n du Entrée.

Ton getchar prend la lettre tapée et lors du tour de boucle suivant getchar prend directement le \n sans te laisser le temps de taper quoi que ce soit.

Solution : utiliser fclean que tu trouveras sur ce forum (fais une recherche sur fclean).

fclean sera a utiliser après chaque getchar.

Voilà.

Au fait je n'ai pas regardé le reste du code.

La plupart des implantations de fclean() proposées sur ce forum sont prévues pour fonctionner avec fgets() et prennent un tableau de caractères en argument duquel elles eliminent le caractère de fin de ligne saisi par fgets(). Le tampon du flux entrant est purgé lorsque saisie a été tronquée par fgets. Voici une implantation qui permet d'utiliser fclean() indifféramment pour faire le ménage après utilisation de fgets() ou fclean():

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
#include <stdio.h>
#include <string.h>
 
void fclean(char *tampon, FILE *fp)
{
    if (fp != NULL)
    {
        char *pc = NULL;
 
        if (tampon != NULL)
        {
            pc = strchr(tampon, '\n');
        }
 
        if (pc != NULL)
        {
            *pc = 0;
        }
        else
        {
            int c;
 
            while ((c = fgetc(fp)) != '\n' && c != EOF)
            {
            }
        }
    }
}

Lorsque tu utilises la fonction fclean() ci-dessus après getchar(), il faut passer NULL en 1er argument.

Thierry

[Angelina007]

Merci beaucoup de votre aide

Par contre, je n'ai pas tout bien compris la fonction
void fclean(char *tampon, FILE *fp)
Notemment à quoi correspond FILE *fp

Par contre, grâce à votre aide, j'ai trouvé deux solutions :

Code :

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

1
2
3
4
5
6
7
8
9
10
	do{
		printf("Entrer la valeur de c : \n");
		car=getchar();
		if(car!='.' && car !='\n'){
			push (&p, car);
			printf("Valeur Pushe !!\n");
		}
		else printf("Valeur non Pushe !!\n");
	}while(car!='.');

et

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

static void purger(void){
    int c;
    while ((c = getchar()) != '\n' && c != EOF){}
}
	do{
		printf("Entrer la valeur de c : \n");
		car=getchar();
		purger();
		if(car!='.'){
			push (&p, car);
			printf("Valeur Pushe !!\n");
		}
		else printf("Valeur non Pushe !!\n");
	}while(car!='.');

Mais je pense que la 2è solution est plus "propre" à moins que je me trompe

Merci beaucoup !!

[Emmanuel Delahaye]

Merci beaucoup de votre aide

Par contre, je n'ai pas tout bien compris la fonction
void fclean(char *tampon, FILE *fp)
Notemment à quoi correspond FILE *fp

C'est le flux entrant. Par exemple, ici, c'est stdin...

Pour les raison expliquées ici, je conseille de ne pas s'éloigner des solutions proposées (fgets(), fclean() etc.), à moins d'être un expert du langage C.

[Angelina007]

Cela signifierait-il que les méthodes que j'emploie ne sont pas très "propres" ?