[C] Problème de boucle While

Problème de boucle While [C] - C - Programmation

Marsh Posté le 08-01-2008 à 18:32:29    

Salut à tous! Et merci de me filer un coup de main! :wahoo:

 

Avant de commencer, je précise que je débute en C. :sarcastic:

 

Alors voilà j'ai un soucis avec une boucle while. Le but du programme et de réaliser un petit jeu Pierre, Feuille, Ciseaux. L'ordi tire au hasard un nombre correspondant à un des 3 choix. Jusque là pas de soucis.

 

Le joueur qui gagne remporte 1 euros et dès qu'un joueur atteint 10 euros, le programme est censé s'arrêter.
Et c'est là que se trouve le soucis... Le programme continue, même si l'un des joueurs dépasse 10 euros...
Je comprends pas pourquoi, pourriez-vous me l'expliquer s'il vous plaît??

 


Précision: Le code source se compile bien sans soucis... :sol:
Voici le code:

 
Code :
  1. /* File: HAYS-JEAN-GABRIEL-simple.c
  2. Auteurs: Jean-Gabriel HAYS
  3. cree le: 08/01/2008*/
  5. #include <stdio.h>
  6. #include <stdlib.h>
  7. #include <time.h>
  10. void choixHumain(long *humain);
  11. void choixOrdi(long *ordi);
  12. void compare(long choixH, long choixO, long *pointeurHumain, long *pointeurOrdi);
  15. int main()
  16. {
  17.     long aleatoire = 0;
  18.     const long MAX = 3;
  20.     long eurosHumain = 0, eurosOrdi = 0;
  21.     while (eurosHumain <= 10 || eurosOrdi <= 10)
  22.     {
  23.         srand(time(NULL)); //Initialisation du générateur de nombres
  24.         aleatoire = rand() % MAX; //L'ordinateur génère un nombre compris entre 0 et 2 inclus.
  27.         long selection = 0;
  29.         printf("Veuillez faire votre choix:\n0 = Ciseaux\n1 = Pierre\n2 = Papier\nVotre sélection: " );
  30.         scanf("%ld", &selection);
  33.         choixHumain(&selection);
  34.         choixOrdi(&aleatoire);
  37.         compare(selection, aleatoire, &eurosHumain, &eurosOrdi);
  39.         printf("\nVous avez %ld euros et l'ordinateur possède %ld euros\n", eurosHumain, eurosOrdi);
  40.     }
  41. return 0;
  42. }
  46. void choixHumain(long *humain)
  47. {
  48.     switch (*humain)
  49.     {
  50.         case 0:
  51.         printf("\nVous avez choisi les Ciseaux\n" );
  52.         break;
  54.         case 1:
  55.         printf("\nVous avez choisi la Pierre\n" );
  56.         break;
  58.         case 2:
  59.         printf("\nVous avez choisi le Papier\n" );
  60.         break;
  61.     }
  62. }
  66. void choixOrdi(long *ordi)
  67. {
  68.     switch (*ordi)
  69.     {
  70.         case 0:
  71.         printf("\nL'ordinateur a choisi les Ciseaux\n" );
  72.         break;
  74.         case 1:
  75.         printf("\nL'ordinateur a choisi la Pierre\n" );
  76.         break;
  78.         case 2:
  79.         printf("\nL'ordinateur a choisi le Papier\n" );
  80.         break;
  81.     }
  82. }
  86. void compare(long choixH, long choixO, long *pointeurHumain, long *pointeurOrdi)
  87. {
  89.     if (choixH == choixO)
  90.     {
  91.         printf("\nEgalité! Personne ne perd\n\n" );
  92.     }
  94. /*Cas pour lesquels l'ordinateur gagne*/
  95.     if (choixH == 0 && choixO == 1)
  96.     {
  97.         printf("\nLa pierre casse les ciseaux.\nVous avez perdu!\n\n" );
  98.         *pointeurOrdi += 1;
  99.     }
  100.     if (choixH == 1 && choixO == 2)
  101.     {
  102.         printf("\nLe papier enveloppe la pierre.\nVous avez perdu!\n\n" );
  103.         *pointeurOrdi += 1;
  104.     }
  105.     if (choixH == 2 && choixO == 0)
  106.     {
  107.         printf("\nLe ciseau coupe le papier.\nVous avez perdu!\n\n" );
  108.         *pointeurOrdi += 1;
  109.     }
  113. /*Cas pour lesquels le joueur Humain gagne*/
  114.     if (choixH == 1 && choixO == 0)
  115.     {
  116.         printf("\nLa pierre casse les ciseaux.\nVous avez gagné!\n\n" );
  117.         *pointeurHumain += 1;
  118.     }
  119.     if (choixH == 2 && choixO == 1)
  120.     {
  121.         printf("\nLe papier enveloppe la pierre.\nVous avez gagné!\n\n" );
  122.         *pointeurHumain += 1;
  123.     }
  124.     if (choixH == 0 && choixO == 2)
  125.     {
  126.         printf("\nLe ciseau coupe le papier.\nVous avez gagné!\n\n" );
  127.         *pointeurHumain += 1;
  128.     }
  129. }

Message cité 1 fois
Message édité par janothays le 08-01-2008 à 18:33:52

---------------
http://www.mandrivalinux-online.eu
Reply

Marsh Posté le 08-01-2008 à 18:32:29   

Reply

Marsh Posté le 08-01-2008 à 20:00:28    

unidentified a écrit :

utilise un if ...


 
Bah, ouais, c'est clair que sa boucle va boucler après ça ....
 
Perso, j'aurais plutôt utilisé un && au lieu de || dans le while. Le && voulant dire dans ce contexte : si un des joueurs à 10 pts, alors on dégage.

