問題與調用方法

Question

我正在做一個戰艦遊戲。我想要放置3個長度爲3個瓷磚的小船，而不要重疊。我已經寫以下代碼：

var boatGrid = { 

    placeBoat : function() { 
     for (boatNum = 1; boatNum < 4; boatNum++) { 
      console.log("boat placed now"); 
      this.selectPos(); 
      document.getElementById("boattest").innerHTML = boatPos; 
      if (document.getElementById(boatPos).hasBoat == 1) { 
       this.placeBoat(); 
       document.getElementById("boattest").innerHTML = "failed"; 
      } 
      else { 
       this.buildBoat();   
      } 
     } 
    }, 

    selectPos : function() { 
      xPos = Math.floor(Math.random() * 8 + 1); 
      yPos = Math.floor(Math.random() * 10 + 1); 
      boatPos = "cell_" + xPos + "_" + yPos; 
    }, 

    buildBoat : function() { 
     for (boatLen = 1; boatLen < 4; boatLen++) { 
      boatPos = "cell_" + xPos + "_" + yPos; 
      xPos = xPos + 1; 
      document.getElementById(boatPos).hasBoat = 1; 
      document.getElementById(boatPos).style.backgroundColor = "brown"; 
      console.log("placed one tile"); 
     } 

    }, 

    clearTable : function() { 
     for (y = 1; y < 11; y++) { 
      for (x = 1; x < 11; x++) { 
       boatPos = "cell_" + x + "_" + y; 
       document.getElementById(boatPos).hasBoat = 0; 
       document.getElementById(boatPos).boatHere = 0; 
       document.getElementById(boatPos).style.backgroundColor = "#34B0D9" 
       document.getElementById(boatPos).innerHTML = "<pre>  </pre>"; 
       boatGrid.hasChecked = 0; 
      } 
     } 
    }, 
} 

當選擇兩次（可變boatPos存儲座標）的座標，該函數placeBoat（）應該記得本身。但是，實際發生的情況是buildBoat（）函數被調用兩次。你們中的任何人都能看到問題嗎？我目前認爲使用placeBoat（）函數不得不從內部回想自己是一個問題，但我不知道如何解決這個問題。所有幫助讚賞。

Answer 1

我將存儲boatPos作爲內部boatGrid全局變量，像：

var boatGrid = { 
    boatPos: 0, 

    placeBoat : function() { 
     for (boatNum = 1; boatNum < 4; boatNum++) { 
      //same code as before 
      if (this.boatPos == 1) { 
       this.placeBoat(); 
      // same code as before 
     } 
    }, 

    buildBoat : function() { 
     for (boatLen = 1; boatLen < 4; boatLen++) { 
      //same code as before 
      this.boatPos = 1; 
      // same code as before 
     } 
    }, 
} 

如果你需要存儲更多的屬性在boatPos（因爲它似乎），使用對象：

boatPos = { 
    hasBoat: 0, 
}, 

然後，在功能：

if (this.boatPos.hasBoat == 1) 

和

this.boatPos.hasBoat = 1;