Condition de sortie de boucle.

**CCACC02** · 27/09/2018, 10h51

Je reviens encore avec mon jeu de hasard, j'ai du refaire tout, j'étais hors sujet.... mais petit à petit, je progresse.
Au démarrage de ma page, je sollicite l'utilisateur pour un nombre de coups maximums et j'incrémente un compteur à chaque proposition. Le compteur augmente bien mais la sortie de boucle ne se réalise pas quand le compteur dépasse le nombre de tentatives maximum. Pour le reste, tout semble fonctionner même si l'alerte vous avez perdu s'affiche après l'alerte vous avez gagné mais si déjà je pouvais arrêter la boucle au bon moment ... un problème à la fois.

Code HTML :

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
<!DOCTYPE html>
<html lang="fr">
<head>
	<meta http-equiv="Content-Type" content="text/html; charset=UTF-8"/>
	<meta name="TITLE" content="J'apprends le JavaScript ):"/>
	<title>Je débute en javaScript ;)</title>
	<!--link href="styles.css" rel="stylesheet" type="text/css"/-->
	<script type="text/javascript" src="fichier_externe2.js"></script>
 
</head>
<body>
	<div id="wrapper">
		<header>
			<h1>Le JavaScript en s'amusant ... </h1>
				<form name="saisie" onsubmit="return Jouer()" method="post">
  					<input type="text"  name="user_Number" placeholder="Votre entrée ..." />
					<input type="submit" value="OK" />
 
				</form>
		</header>
		<main>
 
 
		</main>
	</div>
</body>
</html>

fichier java :

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
var nb_Coup = prompt("Merci de vouloir indiquer le nombre de tentatives maximales :", "Nombres de tentatives");
var compteur = 0;
var numberToFind = Math.floor(Math.random() * 101);
 
 
function Jouer() {
    console.log(numberToFind);
    console.log(nb_Coup + " tentatives max");
 
    var user_Number = document.forms["saisie"]["user_Number"].value;
 
    do {
        compteur++;
        console.log(compteur);
        if (user_Number !== "") {
            if (isNaN(user_Number)) {
                alert("Ce n'est pas un un nombre !");
                return false;
            }
        } else {
            alert("Ce champ doit être renseigné");
            return false;
        }
 
        if (user_Number < numberToFind) {
            alert("le nombre est trop petit");
            return false;
        } else if (user_Number > numberToFind) {
            alert("Votre nombre est trop grand");
            return false;
        } else {
            alert("Bravo ! Vous avez gagné :)");
        }
 
    }
    while (nb_Coup <= compteur);
    // alert("Vous avez perdu !");
 
 
}

Merci de votre aide

**badaze** · 30/09/2018, 00h31

Ca ne fonctionne pas parce que conceptuellement parlant ça ne peut pas fonctionner.

En effet tu fais une boucle pour qui traite l'ensemble des essais pour une tentative alors qu'il ne faut faire qu'une seule comparaison pour chaque tentative. Bref ta boucle ne sert à rien.

Code HTML :

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
63
64
65
66
67
68
69
70
71
72
<!DOCTYPE html>
<html lang="fr">
<head>
 <meta http-equiv="Content-Type" content="text/html; charset=UTF-8"/>
 <meta name="TITLE" content="J'apprends le JavaScript ):"/>
 <title>Je débute en javaScript ;)</title>
 <!--link href="styles.css" rel="stylesheet" type="text/css"/--> <script>
  var nb_Coup = prompt("Merci de vouloir indiquer le nombre de tentatives maximales :", "Nombres de tentatives");
  var compteur = 0;
  var numberToFind = Math.floor(Math.random() * 101);
   
  function Jouer() {
   console.log(numberToFind);
   console.log(nb_Coup + " tentatives max");
   
   var user_Number = document.forms["saisie"]["user_Number"].value;
   
   compteur++;
   console.log(compteur);
   if (user_Number !== "") {
    if (isNaN(user_Number)) {
     alert("Ce n'est pas un un nombre !");
     return false;
    }
   } else {
    alert("Ce champ doit être renseigné");
    return false;
   }
   
   if (compteur >= nb_Coup)
   {
    alert("perdu !!! le nombre à trouver était "+numberToFind);
    return true;
   }
   
   document.getElementById('coups').innerHTML = "Reste : "+(nb_Coup-compteur);
   
   if (user_Number < numberToFind) {
    alert("le nombre est trop petit");
    return false;
   } else if (user_Number > numberToFind) {
    alert("Votre nombre est trop grand");
    return false;
   } else {
    alert("Bravo ! Vous avez gagné :)");
    return true;
   }
   
  }
 </script>
 
