防止「太多遞歸」中的jQuery錯誤

Question

編輯**

我有這樣的單擊事件

$('.next-question').click(function() { 

    $('td').removeClass('highlight-problem'); 
    var r = rndWord; 
    while (r == rndWord) { 
     rndWord = Math.floor(Math.random() * (listOfWords.length)); 
    } 
    $('td[data-word="' + listOfWords[rndWord].name + '"]').addClass('highlight-problem'); 
    $('td[data-word=' + word + ']').removeClass('wrong-letter').removeClass('wrong-word').removeClass('right-letter'); 
    var spellSpace = $('td[data-word=' + listOfWords[rndWord].name + ']').hasClass('right-word'); 
    if (spellSpace) { 

     $('.next-question').trigger('click'); 

    } else { 

     $("#hintSound").attr('src', listOfWords[rndWord].audio); 
     hintSound.play(); 
     $("#hintPic").attr('src', listOfWords[rndWord].pic); 
     $('#hintPic').show(); 
     $('#hintPicTitle').attr('title', listOfWords[rndWord].hint); 
     $('#hintPicTitle').show(); 

    } 

}); 

在調試中它說too much recursion這意味着它是在某種無限循環的控制檯在此刻。我認爲這是因爲if聲明中的trigger("click")事件，因爲我在網上看到類似的東西。

基本上，我想說，如果給定的詞有類right-word然後繼續前進（因此觸發），否則......

有另一種方式來寫它不會崩潰？

這裏是一個小提琴：http://jsfiddle.net/Dxxmh/112/

說明：點擊右側的字母拼寫在網格中的高亮區域（的圖像，以幫助你拼寫的話是不可用的小提琴，所以你必須拼他們使用控制檯，通過查找TD的）

Answer 1

我會做這樣的事情：

if (spellSpace) { 
      if(score.right != 4) 
       $('.next-question').trigger('click'); 

我看到像if(score.right == 4)意味着比賽結束。結束後 - 你根本沒有任何詞語（或者根本沒有「正確」的詞語，不確定），這就是爲什麼它永不停止的原因。它只是觸發點擊永遠，而不是停止做任何事情，並等待用戶點擊重新啓動按鈕。

我想這種情況是不夠的。不知道如何計算和處理錯誤的單詞數量。但它應該足以推進並根據您的程序邏輯建立正確的條件。您開始的任何遞歸（並且您使用觸發器（「click」）開始）必須具有停止條件。

Answer 2

嘗試使用此：

$('.next-question').click(function (event) { 
    event.preventDefault(); 
}); 

Answer 3

.trigger('click')只會調用監聽一次。您是否打算僅在該情況下遵循鏈接？在這種情況下，您可以在else方案中使用return false。

Answer 4

這不是一個jQuery的問題：你手動從處理程序中觸發同樣的事件：

$('.next-question').trigger('click'); 

現在，如果你不小心，這將導致無限循環。解決這個問題最好的辦法是不，但通過使用函數名調用它通過觸發該事件的第二次調用處理程序：

$('.next-question').click(function callMe(event) 
{ 
     //replace: $('.next-question').trigger('click'); 
     //with this: 
     if (spellSpace && event) 
     {//also check if the event has been passed 
      callMe.apply(this,[]);//don't pass event for recursive call 
     } 
});