Verhindern von "zu viel Rekursion" Fehler in jQuery

Question

EDIT **

Ich habe diese Click-Ereignis

$('.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(); 

    } 

}); 

Wenn Debug in der Konsole sagt es too much recursion bedeutet, dass es in einer Art Endlosschleife An diesem Punkt. Ich denke es ist wegen der trigger("click") Veranstaltung in der if Aussage, weil ich etwas ähnliches online gesehen habe.

Grundsätzlich möchte ich sagen, wenn gegebene Wort die Klasse hat right-word dann bewegen (daher der Auslöser), sonst ...

Gibt es eine andere Möglichkeit, es zu schreiben, das wird nicht abstürzen? Hier

ist eine Geige: http://jsfiddle.net/Dxxmh/112/

BETRIEBS: Klicken Sie auf die Buchstaben auf der rechten Seite den markierten Bereich im Raster zu buchstabieren (die Bilder, um Ihnen buchstabieren die Wörter in einer Geige nicht verfügbar sind, so müssen Sie buchstabieren sie mit der Konsole, durch die TDs aufzublicken)

Answer 1

Ich würde so etwas tun:

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

Ich sehe wie if(score.right == 4) bedeutet das Ende des Spiels. Nachdem es beendet ist - Sie haben überhaupt keine Wörter (oder haben einfach keine "richtigen" Wörter, nicht sicher) und deshalb hört es nie auf. Es löst nur den Klick für immer aus, statt alles zu stoppen und darauf zu warten, dass der Benutzer auf die Schaltfläche "Neustart" klickt.

Ich denke, diese Bedingung ist nicht genug. Nicht sicher, wie viele falsche Wörter gezählt und behandelt werden. Aber es sollte reichen, um vorwärts zu gehen und den richtigen Zustand basierend auf Ihrer Programmlogik aufzubauen. Jede Rekursion, die Sie starten (und Sie starten sie mit einem Trigger ("click")), muss eine Stoppbedingung haben.

Answer 2

Versuchen sie, diese zu nutzen:

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

Answer 3

.trigger('click') wird nur den Hörer rufen noch einmal. Wolltest du nur unter diesen Umständen dem Link folgen? In diesem Fall könnten Sie return false in Ihrem else Szenario. Diese

Answer 4

ist kein jQuery Problem: Sie Auslösung manuell das gleiche Ereignis innerhalb der Handler:

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

Nun, dies wird zu einer Endlosschleife führen, wenn man nicht aufpasst. Der beste Weg, dies zu beheben ist nicht die Prozedur aufrufen, indem Sie das Ereignis ein zweites Mal Auslösung, sondern durch Aufruf einer Funktion Namen:

$('.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 
     } 
});