2016-11-06 58 views
-1

快速的問題,爲什麼這段代碼不工作。我有一種感覺,它可能與可變範圍有關,但我無法確定哪裏出了問題。我可以得到要打印到控制檯的div id,但是我得到一個錯誤,說明id attr在語句中是未定義的。JavaScript if statement scope

var game = { 

    currentPlayerTurn: players.firstPlayer.token, 

    displayToken: function() { 
     $('.unit').click(function() { 
      if(game.currentPlayerTurn === 'X' && !$(this).hasClass('selected')) { 
       $(this).addClass('selected').removeClass('unit').text("X"); 
       game.currentPlayerTurn = players.secondPlayer.token; 
      } else if(game.currentPlayerTurn === 'O' && !$(this).hasClass('selected')) { 
       $(this).addClass('selected').removeClass('unit').text("O"); 
       game.currentPlayerTurn = players.firstPlayer.token; 
      } 
      game.win($(this)); 
     }) 
    }, 

    win: function(div) { 
     game.winCombos.forEach(function(element) { 
      element.forEach(function(smell){ 
       if(smell === div.attr('id')) { 
        smell = game.currentPlayerTurn; 
       } 
      })}) 
     console.log(div.attr('id')); 
    }, 



    winCombos: [[1, 2, 3], [4, 5, 6], [7, 8, 9], [1, 4, 7], [2, 5, 8], [3, 6, 9], [1, 5, 9], [3, 5, 7]] 


}; 

game.win(); 
+1

這似乎可能性不大,'div'說法應該是可用?這是整個代碼 – adeneo

+1

是一個至少有一個匹配元素的jquery對象嗎? –

+0

@adeneo - 我認爲div.attr('id')的結果是「undefined」 - 如果div是一個空的jquery結果,或者div jquery object中的第一個元素沒有標識,則會是這種情況 –

回答

0

game.win();沒有被頁面元素調用。你需要給它一些東西。

例子:

<div id="game"></div> 

game.win($("#game"));