Question

我正在尝试将三个随机数分配给三个变量（random1，random2，random3），然后将这些随机变量分配给三个元素。但我不希望它们中的任何一个等于变量Sum，这是两个数字innerHTML值的加法。

所以我使用do...while循环完成了这项工作，但不幸的是do...while循环无法按预期工作。

这是我的代码：

setTimeout(function () {
    z.innerHTML = Math.floor((Math.random() * 3) + 1);

    setTimeout(function applySUM() {
        var Sum = parseInt(document.getElementById('fir').innerHTML) +
            parseInt(document.getElementById('sec').innerHTML);
        ch1.innerHTML = Sum;
    }, 500);

    do {
        var random1 = Math.floor((Math.random() * 3) + 1);
        var random2 = Math.floor(Math.random() * (6 - 4 + 1)) + 4;
        var random3 = Math.floor(Math.random() * (10 - 7 + 1)) + 7;
    } while (random1 == Sum || random2 == Sum || random3 == Sum);

    setTimeout(function func() {
        ch2.innerHTML = random1;
    }, 1000);

    setTimeout(function func() {
        ch3.innerHTML = random2;
    }, 1500);

    setTimeout(function func() {
        ch4.innerHTML = random3;
    }, 2000);

}, 2000);

查看上面的代码，ch2.innerHTML，ch3.innerHTML和ch4.innerHTML似乎不可能等于Sum，但当我测试它时，现实说些别的话。这是为什么？

Answer 1

首先，正如很多人提到的那样，sum变量是ApplySum的本地变量，所以你的代码的其余部分引用了一个全局的Sum变量（默认情况下它是“未定义的”）

另一个问题是，现在你的do-while循环立即运行，没有等待500毫秒超时，并且在Sum被分配给一个值之前。您可以通过将代码放在settimeout回调函数中来解决此问题：

z.innerHTML = Math.floor((Math.random() * 3) + 1);

setTimeout(function applySUM() {
    var Sum = parseInt(document.getElementById('fir').innerHTML) +
        parseInt(document.getElementById('sec').innerHTML);
    ch1.innerHTML = Sum;


    do {
        var random1 = Math.floor((Math.random() * 3) + 1);
        var random2 = Math.floor(Math.random() * (6 - 4 + 1)) + 4;
        var random3 = Math.floor(Math.random() * (10 - 7 + 1)) + 7;
    } while (random1 == Sum || random2 == Sum || random3 == Sum);

    setTimeout(function func() {
        ch2.innerHTML = random1;
    }, 500);

    setTimeout(function func() {
        ch3.innerHTML = random2;
    }, 1000);

    setTimeout(function func() {
       ch4.innerHTML = random3;
    }, 1500);

}, 500);

（我还从其他settimeouts减少了500ms，以补偿它们在第一次超时内被移动）

您可以考虑的另一个微小变化是为每个变量执行单独的循环，而不是为所有变量执行单个循环。

var random1, random2, random3;
do { random1 = Math.floor((Math.random() * 3) + 1);          } while (random1 == Sum);
do { random2 = Math.floor(Math.random() * (6 - 4 + 1)) + 4;  } while (random2 == Sum);
do { random3 = Math.floor(Math.random() * (10 - 7 + 1)) + 7; } while (random3 == Sum);

Answer 2

关于范围的评论看起来像是在正确的轨道上。这是代码的相关部分：

setTimeout(function applySUM() {
    var Sum = parseInt(document.getElementById('fir').innerHTML) +
        parseInt(document.getElementById('sec').innerHTML);
    ch1.innerHTML = Sum;
}, 500);
// Outside of your applySum function, Sum has no meaning

do {
    var random1 = Math.floor((Math.random() * 3) + 1);
    var random2 = Math.floor(Math.random() * (6 - 4 + 1)) + 4;
    var random3 = Math.floor(Math.random() * (10 - 7 + 1)) + 7;
} while (random1 == Sum || random2 == Sum || random3 == Sum);
// Outside of your loop body, random1, random2, and random3 have no meaning
// undefined == undefined => true

也许你把它改成了这个：

var Sum = 0;
setTimeout(function applySUM() {
    Sum = parseInt(document.getElementById('fir').innerHTML) +
        parseInt(document.getElementById('sec').innerHTML);
    ch1.innerHTML = Sum;
}, 500);

var random1 = random2 = random3 = undefined;
do {
    random1 = Math.floor((Math.random() * 3) + 1);
    random2 = Math.floor(Math.random() * (6 - 4 + 1)) + 4;
    random3 = Math.floor(Math.random() * (10 - 7 + 1)) + 7;
} while (random1 == Sum || random2 == Sum || random3 == Sum);

然后你的变量可能在适当的位置有范围。只是预感，这可能还有其他问题。

三重条件做while循环

2 个答案: