我的 For 循环计时不正确，怎么了？

Question

我的 For 循环有问题，因为某些原因它没有正确计时。我让它每 5 秒点击一次我的“外星人”（您将在代码中看到），并且每次购买工人时它都会增加。因此，如果您有 2 个工人，则每 5 秒有 2 个外星人，每 5 秒有 3 个外星人，依此类推。但是，当我有 2 个或更多工人时，就像我想为每个工人点击外星人两次一样。

我想要的： 1 个 worker = 每 5 秒自动点击一次 // 2 个工人 = 每 5 秒自动点击两次 等等

它错误地做了什么： 1 名工人按我的意愿工作// 2名工人=每5秒自动点击两次，每次点击价值2 等等

这是运行 for 循环的变量：

var workerCheck = setInterval(function(){
        var workers = parseInt($("#Workers").text());
            if( workers > 0 ){
                for(var i = 0; i < workers; i++){
                $("#Alien").click();       
                }
            }
    },5000);

这是我整个代码的 pastebin 链接：http://pastebin.com/ApwQgG4w

希望我的问题有意义，例如，如果我有 2 个工人，它将每 5 秒将“外星人”值设置为 4，而我只想要 2，对于 3，我希望每 5 秒有 3 个外星人，而不是 9，等等。

Answer 1

您可能需要在重置之前清除间隔。

当您单击“雇用工人”两次时，会创建两个计时器。这是demonstration。无论按钮被点击多少次，您都只需要一个计时器。

当您调用setTimer() 时，它会返回一个唯一的整数，您可以稍后使用它来引用该计时器。由于您总是将相同的变量workerCheck 设置（覆盖）为计时器的唯一整数，因此workerCheck 变量只能引用最新的计时器。这是demonstration。

我建议你每次重置计时器时清除它：

$(function () {
    $('#Purchase').click(function () {
        automaticCounter();
    });

    // set variable with broader scope
    var workerCheck;

    function automaticCounter() {
        var workers = parseInt($("#Workers").text());
        workers = isNaN(workers) ? 0 : ++workers;

        $("#Workers").text(workers);

        var autoCounter = parseInt($("#Aliens").text());
        autoCounter = isNaN(autoCounter) ? 0 : autoCounter - 10 * workers;

        $("#Aliens").text(autoCounter);

        // clear the timer
        clearInterval(workerCheck);

        // set the timer
        workerCheck = setInterval(function () {
            var workers = parseInt($("#Workers").text());
            if (workers > 0) {
                for (var i = 0; i < workers; i++) {
                    $("#Alien").click();
                }
            }
        }, 5000);
    }
});

WORKING EXAMPLE

Answer 2

希望我理解正确。

这是一个小而酷的小提琴：FIDDLE

您的代码已编辑并且现在可以使用：FIDDLE

var workers = 0;
var workerCheck = setInterval(function(){
    workers = parseInt($("#workers").val());
        if( workers > 0 ){
            for(var i = 0; i < workers; i++){
            $("#Alien").trigger('click');       
            }
        }
},1000);

你的代码有什么问题：我不知道它有点乱，我编辑了它，它现在可以工作了。