Où est la faute ?

Fermé
xam4lor Messages postés 17 Date d'inscription jeudi 10 juillet 2014 Statut Membre Dernière intervention 7 août 2015 - Modifié par KX le 30/11/2014 à 13:03
 Utilisateur anonyme - 9 janv. 2015 à 07:46
Bonjour, est-ce que quelqu'un pourrait me dire où se trouve la faute svp ?
Voici le code html:


<html>

    <head>
 
     <script type="text/javascript" src="Java Script.js"> </script>
     <link rel="stylesheet" href="css.css" />
    
    </head>
 
 
    <body>
  
  <form name="form2">
  
      <center><u style = "color: red"><b><h1 style = "color: red">I. Le nombre mystere</h1></b></u></center>
   <br>
   <u style = "color: #0101DF"><h2 style = "color: #0101DF">Objectif: </h2><br></u>
   <p>Trouver le nombre mystere, il se situe entre 1 et le nombre que vous avez choisi precedement</p>
   <p name = "nbutilisateur2" value = ""></p>
   <br><br>
   
      <p>Entrez votre nombre ici: </p>
            <input type="text" name="input" value=""><p>    </p>
            <input type="button" name="bouton" value="Appuyez sur ce bouton pour valider votre choix" onClick="afficher(form2)"> <br>
  
  </form>
  
    </body>

</html>

Et voici le code Javascript

//Nombre maximum choisi par le joueur
var nbmaxi =  prompt("Quel est le nombre maximum que vous devrez chercher ?","Entrez-le ici");

nbmaxi();


//Le nombre choisi par l'ordinateur (se situe entre 1 et le nombre maximum choisi par l'utilisateur)

var nbordi = Math.floor(Math.random() * nbmaxi + 1);

function afficher(form2) { 

    var nbutilisateur = parseInt(document. form2.input.value); 
  
  //Si le nombre choisi par l'utilisateur n'est pas un nombre, cette fonction va s'executer
    if(isNaN(nbutilisateur)) {
        alert("Le texte que vous avez entré n'est pas un nombre");
    } 
 
  //Sinon, commencer les tests
  
    else {
   
    //si le nombre de l'utilisateur est supérieur au nombre de l'ordi, faire ça
    if(nbutilisateur > nbordi) {
       alert("Le nombre que vous avez sélectionné est trop grand");
    }
    
    else if(nbutilisateur < nbordi) {
       alert("Le nombre que vous avez entré est trop petit");
    }
    
    else(nbutilisateur = nbordi) {
       alert("Vous avez gagné");
    }
 }
}

1 réponse

Utilisateur anonyme
9 janv. 2015 à 07:46
Bonjour

La ligne nbmaxi() est une erreur, puisque nbmaxi n'est pas une fonction.

else(nbutilisateur = nbordi)
est incorrect pour deux raisons : tu pourrais mettre un "if" derrière un else comme tu l'as fait avant , mais comme c'est ton dernier cas, il ne faut ni if ni condition, puisque le else signifie justement "dans les autres cas".
Et s'il y avait une condition, ce serait nbutilisateur == nbordi, avec deux = car avec un seul, tu vas recopier la valeur de nbordi dans nbutilisateur, tu ne vas pas tester l'égalité.

Tu sais qu'il y a une console javascript dans les navigateurs, pour t'indiquer les erreurs ? Sur le mien, c'est en appuyant sur F12. Ça ne dit pas tout, mais ça aide bien.
0