Discussion :

[FidoDido®]

En fait je demande dans une fonction à l'utilisateur de faire un choix et tant que sa réponse ne correspond pas aux choix possible de boucler sur la question avec un while :

Code :

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

1
2
3
4
5
6
7
8
9
 
do
			{
			  printf("Joueur 2 choisissez la couleur d'atout svp :\n(coeur : 0, carreau : 1, trefle : 2, pique : 3)\n");
			  scanf("%d",rep_atout);
			  getchar();
			  printf("%d\n",*rep_atout);
			}
		      while(*rep_atout!=0 || *rep_atout!=1 || *rep_atout!=2 || *rep_atout!=3);

J'utilise un do while parce que je veux que la question soit posée au moins une fois.
Le seul problème c'est qu'à l'exécution, cela boucle sans fin.

Merci !
FidoDido®

(désolé si c'est moche comme code, je débute)

[Trap D]

Déjà un scanf c'est pas beau mais es-tu sûr de ton test ??, ce ne serait pas plutôt

Code :

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

1
2
3
4
5
6
7
8
do
{
           printf("Joueur 2 choisissez la couleur d'atout svp :\n(coeur : 0, carreau : 1, trefle : 2, pique : 3)\n");
           scanf("%d",rep_atout);
           getchar();
           printf("%d\n",*rep_atout);
}
while(*rep_atout!=0 && *rep_atout!=1 && *rep_atout!=2 && *rep_atout!=3);

car avec le || il y en a toujours au moins 1 qui est vrai .
Une méthode simple pour obtenir le test c'est
je veux

Code :

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

*rep_atout ==0 || *rep_atout ==1 || *rep_atout == 2 || *rep_atout ==3

Donc, je vais boucler tant que je n'aurais pas ceci donc

Code :

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

while (!(*rep_atout ==0 || *rep_atout ==1 || *rep_atout == 2 || *rep_atout ==3));

[FidoDido®]

Oups...
Ah bah oui quel con !

Merci !

Bon donc c'est moche un scanf, mais qu'est-ce qui est plus joli, on ne m'a appris que ça pour l'instant...

[Trap D]

Alors on fait comme ça :

Code :

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

1
2
3
char buf[assez_grand];
fgets(buf, sizeof(buf), stdin);
*rep_atout= strtol(buf, NULL, 10);

strtol est une fonction sécurisée qui fait la transformation des chaînes en entiers, il ne faut pas utiliser atoi.
scanf est une fonction très difficile a utiliser car comme son nom l'indique elle suppose que les entrées sont formatées, or la saisie humaine est rarement formatée, (ou alors le format est des plus illogiques )

[Pouic]

http://xrenault.developpez.com/tutoriels/c/scanf/#L4.2

Mais utilise fgets si tu le peux... Cela te simplifiera la vie...