Programme Guess The Number [Résolu]

iamtheone 39 Messages postés lundi 10 avril 2017Date d'inscription 8 mai 2017 Dernière intervention - 19 avril 2017 à 12:37 - Dernière réponse : iamtheone 39 Messages postés lundi 10 avril 2017Date d'inscription 8 mai 2017 Dernière intervention
- 19 avril 2017 à 15:10
Bonjour,
Je suis en train d'apprendre le javascript et j'ai essayé de faire un programme "guess the number" classique. Mais celui-ci ne marche pas... Et même avec les "indications" de la console je ne trouve pas les erreurs.

Voici mon code html :
<!DOCTYPE html>
<html>
    <head>
        <title>Guess the number</title>
    </head>
    <body onLoad="reset()">
        <h2>Guess the number HTML/CSS/JS</h2>
            <p><a href="">Cliquez ici</a> pour découvrir la manière la plus rapide de deviner un nombre compris entre x et y.</p>
        <hr />
            <input type="submit" value="Actualiser" onClick="reset()">
            <p>Vous devez deviner le nombre z compris entre <span id="x"></span> et <span id="y"></span>.</p>
            <p id="so" align="center" color="red"></p>
            <form name="form">
                <input type="number" name="number" value="0"><br /><br />
                <input type="submit" name="submit" value="Soumettre"  onClick="submit()"><br />
            </form>
        <p>Votre nombre d'essais : <span id="trys"></span></p><br /><br /><br /><br /><br /><br />
    </body>
</html>


Et voici mon code javascript :
window.onload = function() {

function reset() {

    x = Math.floor(Math.random() * (-101));
    document.getElementById("x").innerHTML =     "'+x+'";
    y = Math.floor(Math.random() * 101);
    document.getElementById("y").innerHTML = "'+y+'";
    z = Math.random() * (y - x) + x;
    trys = 0;
    document.getElementById("trys").innerHTML = "'+trys+'";

}

function submit() {

    var ztry =     eval(document.form.number.value);
    trys++;
    var so = document.getElementById("so");

    if (ztry > z) {
        so.innerHTML = "c'est moins";
    } elseif (ztry < z) {
        so.innerHTML = "c'est plus";
    } elseif (ztry == z) {
        so.innerHTML = "c'est gagné !!!";
    } else {
        so.innerHTML = "Merci d'entrer un nombre valide.";
        trys--;
    }

}

}


Je tiens à préciser que j'utilise "sololearn" comme compiler.
Merci de me révéler les erreurs que vous voyez. ;)

Afficher la suite 
39Messages postés lundi 10 avril 2017Date d'inscription 8 mai 2017 Dernière intervention

8 réponses

Répondre au sujet
jordane45 19213 Messages postés mercredi 22 octobre 2003Date d'inscriptionModérateurStatut 12 décembre 2017 Dernière intervention - 19 avril 2017 à 15:01
+1
Utile
1
Voici le code corrigé
<!DOCTYPE html>
<html>
  <head>
    <meta charset="UTF-8">
    <title>Guess the number</title>
  </head>
  <body onLoad="reset();">
 
    <h2>Guess the number HTML/CSS/JS</h2>
    <p><a href="">Cliquez ici</a> pour découvrir la manière la plus rapide de deviner un nombre compris entre x et y.</p>
    <hr />
    <input type="button" value="Actualiser" onClick="reset()">
    <p>Vous devez deviner le nombre z compris entre <span id="x"></span> et <span id="y"></span>.</p>
    <p id="so" align="center" color="red"></p>
    <input type="number" id="number" value="0"> 
    <input type="button" name="submit" value="Soumettre" onClick="verifier();"><br />
    <p>Votre nombre d'essais : <span id="trys"></span></p><br /><br /><br /><br /><br /><br />

    <script type="text/javascript">
      //declaration des variables "globales" en dehors de toute fonction.
       var x;
       var y;
       var z = 0;
       var trys;
      function reset() {
        x = Math.floor(Math.random() * (0));
        document.getElementById("x").innerHTML =  x;
        y = Math.floor(Math.random() * 101);
        document.getElementById("y").innerHTML = y;
        z = parseInt(Math.random() * (y - x) + x);
        trys = 0;
        document.getElementById("trys").innerHTML = trys;
      }

      function verifier() {
        var reponse = "";
        var ztry = parseInt(document.getElementById('number').value);
        trys = parseInt(document.getElementById('trys').innerHTML);
        trys++;

        console.log( "z : " + z + " -> ztry: " + ztry);       
        if (ztry > z) {
           reponse = "c'est moins";
        } else if (ztry < z) {
          reponse = "c'est plus";
        } else if (ztry == z) {
          reponse = "c'est gagné !!!";
        } else {
          reponse = "Merci d'entrer un nombre valide.";
          trys--;
        }
        document.getElementById("so").innerHTML = reponse;
        document.getElementById("trys").innerHTML = trys;
      }
   </script>    
  </body>
</html>



