Question

我正在使用iio引擎在javascript中进行BattleShip游戏。

我正在尝试与电脑对战，所以我必须为船只设置一个随机位置（我希望你知道游戏:)）。

我有5艘必须放在网格中的船只（10x10）。问题是该函数非常慢，有时页面根本没有加载。

我想知道这些功能的速度是否有所改善，我有点新手：D

function posShips(size){
    // var size -> size of the ship
    var isOk = false; // flag var to check if the ship is in a right position
    var isOk2 = true; // flag var, become false if the cell is already fill with another ship
    var i; 
    var j;
    var side; // horizontal or vertical
    while(!isOk){
        i = iio.getRandomInt(1,11);
        j = iio.getRandomInt(1,11);
        side = iio.getRandomInt(0,2);
        if((side ? j : i)+size-1 < 11){ // Not out of the array
            for (var k = 0;  k < size; k++) { // Size of the ship
                if(side){
                    if(gridHit[i][j+k].stat == "empty"){ //If is empty put the ship
                        gridHit[i][j+k].stat = "ship";
                        gridHit[i][j+k].setFillStyle("red")
                    }else{ // If not empty
                        isOk2 = false; //Position is not good, do all the thing again.
                        for (var a = 0;  a < size; a++) { // Reset cell
                            gridHit[i][j+a].stat = "empty";
                        }
                        k = 10;
                    }
                }else{
                    if(gridHit[i+k][j].stat == "empty"){ //If is empty put the ship
                        gridHit[i+k][j].stat = "ship";
                        gridHit[i+k][j].setFillStyle("red")
                    }else{ // If not empty
                        isOk2 = false; //Position is not good, do all the thing again.
                        for (var a = 0;  a < size; a++) { // Reset cell
                            gridHit[i+a][j].stat = "empty";
                        }
                        k = 10;
                    }
                }
            };
            if(isOk2)
                isOk = true;
        }
    }
}

Answer 1

不要选择将落在网格外的船位。首先选择方向，然后根据x限制y和size初始位置。例如如果大小为3，则变化坐标的初始值不会高于7。
在您搜索时不要更改阵列。首先进行搜索，然后才更新数组。这样可以避免任何＆＃34;清理＆＃34;操作
尽可能消除重复的深层对象引用。如果针对不同的grid[y][x]重复访问x，请先引用grid[y]，然后将其用于后续访问。
提前摆脱循环，如果前一个已经失败，那么就没有必要继续测试一个位置。
将你的大船放在第一位 - 让小船更容易进入大船之间的空隙。

请参阅http://jsfiddle.net/alnitak/Rp9Ke/了解我的实现，相当于您的函数：

this.place = function(size) {

    // faster array access
    var g = this.grid;

    // initial direction, and vector
    var dir = rand(2);  // 0 - y, 1 - x
    var dx = dir ? 1 : 0;
    var dy = dir ? 0 : 1;  // or 1 - dx

    LOOP: while (true) {
        // initial position
        var x = dir ? rand(10 - size) : rand(10);
        var y = dir ? rand(10) : rand(10 - size);

        // test points
        var n = size, tx = x, ty = y;
        while (n--) {
            if (g[ty][tx]) continue LOOP;
            tx += dx;
            ty += dy;
        }

        // fill points
        n = size;
        while (n--) {
            g[y][x] = size;
            x += dx;
            y += dy;
        }

        break;
    }
};

提高功能速度

1 个答案: