Question

我正在用JavaScript编写一个简单的游戏，当用户将鼠标悬停在给定坐标上时，其得分会增加。我的问题是分数在1之后不会增加。

var x = Math.floor(Math.random() * 500) + 150;
var y = Math.floor(Math.random() * 500) + 150;
document.getElementById("demo2").innerHTML = y;
document.getElementById("demo1").innerHTML = x;

 //live tracking of coordinates
$("div.frame").mousemove(function (event) {
    var xco = event.clientX;
    var yco = event.clientY;
    var xcoor = "X co-ords: " + xco;
    var ycoor = "Y co-ords: " + yco;

    document.getElementById("display1").innerHTML = xcoor;
    document.getElementById("display2").innerHTML = ycoor;

    //keeping score 
    if (xco == x && yco == y) {
        count++;
        console.log(count);
        document.getElementById("score").innerHTML = count;
        generate();
    }

    //Generating Co-ordinates     
    function generate() {
        var x = Math.floor(Math.random() * 500) + 150;
        var y = Math.floor(Math.random() * 500) + 150;

        document.getElementById("demo2").innerHTML = y;
        document.getElementById("demo1").innerHTML = x;

        function points(x, y) {
            if (xco == x && yco == y) {
                count++;
                console.log(count);
                document.getElementById("score").innerHTML = count;
                generate();
            }
        }

        points(x, y);
    }
})

Answer 1

我认为这是因为您的count变量是局部变量，并且仅在if块中存在（如果您希望在其他位置计数），请声明为全局变量

var x = Math.floor(Math.random() * 500) + 150;
var y = Math.floor(Math.random() * 500) + 150;
document.getElementById("demo2").innerHTML = y;
document.getElementById("demo1").innerHTML = x;

var count = 0;

 //live tracking of coordinates
$("div.frame").mousemove(function (event) {
    ...
})

Answer 2

即使添加了全局count变量之后，由于if (xco == x && yco == y)条件在首次运行后也会失败，因此您的代码也无法正常工作。在这里，您将随机生成的x和y与xco，yco进行比较，并更新分数并再次调用generate。

在generate函数上，您正在创建新的局部变量x和y，而不是更新全局x和y。并且随后的比较(xco == x && yco == y)将失败，因为它仍使用旧的x和y。

在您的generate函数上，更新全局x和y应该可以解决此问题，

x = Math.floor(Math.random() * 500) + 150;
y = Math.floor(Math.random() * 500) + 150;

这是完整的代码，我还做了一些小的更改，删除了一些多余和重复的代码

// Code goes here
    $(document).ready(function(){
      
    //on clicking let's play
    var t = 120;    
    var count = 0;    
    
    $("button.play").click(function(event){
        event.preventDefault();
        $(".fadeout").fadeOut(2000, function(){
            $(".heading").text("Click on the Start button when you're ready");
        });
        $(".hidden").attr("hidden", false);
        });
    function start(){
      
      //init game timer
    var Timer = setInterval(function(){
    document.getElementById("time1").innerHTML = t;
        --t;
    if(t < 0){
        clearInterval(Timer);
        document.getElementById("timeout").innerHTML = "Time's Up |";
        $("div.frame").attr("hidden",true);
    }
    },1000);

    var x = Math.floor(Math.random()*500)+150;
    var y = Math.floor(Math.random()*500)+150;
    document.getElementById("demo2").innerHTML = y;
    document.getElementById("demo1").innerHTML = x;
        
    //live tracking of coordinates
     $("div.frame").mousemove(function(event){
        var xco = event.clientX;
        var yco = event.clientY;
        var xcoor = "X co-ords: " + xco;
        var ycoor =  "Y co-ords: " + yco;
        document.getElementById("display1").innerHTML = xcoor;
        document.getElementById("display2").innerHTML = ycoor;
        //keeping score 
        points(xco, yco);
     })
    $("div.frame").mouseout(function(event){
       document.getElementById("display1").innerHTML = " ";
       document.getElementById("display2").innerHTML = " "; 
    })
    //Generating Co-ordinates     
    function generate(){    
      x = Math.floor(Math.random()*500) + 150;
      y = Math.floor(Math.random()*500) + 150;
      document.getElementById("demo2").innerHTML = y;
      document.getElementById("demo1").innerHTML = x;
    }
    function points(xco, yco){
      if(x == xco && y == yco){     
        count++;
        console.log(count);
        document.getElementById("score").innerHTML = count;
        generate();
     }  
    }
    
    }
     $("button.start").click(function(event){
        event.preventDefault();
         start();
         $(this).fadeOut(1500);
         $(this).attr("disabled",true);
         $(".afterstart").fadeOut(1500);
        });
 
    });

/* Styles go here */
   body{
        height: 1200px;
        background-size: cover;
    }
    .jumbotron{
        margin-top: 250px;
        padding: 20px;
        height: 350px;
        background-size: cover;
        background-attachment: fixed;
    }
    .heading{
       font-size: 30px;
        margin-top: 100px;
    }
    .frame{
        height: 1000px;
        width: 1500px;
        border: 10px solid black;
    }
    ul {
    background: #40475A;
    padding: 10px;
    width: 100%;
    }
    .stats{
    font-size: 30px;
    margin-top: 80px;
    }
    .info{
     font-size: 30px;
        color:aliceblue;
    }
.bold{
    font-size: 25px;
    color:antiquewhite;
}

<!DOCTYPE html>
<html lang="en">
<head>
    <meta charset="UTF-8">
  <link rel="stylesheet" href="https://stackpath.bootstrapcdn.com/bootstrap/4.1.3/css/bootstrap.min.css" integrity="sha384-MCw98/SFnGE8fJT3GXwEOngsV7Zt27NXFoaoApmYm81iuXoPkFOJwJ8ERdknLPMO" crossorigin="anonymous">
    <title>Irritate</title>
</head>
 
<body>
    <div>
        <ul class="nav nav-pills pl-5 fixed-top d-flex justify-content-center" style="font-size: 25px">
 
        <li class="nav-item">
      <a class="nav-link" data-toggle="pill" id="display1"></a>
        </li>
 
        <li class="nav-item">
      <a class="nav-link" id="display2"></a>
        </li>
 
        </ul>
 
    </div>

 
   <div class="jumbotron fadeout">
       <div class="display-3 text-muted d-flex justify-content-center"> Hand-Job </div>
        <p class="info"> All you have to do is try to match the co-ordinates that will display on the screen </p>
        <p class="bold"><kbd>PS:</kbd>This game will test your patience</p>
         <form>
          <div class="input-group w-75">
 
                <div class="input-group-prepend">
                       <span class="input-group-text">Name:</span>
                </div>
 
                <input type="text" class="form-control" value="">
 
                <div class="input-group-append">
                           <button class="btn btn-outline-success play">Let's Play</button>
                </div>
          </div>
       </form>
   </div>
   <div class="hidden" hidden="true">
   <div>
       <p class="text-danger heading  afterstart"></p>
   <button class="btn btn-danger w-25 start">Start</button>
    <div  class="d-flex d-flex-inline stats">
 
    <p class="ml-3">X Co-ordinate:<span id="demo1" class="ml-2"></span><span> | </span></p>
    <p class="ml-3">Y Co-ordinate:<span id="demo2" class="ml-2"></span><span> | </span></p>
 
    <p class="ml-3">Time Left:<span id="time1" class="ml-2"></span> <span> | </span></p>
    <p id="timeout" class="ml-3"></p>
    <p>Score: <span id="score"></span></p>    
    </div>
   </div>
   <div class="frame ml-2">
   </div>
   <p id="result"></p>
   </div>
  <script src="https://code.jquery.com/jquery-3.3.1.min.js"></script>
<script src="https://cdnjs.cloudflare.com/ajax/libs/popper.js/1.14.3/umd/popper.min.js" integrity="sha384-ZMP7rVo3mIykV+2+9J3UJ46jBk0WLaUAdn689aCwoqbBJiSnjAK/l8WvCWPIPm49" crossorigin="anonymous"></script>
<script src="https://stackpath.bootstrapcdn.com/bootstrap/4.1.3/js/bootstrap.min.js" integrity="sha384-ChfqqxuZUCnJSK3+MXmPNIyE6ZbWh2IMqE241rYiqJxyMiZ6OW/JmZQ5stwEULTy" crossorigin="anonymous"></script>
</body>
 
</html>

此处是上述代码https://plnkr.co/edit/7ia7MkMimzlsAxTCWAc9?p=preview的首字母缩写网址

分数不增加

2 个答案: