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" ?

[Sve@r]

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

Exact. scanf est une fonction déstinée à traiter une entrée formatée. Exemple: tu veux traiter une info contenant "12 c toto" tu peux utiliser scanf("%d %c %s").
Maintenant scanf() utilisé pour traiter ce qui vient de l'utilisateur est une très mauvaise idée car l'utilisateur peut taper n'importe quoi. Dans le meilleur des cas tu te retrouves toujours avec ce "\n" qui est forcément tapé pour valider la saisie et qui est enregistré aussi dans le buffer.

Une première solution est de supprimer ce "\n" par un getchar() juste après mais si l'utilisateur tape "zorglub" quand tu attends un nombre ça va pas t'avancer beaucoup...

La solution habituellement préconisée est d'utiliser fgets() pour traiter ce qui vient du clavier comme une chaîne (le "\n" est inclus dedans) puis de traiter cette chaîne par sscanf() pour en extraire les infos qui t'intéressent. Et le reste tu le laisses tomber (ou tu t'en sers pour prévenir l'utilisateur qu'il a mal rentré ses trucs).

Bien que scanf() soit une fonction habituellement présentée en premier aux débutants (car sa mise en oeuvre ne nécessite pas de grandes connaissances et permet de suite de commencer à faire de petits trucs) il convient, quand on commence à vouloir faire un travail plus important, de s'en séparer assez vite...

[Thierry Chappuis]

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

Je n'ai pas regardé en détails, mais la deuxième de tes solutions convient. Ta fonction est équivalente à un appel à fclean(NULL, stdin) proposé plus haut.

Thierry

[Angelina007]

Merci beaucoup pour ces compléments !
Je vois un peu plus clair maintenant