为什么我的 if-else 块没有被命中，尽管它应该被命中？ （只需要另一双眼睛。）

Question

我正在用 Java 制作一款落沙风格的游戏，但我的 if-else 块遇到了奇怪的问题。在我的 doGravity() 方法中，我有各种条件块，它们会导致不同的事情发生，并且由于某种奇怪的原因，一个块永远不会被击中。

当我用这个块计算每个条件被命中的次数时，左侧和右侧的块几乎均匀地被命中：

else if(world[x][y+1]==EMPTY && (x-1 >= 0) && world[x-1][y+1] == EMPTY && (x+1 < world.length) && world[x+1][y+1]==EMPTY) {
    int r = rand.nextInt(50);
    if(r == 0) {
        world[x-1][y+1] = world[x][y];
        //System.out.println("GO: right");
        countRight++;
    }
    else if(r == 1) {
        world[x+1][y+1] = world[x][y];
        //System.out.println("GO: left");
        countLeft++;
    }
    else {
        world[x][y+1] = world[x][y];
        countCenter++;
    }
    world[x][y] = EMPTY;
}

接下来是这个条件，它也均匀地分布在左侧和右侧。

else if((x-1 >= 0) && world[x-1][y+1] == EMPTY && (x+1 < world.length) && world[x+1][y+1]==EMPTY) {
    if(rand.nextBoolean()) {
        world[x-1][y+1] = world[x][y];
        //countLeft++;
    }
    else {
        world[x+1][y+1] = world[x][y];
        //countRight++;
    }
    world[x][y] = EMPTY;
}

但是当我计算这些块时，左侧的块永远不会被击中，即使左侧的空间打开。我觉得这可能只是一些愚蠢的错字，由于某种原因我看不到。

else if((x-1 >= 0) && world[x-1][y+1] == EMPTY) {
    world[x-1][y+1] = world[x][y];
    world[x][y] = EMPTY;
    countLeft++;
    System.out.println("Hit Left");
}
else if((x+1 < world.length) && world[x+1][y+1] == EMPTY) {
    world[x+1][y+1] = world[x][y];
    world[x][y] = EMPTY;
    countRight++;
    System.out.println("Hit Right");
}

更新：如果我在末尾注释掉left块，则绝对不会发生任何变化。沙子的作用完全相同。如果我在末尾注释掉 right 块，则其作用与注释掉两个块相同。我无法弄清楚这一点。它应该有效...但事实并非如此。

更新：这是完整的源代码。我不知道这可能是什么。事实上，它会让我发疯。 http://pastebin.com/mXCbCvmb

Answer 1

您的pastebin代码确实显示“Hit left”，您只需将世界的创建（第65行pastebin）更改为

    world = new Color[worldWidth][worldHeight+1];

因为我假设的y+1部分。除此之外，它向左和向右生长。

编辑： http://pastebin.com/GVmSzN4z 我用你的 doGravity 稍微调整了一下，让水滴更多一点对称。

Answer 2

我在发布的代码中没有看到任何奇怪的东西......但是第二个块开头的“else”让我认为可能在您希望由“left”处理的情况下执行上述条件案件。
该部分之前的 if 中的条件是什么？

编辑

在检查了完整的源代码后，我终于找到了问题所在。你的 doGravity 更新函数总是从左到右，这就引入了不对称性。通过改变它使得奇数/偶数扫描线的更新方向在左→右和右→左之间交替，不对称性消失。

private void doGravity() {
    for(int i = worldHeight - 1; i >= 0; i--) {
        if (i % 2 == 0)
        {
            for(int j = 0; j < worldWidth; j++) {
                if(world[j][i] != EMPTY) {
                    if(hasGravity(world[j][i])) {
                        dropParticle(j, i);
                    }
                }
            }
        } 
        else
        {
            for(int j = worldWidth-1; j >= 0; --j) {
                if(world[j][i] != EMPTY) {
                    if(hasGravity(world[j][i])) {
                        dropParticle(j, i);
                    }
                }
            }
        }
    }
}

Answer 3

我从粘贴箱下载了你的代码，我做的第一件事就是提取这个方法并使用它而不是所有嵌入的数组单元格检查，这样我就可以设置一个断点并查看 x 和 < code>y 以及该索引单元格的内容是什么。

private boolean isEmpty(final int x, final int y)
{
    return world[x][y] == EMPTY;
}

我会将所有 EMPTY 检查提取为更具可读性的内容，例如 isLeftEmpty(x,y) 和 isRightEmpty(x,y) 和 < code>isNextLeftEmpty(x,y) 它将帮助您推断代码中逻辑的正确性。

我还将 (x + 1 < world.length) 提取到 isNextXOutsideWorld(x)，这将有助于记录您的意图并帮助推理您想要的逻辑以及。

这还有一个副作用，即简化 if/elseif/else 语句中的逻辑。

我进行了一些简短的调试，并让它运行了几分钟，得出的结论是以下行始终匹配并取代下一个 else if 语句。

else if ((x + 1 < world.length) && isEmpty(x + 1, y + 1) && 
         (x - 1 >= 0) && isEmpty(x - 1,y + 1))

当我运行它时总是正确的，所以它永远不会到达下一个语句，

else if ((x - 1 >= 0) && isEmpty(x - 1,y + 1))

我会尝试将每个 else/if 语句分解为具有描述性名称的方法调用，并且将所有这些语句按顺序使用策略 模式，因为它们都是互斥的。这么大的方法肯定是有代码味道的，再加上那些else/if块，臭味系数很高。

很难从 if/elseif/else 块中的所有噪音推断出您的预期行为。