2016-11-06 84 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"));