Message cité 1 fois
Message édité par tpierron le 08-01-2008 à 20:00:42
Reply

Marsh Posté le 08-01-2008 à 20:22:48    

tpierron a écrit :

Perso, j'aurais plutôt utilisé un && au lieu de || dans le while. Le && voulant dire dans ce contexte : si un des joueurs à 10 pts, alors on dégage.


 
+1
Je voudrais juste expliquer la condition telle qu'elle est écrite dans le code pour une meilleure compréhension :

while (eurosHumain <= 10 || eurosOrdi <= 10)


signifie : "tant que eurosHumain <= 10 ou eurosOrdi <= 10, alors boucler"
 
Donc ton while bouclera tant qu'au moins l'un des joueurs aura moins de 10€. Ce qui n'est pas ce que tu veux. Pour la suite de l'explication, lire le post de tpierron.

Reply

Marsh Posté le 08-01-2008 à 21:49:13    

Merci à vous! :hello:  
J'ai compris pourquoi il fallait mettre && plutôt que ||
 
Ça marche niquel... A un détail près, ça allait jusqu'à 11 effectivement. J'ai donc mis:
 

Code :
  1. while (eurosHumain < 10 && eurosOrdi < 10)


 
plutôt que:
 

Code :
  1. while (eurosHumain <= 10 && eurosOrdi <= 10)


---------------
http://www.mandrivalinux-online.eu
Reply

Marsh Posté le 08-01-2008 à 23:28:21    

Une heuristique utile pour ce jeu contre un joueur humain:
- Jouer le coup qui aurait été le coup perdant lors de la partie précédente
 
En effet, des etudes statistiques montrent qu'un joueur humain a tendance a jouer le coup qui l'aurait battu au tour précédent.
 
A+,


---------------
There's more than what can be linked! --    Iyashikei Anime Forever!    --  AngularJS c'est un framework d'engulé!  --
Reply

Marsh Posté le 09-01-2008 à 09:24:55    

Comme ça y'a qu'a jouer le coup qui aurait été le coup gagnant lors de la partie précédente, et on est sûr de gagner :D

Reply

Marsh Posté le 09-01-2008 à 09:51:21    

Non, c'est le coup perdant de la partie precedente qui devient le coup gagnant de la partie suivante (ou plus justement, qui statistiquement, a de plus grandes chances d'être le coup gagnant). Le coup qui aurait été le coup gagnant lors de la partie précédente, c'est ce que va avoir tendance a jouer le joueur humain.
 
 
Apres, quand on veut aller plus loin dans l'heuristique, il faut aussi tenir compte d'une tendance a jouer un peu plus caillou que les deux autres chez les joueurs humains.
L'article d'ou je tire ces infos: http://media.newscientist.com/data [...] 263566.pdf
Quelques liens sur une compet entre programmes jouants l'un contre l'autre décrite dans l'article:
http://cplus.about.com/od/programm [...] lenge1.htm
(le code du vainqueur actuel) http://cplus.about.com/library/dow [...] eleton.cpp
 
 
A+,


---------------
There's more than what can be linked! --    Iyashikei Anime Forever!    --  AngularJS c'est un framework d'engulé!  --
Reply

Marsh Posté le 09-01-2008 à 11:37:46    

ta notion d'initialisation est assez floue.

Message cité 1 fois
Reply

Marsh Posté le 10-01-2008 à 00:02:48    

Taz a écrit :

ta notion d'initialisation est assez floue.


Tu veux dire quoi par là? :heink:

Reply

Marsh Posté le 10-01-2008 à 11:20:32    

Tu initialises la seed de ta fonction aléatoire dans une boucle. Pour peu que tu joues deux fois avant que l'heure ne change (à la seconde près), ton nombre aléatoire sera le même ces deux fois.
 
Tu pourrais donc par exemple sortir ton srand(time(NULL)); de ta boucle while pour éviter ce comportement.

Message cité 1 fois
Reply

Marsh Posté le 10-01-2008 à 11:20:32   

Reply

Marsh Posté le 10-01-2008 à 12:50:47    

OK merci, je vais changer ça. ;-)


---------------
http://www.mandrivalinux-online.eu
Reply

Marsh Posté le 11-01-2008 à 17:56:59    

ccp6128 a écrit :

Tu initialises la seed de ta fonction aléatoire dans une boucle. Pour peu que tu joues deux fois avant que l'heure ne change (à la seconde près), ton nombre aléatoire sera le même ces deux fois.
 
Tu pourrais donc par exemple sortir ton srand(time(NULL)); de ta boucle while pour éviter ce comportement.


+1
 
Détails : http://mapage.noos.fr/emdel/notes.htm#rand
 


---------------
Des infos sur la programmation et le langage C: http://www.bien-programmer.fr Pas de Wi-Fi à la maison : http://www.cpl-france.org/
Reply

Marsh Posté le 11-01-2008 à 17:58:27    

janothays a écrit :


Avant de commencer, je précise que je débute en C.
<...>


C'est plutôt joliment codé pour un débutant...
 
(enfin, à première vue...)


---------------
Des infos sur la programmation et le langage C: http://www.bien-programmer.fr Pas de Wi-Fi à la maison : http://www.cpl-france.org/
Reply

Marsh Posté le 12-01-2008 à 00:59:50    

unidentified a écrit :

Ouais ... Je dirais même qu'il y a la patte du site du zero là-dedans [:figti]


A cause des 'long' ? Oui, on a ralé, ça devrait changer...
 


---------------
Des infos sur la programmation et le langage C: http://www.bien-programmer.fr Pas de Wi-Fi à la maison : http://www.cpl-france.org/
Reply

Sujets relatifs:

Leave a Replay

Make sure you enter the(*)required information where indicate.HTML code is not allowed