Question

到目前为止，我正在努力制作船上放置船只的功能，我在检查功能方面遇到了一些问题。我的基本想法是有一个方法可以在按钮点击时调用：

$("#dodaj1x1").click(function(){
        var raspolozivo=parseInt($("#raspolozivo1x1").text());
        if(raspolozivo>0){

            generisi1x1();//call for function that generate random field

            var novoRaspolozivo= raspolozivo-1;
            $("#raspolozivo1x1").html(novoRaspolozivo);
        }
        else{
            alert("Rasporedjeni svi raspolozivi brodovi ovog tipa!");
        }
    });

它会调用函数来生成随机字段：

function generisi1x1(){
            var minR = 0;
            var maxR = 9;
            var minK = 0;
            var maxK = 9;
            randRed=Math.floor(Math.random() * (maxR - minR + 1)) + minR;
            randKol=Math.floor(Math.random() * (maxK - minK + 1)) + minK;
            proveri1x1(randRed,randKol);//call to function to check is field available
    }

比函数generisi1x1（）调用函数检查该字段是否可用：

function proveri1x1(randRed,randKol){
        for(i=randRed-1;i<randRed+2;i++){
            for(j=randKol-1;j<randKol+2;j++){
                if($(".red"+i+".kolona"+j+"").hasClass('deoBroda')){
                    alert("red:"+" "+i+" kolona:"+j);
                    generisi1x1();
                }
                else { postavi1x1(randRed,randKol);}
            }
        }
    }

我的问题是，有时候这项工作很棒（至少看起来效果很好，也许是纯粹的运气），有时它只生成3艘船1x1（应该有4艘），有时它会向我展示有关问题的信息并生成5艘船（右边4个，坏1个）等。

不良案例的打印屏幕：Added ship 1x1 on position 5,3 right next to ship 4x1

以下是整个代码的实时演示：Live demo

到目前为止，我已经可以插入4x1和1x1的船只，只检查1x1，计划对所有船只都做同样的事情，任何帮助都会很棒。

Answer 1

如果proveri1x1()执行检查并返回true或false，generisi1x1()执行postavi1x1()操作，您会发现更容易理解

function generisi1x1() {
    var minR = 0, maxR = 9, minK = 0, maxK = 9;
    randRed = Math.floor(Math.random() * (maxR - minR + 1)) + minR;
    randKol = Math.floor(Math.random() * (maxK - minK + 1)) + minK;
    if(proveri1x1(randRed, randKol)) { //call to function to check is field available
        postavi1x1(randRed,randKol);//set
    } else {
        generisi1x1();//try again
    }
}

function proveri1x1(randRed, randKol) {
    for(var i=randRed-1; i<randRed+2; i++) {
        for(var j=randKol-1; j<randKol+2; j++) {
            if($(".red" + i + ".kolona" + j).hasClass('deoBroda')) {
                return false;
            }
        }
    }
    return true;//<<<< note the position of this return statement
}

我正在努力创造＆＃34;沉没船只＆＃34;使用jQuery

1 个答案: