Javascript-重复的if语句不起作用

Question

我的代码有效（不会失败），但它做的事情不正确。

每当我想生成一个新的“块”时，我都会调用“生成”函数，每次调用该函数时都会将一个新的数字传递给该函数。 它可以生成块，但不会生成我想要的块。

如果上一代是空间，我希望它生成一个空间，一个跳跃，一个滑动或一个间隙。 但是，如果上一代不是一个空间，请生成一个空间。 它不会那样做。 有时彼此之间会产生2个间隙，2个跳跃或2个滑行，我不知道为什么... ???

这是我的代码：

var ptg = 'space'; // what was previously generated to top
var wtgt; // what is currently being generated to top
var chunktogenerateto = 0;

function generate(a){
    chunktogenerateto = a;

    var rand1 = Math.floor(Math.random()*100) + 1;

    if(ptg == 'space' && rand1 <= 25){
        wtgt = 'space';
    }
    else if(ptg == 'space' && rand1 <= 50 && rand1 > 25){
        wtgt = 'jump';
    }
    else if(ptg == 'space' && rand1 <= 75 && rand1 > 50){
        wtgt = 'slide';
    }
    else if(ptg == 'space' && rand1 > 75){
        wtgt = 'gap';
    }
    else{
        wtgt = 'space';
    }

    ptg = wtgt;

    topGen(wtgt);
}

function topGen(g){
    document.getElementById('t' + chunktogenerateto).setAttribute('src','images/terrain/t' + g + '.png');
}

我希望这不是错字...帮助！

“生成”的调用来自哪里：

var chunkpos = new Array();
chunkpos[0] = -100;
chunkpos[1] = 0;
chunkpos[2] = 100;
chunkpos[3] = 200;
chunkpos[4] = 300;
chunkpos[5] = 400;
chunkpos[6] = 500;
chunkpos[7] = 600;
chunkpos[8] = 700;
chunkpos[9] = 800;
chunkpos[10] = 900;
chunkpos[11] = 1000;
var temppos = new Array();
var time1;
var millis1;
var time2;
var millis2;
var millis3;
var firstreset = true;
var pos;
var poschange;

function moveLevel(){
    if(firstreset == true){
        resetTime();
    }
    var time2 = new Date();
    var millis2 = time2.getTime();
    var millis3 = millis2 - millis1;
    poschange = Math.floor(millis3 / 5);
    for(i = 0; i < chunkpos.length; i++){
        temppos[i] = chunkpos[i] - poschange;
        if(temppos[i] <= -150){
            generate(i);
            temppos[i] += 1200;
        }
        pos = temppos[i];
        document.getElementById('chunk' + i).setAttribute('style','left: ' + pos + 'px;');
    }
}



function resetTime(){
    time1 = new Date();
    millis1 = time1.getTime();
    if(firstreset != true){
        for(i = 0; i < chunkpos.length; i++){
            chunkpos[i] = temppos[i];
        }
    }
    firstreset = false;
    setTimeout('resetTime()',1000);
}

调用“ moveLevel”的位置来自：

window.onload = function(){
    if(test = 'runnable')
    {
        gameLoop();
    }
    else
    {
        document.getElementById('gm').innerHTML = (gm + 'Failed to run game.');
    }
}

function gameLoop(){
    if(currentscreen == 'playing'){
        moveLevel();
    }
    setTimeout('gameLoop()',0);
}

这是一个包含所有代码的zip文件的下载链接： ParkourFreak.zip我遇到麻烦的代码在scripts / generation.js下。 游戏主页面（可见生成的页面）是index.html。

Answer 1

而不是每个以“ if（ptg =='space'... etc”开头的if语句-在一个简单的“ if not space return space”中首先执行此操作。

之后，您可以随机开始-应该适合您。

Answer 2

由于以下代码，大约有四分之一的时间您以前有一个空间，它将创建另一个空间：

if(ptg == 'space' && rand1 <= 25){
    wtgt = 'space';
}

请注意，它是在先前创建空间时创建空间的。

---

边注：

此行无需&& rand1 > 25 ：

else if(ptg == 'space' && rand1 <= 50 && rand1 > 25){

由于您以前使用过if(ptg == 'space' && rand1 <= 25){ ，所以从ptg == 'space' ， rand1不能<= 25 。

（在以下情况下， && rand2 > 50同样。）

Answer 3

您确定要增加chunktogenerateto吗？

当前代码似乎保持为0，是在您未显示的细分中吗？

编辑：哦，这是函数的参数。 尝试记录该变量，也许是问题所在？

Answer 4

// elsewhere
var wtgts = ['space','jump','slide','gap'];

// in the function
if(ptg !== 'space') {
    wtgt = 'space';
} else {
    var randidx = Math.floor(Math.random()*4);
    wtgt = wtgts[randidx];
}

甚至在函数中：

var newidx = (ptg == 'space') ? Math.floor(Math.random()*4) : 0;
wtgt = wtgts[newidx];

1. 权重相等，因此只需从0到3中选择一个随机索引，然后使用该索引从数组中选择一个动作即可。
2. 分开将ptg作为空间的逻辑，以便只测试一次
3. 考虑将动作表示为数字而不是字符串，因为赋值，比较等将更快，更可靠。

Answer 5

我认为问题在于一代人，而实际上我要做的就是改变这一点：

if(temppos[i] <= -100){
    generate(i);
    temppos[i] += 1200;
}

对此：

if(temppos[i] <= -100){
    generate(i);
    chunkpos[i] += 1200;
    temppos[i] += 1200;
}