Aucun affichage du résultat de ma fonction

**CCACC02** · 19/09/2018, 11h53

Bonjour,

Je cherche à faire un petit jeu de hasard... sans recopier bêtement une solution toute faite sur internet pour comprendre le fonctionnement de la programmation jS.

voici mon html (tout simple ) :

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
<!DOCTYPE html>
<html lang="fr">
<head>
	<meta http-equiv="Content-Type" content="text/html; charset=UTF-8"/>
	<meta name="TITLE" content="Mon jeu de Hasard ):"/>
	<title>Devine un nombre .... compris entre 1 et 100 ;)</title>
	<link href="styles.css" rel="stylesheet" type="text/css"/>
	<script type="text/javascript" src="gen_number.js"></script>
</head>
<body>
	<div id="wrapper">
		<header>
			<h1>Le JavaScript en s'amusant ... Jeu de hasard et de découverte</h1>
 
		</header>
		<main>
			<form>
				<input type="button" onclick="partie()" value="Jouer"/>
 
			</form>
		</main>
	</div>
</body>
</html>

et mon Javascript

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
function partie() {
var X = Math.floor(Math.random()*101);
var nb; 
var cpt = 0;
var affichage = "Donnez-moi un chiffre entre 1 et 100 ..." ;
 
 
do{
	nb = prompt(affichage);
	if ( nb==null )
		return 0;
	cpt ++;
 
	if ( nb<X )
		affichage = "Le nombre indiqué est trop petit !";
	else	
		affichage = "Le nombre indiqué est trop grand !";
	}
	while ( nb!=X )
		return cpt;
}
function compteur(){
  var cpt = 0;
  var compteur = 0;
  var cpt_max = 10;
  var continuer;
  do{
    compteur = partie();
    if (compteur)
      {
        cpt ++;
        if ( compteur < cpt_max )
          continuer = confirm("Bravo tu as gagné !");
      }
    else continuer = false;
 
  }
  while ( continuer )
}

Tout semble bien fonctionner mais quand on trouve le résultat, cela ne dit pas que l'on a gagné... Qu'aurais-je oublié?

Merci

**CosmoKnacki** · 19/09/2018, 14h26

Ce test est faux (erreur de logique), je te laisse trouver pourquoi:

Code :

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

1
2
3
4
if ( nb<X )
    affichage = "Le nombre indiqué est trop petit !";
else	
    affichage = "Le nombre indiqué est trop grand !";

La variable cpt dans la fonction compteur ne sert à rien.

**SpaceFrog** · 19/09/2018, 14h31

je te donnes ce qu'il faut pour le corriger

Code :

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

{{}}

**CCACC02** · 19/09/2018, 14h49

Envoyé par CosmoKnacki

Ce test est faux (erreur de logique), je te laisse trouver pourquoi:

Code :

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

1
2
3
4
if ( nb<X )
    affichage = "Le nombre indiqué est trop petit !";
else	
    affichage = "Le nombre indiqué est trop grand !";

La variable cpt dans la fonction compteur ne sert à rien.

Si le test est faux, pourquoi fonctionne t il?
mais je peux remettre toutes les conditions si il le faut

Code :

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

1
2
3
4
5
6
7
8
if ( nb<X )
		affichage = "Le nombre indiqué est trop petit !";
	else	
		if ( nb>X )
			affichage = "Le nombre indiqué est trop grand !";
	}
	while ( nb!=X )
		return cpt;

Invité · 19/09/2018, 15h14

Bonjour,

1- if, else, while,... : mets SYSTÉMATIQUEMENT les accolades { ... }, même s'il n'y a qu'une seule instruction après.

2- INDENTE proprement ton code.

3- à aucun moment tu ne traites le cas : nb==X.

[EDIT] Je savais bien que j'avais déjà vu ça : Jeux du plus ou moins

**CCACC02** · 19/09/2018, 16h10

Je suis tout a fait d'accord avec toi @jreaux62, mais comme il n'y avait qu'une seule instruction, j'avais pris la liberté de les enlever. Je les remets mais le résultat final qui est de dire que l'on a trouvé le bon chiffre ne fonctionne pas.

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
function partie()
  {
    var nb; 
    var cpt;
    var affichage = "Donnez-moi un chiffre entre 0 et 100 ..." ;
 
 
        do{
	          nb = prompt(affichage);
	          if ( nb==null ){
		            return 0;
	              cpt ++;}
 
            if ( nb === X ){
               affichage = "Bravo tu as trouvé en " + cpt + " coup(s).";
                cpt ++;}
               else
                  if ( nb<X ){
		                affichage = "Le nombre indiqué est trop petit !";
                    cpt ++;}
	                else	
		                if ( nb>X ){
			              affichage = "Le nombre indiqué est trop grand !";
                    cpt ++;}
                     else
                       if (cpt=5){
                      affichage=" Tu as perdu";}
	         }
	      while ( cpt<=5 )
 
  }

