尝试创建基本的数学游戏

时间:2015-02-24 11:55:05

标签: javascript html math

我正在做一个数学游戏。我有这个HTML / JavaScript代码。它会生成一个随机数,但是当我输入正确答案时,它仍会显示错误的'。我不确定出了什么问题。

这是HTML代码:

<table width="400" border="1" align="center">
  <tr>
    <td><div id="number1">1</div></td>
    <td><div>+</div></td>
    <td><div id="number2">2</div></td>
    <td><div>=</div></td>
    <td><input type="text"></input></td>
    <td><input type="button" value="Check"></input></td>

  </tr>
</table>

这是我的JavaScript代码:

//random number appear when start game
var number1;
var number2;

number1 = Math.floor((Math.random() * 10) + 1);
number2 = Math.floor((Math.random() * 10) + 1);
document.getElementById("number1").innerHTML=number1;
document.getElementById("number2").innerHTML=number2;

//Answer
var answer = number1 + number2;

//add click handler with check answer
var checkAnswer = document.querySelector('input[type=text]');
var value = checkAnswer.value;
var btn = document.querySelector('input[type=button][value=Check]');


btn.onclick = function() 
{
  if (value == answer) 
       {
          alert('You are correct');
       }
   else{
          alert('You are incorrect, the answer was ' + answer);
       }

 document.querySelector('input[type=text]').value = "";               
 document.getElementById('number1').innerHTML = "";
 document.getElementById('number2').innerHTML = ""; 
 number1 = Math.floor((Math.random() * 10) + 1);
 number2 = Math.floor((Math.random() * 10) + 1);
 document.getElementById('number1').innerHTML = number1; 
 document.getElementById('number2').innerHTML = number2;

 answer = number1 + number2


};

3 个答案:

答案 0 :(得分:2)

.onclick方法中移动以下行。

var checkAnswer = document.querySelector('input[type=text]'); //needn't be moved in method
var value = checkAnswer.value;

您可以从JSFiddle验证结果。

问题是当你加载页面时变量value用空字符串初始化,就像加载页面时值为空。

答案 1 :(得分:1)

您不会更新value

在按钮单击事件中更新值。 JSFiddle

btn.onclick = function() 
{
    value = checkAnswer.value;

    ...

答案 2 :(得分:1)

将答案与checkAnswer中的值进行比较:

if (checkAnswer.value == answer) { 
  //correct 
}else{
  //incorrect
}