可変条件が満たされている場合のみjQuery onclick

Question

クリックしたときに、「好き」のクラスを持つdivの数が0より大きい場合、そのコードを実行する必要があります。問題がありますボタンはまだコードを実行しません。私は同じ結果に.lengthと.size（）の両方を試みたことに注意してください。

$(document).ready(function() { 
countlikes = $('[id^=post_].like').length; 
likestatus = 1; 
$('#show_likes').on('click', function() { 
    if (countlikes >0) { 
     likestatus++; 

     $('[id^=post_].like').toggle(); 

     if (likestatus % 2 == 0) { 
      $('#hidelikedbtn').removeClass('hidelikedimgoff').addClass('hidelikedimgon'); 
     } else { 
      $('#hidelikedbtn').removeClass('hidelikedimgon').addClass('hidelikedimgoff'); 
     } 

    return false; 

    } else { 
     return false; 
    } 
}); 
}); 

Answer 1

現在のカウントではなく元のcountlikesの値を使用してください。ハンドラの中で代入を動かすと、それはうまくいくはずです。 （変数が実際に必要な瞬間に1つの場所で実際に使用されているだけですが、if文で直接$('[id^=post_].like').lengthをテストすることができます）。

また、「1より大きい」とコードでは "> 0"と表示されるため、それが機能していても、あなたが記述したことはしません。

また、最後else場合は、冗長問題とは無関係です：ifあなたはfalseを返し、それ以外にあなたがfalseを返すので、なぜかの外return false;を移動し、それ以外は削除しないの終わりに？

Answer 2

あなたの 'countlike'変数の前にvarが必要です。

また、他の変数の前に同じことが必要です。クラスは、ページのロード後に変化する「ように」との要素数が、問題はあなたが現在そのたびにハンドラが、それはなり走る、一度だけcountlikesに外あなたのクリックハンドラを値を割り当てることであると仮定すると