返信falseが動作しない

Question

誰かがなぜreturn falseが動作しないのか教えていただけますか？私は、電流が黄色であるかどうかをチェックしたい。それが黄色のクラスであれば何もしません（falseを返します）。問題は、ボタンをクリックしてそのボタンを再度実行するときですが、私はそれを避けたいと思います。 ここはthe Fiddle of the problemです。

/*INSIDE THIS CODE RETURN FALSE NOT WORKING!!*/ 
$('.yellowcontroll').click(function(){ 

    if($yellow.after('.current').length){ 
     $('.yellow').show(); 
     $('.slideswrapper:not(:animated)').animate({'marginLeft':'-=100'},1000,function(){ 
      $('.current').removeClass('current').hide(); 
      $('.yellow').addClass('current'); 
      $('.slideswrapper').css({'margin-left':'0px'}); 

      if($('.current').is($('.slideswrapper div:last'))){ 
       $('.blue,.green,.red').clone().insertAfter('.slideswrapper div:last'); 
      } 

      if($('.current').is($('.red'))){ 
       $('.red').prevAll().remove(':not(.yellow)'); 
       $('.yellow').insertAfter($('.slideswrapper div:last')); 
      } 

      /*THIS IS NOT WORKING AS EXPECTED!!*/  
      if($('.current').is($('.yellow'))){ 
       return false; 
      } 
     }); 
    } 

}); 

Answer 1

問題は、イベントコールバックではなく、コールバックからアニメーションにfalseを返すことです。

あなたが探していると、あなたは、あなたが条件を移動し、クリックコールバックの前にfalseを返すことができます秒の時間をクリックしたときに発生する何のためにある場合：

$('.yellowcontroll').click(function(){ 

    /* MOVE THIS TO THE BEGINNING OF THE CLICK CALLBACK */  
    if($('.current').is($('.yellow'))){ 
     return false; 
    } 

    if($yellow.after('.current').length){ 
     $('.yellow').show(); 
     $('.slideswrapper:not(:animated)').animate({'marginLeft':'-=100'},1000,function(){ 
      $('.current').removeClass('current').hide(); 
      $('.yellow').addClass('current'); 
      $('.slideswrapper').css({'margin-left':'0px'}); 

      if($('.current').is($('.slideswrapper div:last'))){ 
       $('.blue,.green,.red').clone().insertAfter('.slideswrapper div:last'); 
      } 

      if($('.current').is($('.red'))){ 
       $('.red').prevAll().remove(':not(.yellow)'); 
       $('.yellow').insertAfter($('.slideswrapper div:last')); 
      } 
     }); 
    } 

}); 

Answer 2

あなたのフィドルのコードあなたの質問に基づいて、あなたは間違った場所にあなたのロジックを持っているようです。あなたのクリックイベントの一番上にreturn falseロジックを配置してみてください。

$('.yellowcontroll').click(function(){ 
    if($('.current').is($('.yellow'))){ 
     return false; 
    } 
    ... 
    }); 

This fiddleは、あなたが欲しいものを行う必要があります。

http://jsfiddle.net/mihaifm/3YLEg/2/

$('.yellowcontroll').click(function(){ 
      /*THIS IS NOT WORKING AS EXPECTED!!*/  
      if($('.current').is($('.yellow'))){ 
       return false; 
       } 

Answer 3

は、私はあなたのクリックハンドラの先頭でそのコードスニペットを入れたいと思います以下のように。

DEMO

if ($('.current').hasClass('yellow')) { 
     return false; 
    } 

Answer 4

移動以下とhasClassを使用するなどのクリックハンドラの先頭に条件付きの偽コード：

Answer 5

if($('.current').is('.yellow')){ 
    return false; 
}