Discussion :

[Margatthieu]

Bonjour

J'essaye en vain de récuperer une saisie utilisateur, pour ensuite la comparer
(et faire un menu basique)

j'essaye avec getchar et strcmp mais j'ai constamment des erreurs du style
`strcmp' makes pointer from integer without a cast

Quelle est la bonne méthode pour récuperer une saisie utilisateur et la comparer ?

Merci d'avance
Matthieu

[Fabllot]

Bonjour!

Pour la saisie utilisateur :
http://c.developpez.com/faq/?page=cl...EEN_gets_fgets

Pour la comparaison :
strcmp ou strncmp - ne pas oublier l'inclusion de string.h...

[Margatthieu]

merci pour les infos

Pourquoi ne pas utiliser scanf ?

[Emmanuel Delahaye]

merci pour les infos

Pourquoi ne pas utiliser scanf ?

Parce que c'est trop complexe. Si tu y tiens, forme toi :

http://xrenault.developpez.com/tutoriels/c/scanf/

[Margatthieu]

merci Emmanuel, je vais utiliser fgets

J'utilise le code ci dessous mais il ne demande pas la saisie utilisateur
il passe directement au printf (...)
et donc n'affiche rien ...

Code :

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

1
2
3
char cResponse[30];
fgets(cResponse, sizeof cResponse, stdin); 
printf ("cResponse: %s", cResponse);

Tu n'aurais pas une idée du pb ?

Merci
Matthieu

[hegros]

Ajoute un fflush(stdout) après le printf

Sinon pour que le programme te demande la saisie il faut faire juste avant le fgets

Code :

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

puts("Attente de la saisie utilisateur");

[Margatthieu]

ça ne fonctionne toujours pas ... je vois vraiment pas mon erreur ...
le compilation fonctionne bien

voici le code complet (juste la fonction)
j'ai mis en rouge le 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
void ajouterModifierPersonne (ListePersonnes *listeP)
{
	char cResponse[30];
	
	/*On crée une nouvelle personne sans l'ajouter à la liste*/
	Personne *laNewPersonneP = nouvellePersonne();
	
	Personne *personneExistante = rechercherPersonne (listeP, laNewPersonneP->nom);

	/*Si la personne n'existe pas, on la crée et return*/
	if (personneExistante == NULL)
		ajouterPersonne (listeP, laNewPersonneP);
	else
	{
		if ( (strcmp(laNewPersonneP->telephone, personneExistante->telephone)) != 0 )
		{
			printf ("\n");
			printf ("La personne existe déja\n");
			printf ("Le numéro de telephone est différent\n");
			printf ("Voulez vous remplacer la personne existante ? (Y/N)\n");
			printf ("\n");
			
			/*saisie de la réponse utilisateur*/
			puts("Attente de la saisie utilisateur");
			fgets(cResponse, sizeof cResponse, stdin); 
			printf ("cResponse: %s", cResponse);
			fflush(stdout);
			/* code à faire ...*/
		}
		else
		{
			printf ("\n");
			printf ("La personne existe déja\n");
			printf ("Le numéro de telephone est identique\n");		
		}
	}
}

Est ce que tu vois le pb ?

[hegros]

Tu veux dire que ca affiche 'attente saisie utilisateur' et qu'ensuite ca t'affiche directement 'Creponse' ? Sinon pourquoi déclarer CReponse au tout début autant le faire dans l'alcolade de ton if avant ton fgets.

[Margatthieu]

oui, c'est ça

mais je prefererais qu'il attende la saisie d'un caractére ...

[Médinoc]

Es-tu sûr qu'il n'y a pas eu une saisie avant ?
Une saisie qui aurait laissé le \n dans le buffer d'entrée ?

[Margatthieu]

merci Medinoc mais j'ai rajouté un fflush(stdin); juste aprés le puts, et même probleme ...

[Médinoc]

La fonction fflush() n'est définie que pour les flux sortants.

Ce qui fait que fflush(stdin) est un comportement indéfini, c'est-à-dire un bug.

[Margatthieu]

et comment je peux m'assurer que le buffer d'entrée soit vide ?

[Margatthieu]

il doit surement y avoir un pb de buffer d'entrée comme tu dis puisque j'ai essayé avec getchar()
et ça à fonctionné 1 fois et maintenant ça ne fonctionne plus ...

Qqu'un aurait il une idée ?

[Margatthieu]

c'est bon:

Code :

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

1
2
3
4
5
6
void purge(void)
{
	int c;
	while((c=getchar())!=EOF && c != '\n')
	{ }
}