Problème de SCANF dans une boucle WHILE

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:

---

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)

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

Code:

---

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:

---

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

---

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

---

Oups...
Ah bah oui quel con !

Merci ! :oops:

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

Alors on fait comme ça :

Code:

---

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

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

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