**Watilin** · 20/09/2018, 22h37

Envoyé par jreaux62

1- if, else, while,... : mets SYSTÉMATIQUEMENT les accolades { ... }, même s'il n'y a qu'une seule instruction après.

Envoyé par CCACC02

Je suis tout a fait d'accord avec toi @jreaux62, mais comme il n'y avait qu'une seule instruction, j'avais pris la liberté de les enlever.

Retirer les accolades fait partie de ces choses qu’on a envie de faire quand on débute, tout comme écrire du code astucieux mais difficile à relire (par exemple, pour la satisfaction d’une fonction qui tient en une seule ligne). Avec l’expérience on apprend que la lisibilité du code est importante, et que ce n’est pas idiot d’écrire du code qui a l’air idiot. La clarté avant tout.

Cela étant dit, j’ajoute mes deux centimes avec quelques habitudes à prendre quand on débute :

Avoir toujours la console ouverte avec F12
Utiliser console.log
Adopter le mode strict
Se méfier des opérateurs de comparaison non stricts
etc.

**CosmoKnacki** · 19/09/2018, 16h20

Envoyé par CCACC02

Si le test est faux, pourquoi fonctionne t il?

Il ne fonctionne pas, il a juste l'air de fonctionner: Le fait que nb ne soit pas inférieur à X n'implique pas que nb soit supérieur à X, car il peut être aussi égal.

mais je peux remettre toutes les conditions si il le faut

Code :

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

1
2
3
4
5
6
7
8
if ( nb<X )
		affichage = "Le nombre indiqué est trop petit !";
	else	
		if ( nb>X )
			affichage = "Le nombre indiqué est trop grand !";
	}
	while ( nb!=X )
		return cpt;

C'est mieux, mais comme dit plus haut, le cas de l'égalité n'est pas traité et ça serait plus clair avec des accolades (ou à la limiter en plaçant juste le else et le if sur la même ligne comme pour un elseif ou elif dans d'autres langages et en indentant rigoureusement).

**noboxstyle** · 20/09/2018, 11h34

Conseil 1 : écrire des variables au nom explicite.
var X ce n'est pas explicite. Quelqu'un qui relit ton code doit tout de suite comprendre de quoi il s'agit.
Idem pour le nom des fonctions.

Conseil 2 : par convention on code en anglais

.

Conseil 3 : N'hésite pas à découper ton code en fonction claire. Une fonction == une tâche.

Conseil 4 : Si tu peux éviter de mettre des valeurs en dur qui vont en plus être réutilisés ou succeptible d'être modifiés par la suite (imagine que tu veuilles faire des niveaux dans ton jeu et qu'après le joueur puisse trouver des nombres entre 1 et 1000), n'hésite pas à utiliser des variables globales. (pour Min/Max number par exemple).

Je te donne ici un code fonctionnel pour ton problème.
Pour le compteur, je te laisse réfléchir à une solution correcte et bien implémentée pour le cas ou dans ton jeu si le count > 5 tu perds.
Suis l'architecture du code ci-dessous. Fais des fonctions, et surtout refactorise toujours.

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
 
var count = 0;
var MAX_NUMBER = 100;
var MIN_NUMBER = 1;
 
//  initialise la partie 
function StartGame() {
  var numberToFind = Math.floor(Math.random() * (MAX_NUMBER + 1));
  userNumber = prompt("Donnez-moi un chiffre entre " + MIN_NUMBER + " et " + MAX_NUMBER);
  CompareValues(userNumber);
}
 
// compare la valeur saisie par l'utilisateur et celui à trouver
function CompareValues(userNumber) {
  do {
    if (userNumber < numberToFind) {
      count++;
      prompt("Le nombre indiqué est trop petit !");
    }
    if (userNumber > numberToFind) {
      count++;
      prompt("Le nombre indiqué est trop grand !");
    }
  } while (numberToFind != userNumber);
 
  prompt("Bravo tu as gagné en " + count + " essais");
}

ps : dans ta page HTML n'oublie pas de changer ton onclick :

Code :

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

onclick="StartGame()"

**ProgElecT** · 20/09/2018, 12h35

Un bon schéma, par contre

Envoyé par noboxstyle

.....
Conseil 2 : par convention on code en anglais

.....

Tout dépend de ton entreprise, ou de ta langue natal et de à qui est destiné TON programme source, moi j'ai toujours commenté en français par défaut, et si le source était destiné à l'étranger, là, je commentai en français et en anglais.
Pour moi il n'y a pas de convention, ou du moins on peut/doit défendre sa langue.

Aucun affichage du résultat de ma fonction

JavaScript

Vue hybride

Discussions similaires

Partager

Partager