Il manquait plein de choses ... comme par exemple la déclaration des variables "globales" x y z ...
Cette réponse vous a-t-elle aidé ?  
iamtheone 39 Messages postés lundi 10 avril 2017Date d'inscription 8 mai 2017 Dernière intervention - 19 avril 2017 à 15:10
Un grand merci, ça marche très bien. Je vais regarder comment tu as fais pour ne plus faire les mêmes erreurs.
Commenter la réponse de jordane45
jordane45 19213 Messages postés mercredi 22 octobre 2003Date d'inscriptionModérateurStatut 12 décembre 2017 Dernière intervention - 19 avril 2017 à 12:43
0
Utile
2
Bonjour,

Remplace
  <input type="number" name="number" value="0">
 <input type="submit" name="submit"  value="Soumettre"  onClick="submit()"> 

par :
 <input type="number" id="number" value="0">
 <input type="button" name="submit"  value="Soumettre"  onClick="submit()">


et dans le js
 var ztry =     eval(document.form.number.value);


remplace par :
 var ztry =     parseInt(document.getElementById('number').value);



iamtheone 39 Messages postés lundi 10 avril 2017Date d'inscription 8 mai 2017 Dernière intervention - 19 avril 2017 à 13:32
Merci de ta réponse mais le code ne marche toujours pas.
Voici les erreurs :
Uncaught SyntaxError: unexcpected {
Line : 23
Uncaught ReferenceError: reset is not defined
Line : 10
(et quand je clic sur le bouton soumettre :)
Uncaught TypeError: submit is not a function
Line : 14
jordane45 19213 Messages postés mercredi 22 octobre 2003Date d'inscriptionModérateurStatut 12 décembre 2017 Dernière intervention > iamtheone 39 Messages postés lundi 10 avril 2017Date d'inscription 8 mai 2017 Dernière intervention - 19 avril 2017 à 14:09
tu as donc des erreurs dans ton code ... des accolades mal fermées entre autre ....
Pour pouvoir t'aider il nous faudrait le code COMPLET de ta page..... (tel qu'écrit dans ton (tes ?) fichiers.
Commenter la réponse de jordane45
iamtheone 39 Messages postés lundi 10 avril 2017Date d'inscription 8 mai 2017 Dernière intervention - 19 avril 2017 à 14:14
0
Utile
2
C'est le code COMPLET de ma page. Je rappele que j'utilise sololearn.
jordane45 19213 Messages postés mercredi 22 octobre 2003Date d'inscriptionModérateurStatut 12 décembre 2017 Dernière intervention - 19 avril 2017 à 14:20
oui... mais tu l'a placé dans deux balises de code différentes ....
Je voudrais voir le code html contenant la parti javascript ... TEL QUE TU L'AS ECRIT dans ton fichier html.
Et puis... avec les modifications que je t'ai donné .... car pour l'instant je n'ai que ton code d'origine ......
iamtheone 39 Messages postés lundi 10 avril 2017Date d'inscription 8 mai 2017 Dernière intervention > jordane45 19213 Messages postés mercredi 22 octobre 2003Date d'inscriptionModérateurStatut 12 décembre 2017 Dernière intervention - 19 avril 2017 à 14:41
ça ?

<!DOCTYPE html>
<html>
    <head>
        <title>Guess the number</title>
        <script>
        
function reset() {

    x = Math.floor(Math.random() * (-101));
    document.getElementById("x").innerHTML =     "'+x+'";
    y = Math.floor(Math.random() * 101);
    document.getElementById("y").innerHTML = "'+y+'";
    z = Math.random() * (y - x) + x;
    trys = 0;
    document.getElementById("trys").innerHTML = "'+trys+'";

}

function submit() {

    var ztry = parseInt(document.getElementById('number').value);
    trys++;
    var so = document.getElementById("so");

    if (ztry > z) {
        so.innerHTML = "c'est moins";
    } elseif (ztry < z) {
        so.innerHTML = "c'est plus";
    } elseif (ztry == z) {
        so.innerHTML = "c'est gagné !!!";
    } else {
        so.innerHTML = "Merci d'entrer un nombre valide.";
        trys--;
    }

}

        </script>
    </head>
    <body onLoad="reset()">
    
        <h2>Guess the number HTML/CSS/JS</h2>
            <p><a href="">Cliquez ici</a> pour découvrir la manière la plus rapide de deviner un nombre compris entre x et y.</p>
        <hr />
            <input type="submit" value="Actualiser" onClick="reset()">
            <p>Vous devez deviner le nombre z compris entre <span id="x"></span> et <span id="y"></span>.</p>
            <p id="so" align="center" color="red"></p>
            <form name="form">
              <input type="number" id="number" value="0"> 
              <input type="button" name="submit" value="Soumettre" onClick="submit()"><br />
            </form>
        <p>Votre nombre d'essais : <span id="trys"></span></p><br /><br /><br /><br /><br /><br />
        
    </body>
</html>
Commenter la réponse de iamtheone