jQuery si hasclass alors la fonction

C'est une animation qui ne devrait se déclencher si le div rip_tab a la classe 'rip_tab_ripped', qui est appliqué après le div est cliqué. Cependant, l'animation se déclenche avant même la rip_tab_ripped classe a été activée. Chaque fonction séparément, sans la clause if. Toute aide serait appréciée--

var sauceSquirt = {
    init: function() {

        $("#rip_tab").click(function() {
            $(this).toggleClass("rip_tab_ripped");
        });



        function fireA() {
            $("#sauceRed").switchClass("sauce_hide", "sauceRedGo", 500)
        }

        function fireB() {
            $("#sauceBlue").switchClass("sauce_hide", "sauceBlueGo", 500)
        }

        if ($('#rip_tab').hasClass("rip_tab_ripped")) {


            $('#packet').click(function() {

                var events = [fireA, fireB];

                //declare counter
                if (!this.counter) {
                    this.counter = 0;
                }

                events[this.counter]();
                this.counter = (this.counter + 1) % 3;
            });



        }

    }

}

$(document).ready(function() {
    sauceSquirt.init();

});​

OriginalL'auteur Commandrea | 2012-07-26

  1. 8

    Dirait que vous rencontrez des problèmes avec cette partie:

    if ($('#rip_tab').hasClass("rip_tab_ripped")) {
    $('#packet').click(function() {

       var events = [fireA, fireB];

       //declare counter
       if(!this.counter) { this.counter = 0; }

       events[this.counter]();
       this.counter = (this.counter + 1) % 3;
    });
}

    Pouvez-vous modifier:

    $('#packet').click(function() {
    if ($('#rip_tab').hasClass("rip_tab_ripped")) {

           var events = [fireA, fireB];

           //declare counter
           if(!this.counter) { this.counter = 0; }

           events[this.counter]();
           this.counter = (this.counter + 1) % 3;
    }
    return false;
});

    Vous pouvez aussi jeter un oeil à jQuery Promesse

    OriginalL'auteur sabithpocker

  2. 0

    Résolution est de déplacer le si ($ état de la ligne ci-dessous cliquez sur le relevé de compte.

    OriginalL'auteur Commandrea

  3. 0

    Quel est le paquet?
    À mon avis, il y a cette déclaration doit être écrit comme ceci:

    this.counter = this.counter % 2;

    OriginalL'auteur Prince

  4. 0

    Rappelez-vous que tout dans cette méthode est appelée comme vous l'initialiser, alors que vous voulez retarder la vérification de classe jusqu'à ce que le #packet est cliqué:

    $('#packet').click(function() {
    if ( $('#rip_tab').is('.rip_tab_ripped') ) {
        var events = [fireA, fireB];

        //declare counter
        this.counter = this.counter || 0; //set to 0 if value is null

        events[this.counter]();
        this.counter = ++this.counter % 2; //You only have two methods
    }
});

    OriginalL'auteur nbrooks