</head>
<body>
 <div id="wrapper">
  <header>
   <h1>Le JavaScript en s'amusant ... </h1>
    <form name="saisie" onsubmit="return Jouer()" method="post">
       <input type="text"  name="user_Number" placeholder="Votre entrée ..." />
     <input type="submit" value="OK" />
    </form>
    <div id="coups"></div>
  </header>
  <main>
 
 
  </main>
 </div>
 <script>
 document.getElementById('coups').innerHTML = "Reste : "+(nb_Coup-compteur);
 </script>
</body>
</html>

Invité · 30/09/2018, 09h21

Bref ta boucle ne sert à rien.

jpense que sa boucle sert bien. il s'est juste planté il incrémente compteur, mais compare qq <= compteur qui sera tjs vrai tant que compteur augmente...

1) la comparaison à faire étant (en fait) compteur <= nb_coup

@ccacc02
ps: c'est bien d'indenter ton code, je suis pas intervenu sur ton autre discussion parce que ca faisait saigner les yeux

2) tes tests de sortie de boucle sont pas bon. Tu veux sortir de la boucle SI tu as trouvé le résultat...sinon tu restes dans ta boucle

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
    if (user_Number < numberToFind) {
        alert("le nombre est trop petit");
        //return false;
    } else if (user_Number > numberToFind) {
        alert("Votre nombre est trop grand");
        //return false;
    } else {
        alert("Bravo ! Vous avez gagné :)");
        return true //<--------
    }

**badaze** · 30/09/2018, 13h47

Envoyé par galerien69

jpense que sa boucle sert bien. il s'est juste planté il incrémente compteur, mais compare qq <= compteur qui sera tjs vrai tant que compteur augmente...

La boucle ne sert à rien puisqu'elle compare n fois la même valeur. De facto une seule fois suffit !

De plus le code que j'ai posté fonctionne (du moins je l'espère) sans la moindre boucle. C'est un indice. Non ?

**psychadelic** · 30/09/2018, 15h04

Si, la boucle ne sert à rien.

le nombre de coup est incrémenté à chaque fois que l'utilisateur "propose" sa réponse.

Invité · 30/09/2018, 17h43

La boucle ne sert à rien puisqu'elle compare n fois la même valeur. De facto une seule fois suffit !

le jeu que ccacc02 c'est le truc de base que t'apprends au début:
tu donnes 7 chances (ou plus) pour trouver un nombre, soit tu le trouves et c'est bon, soit tu le trouves pas et t'as perdu.
ici, la boucle, c'est le nombre de chances que tu as.

alors oui tu peux faire une récursive si çà te plait, (ou passer par le dom), mais je persiste à dire que le but de l'exercice c'est de faire une boucle

De plus le code que j'ai posté fonctionne (du moins je l'espère) sans la moindre boucle. C'est un indice. Non ?

concernant ton code, c'est l'approche que j'aime pas trop (çà n'engage que moi). Quand quelqu'un poste une question, je trouve pas top de lui dire: ce que tas fait c'est usuelle: fais autrement. (éventuellement si çà peut pas marcher c'est différent). Je trouve bien plus pertinent de lui donner la réponse précise à la question qu'il pose:
"mais la sortie de boucle ne se réalise pas quand le compteur dépasse le nombre"
et en l'occurrence c'est parce qu'il s'est planté dans l'ordre des opérandes

la après tu peux discuter de l'intérêt de la boucle etc, mais pour moi la première réponse à donner c'est celle qui correspond à la question posée.

qu'ils y aient des erreurs ou pas dans ton code en fait c'est pas très important, mais je me doute que tu l'as testé (ce qu'on fait tous bien sûr

)

ps: je dis pas que ta code sert à rien, c'est bien de proposer une approche différente

Condition de sortie de boucle.

JavaScript

Vue hybride

Discussions similaires

Partager

Partager