ページスクロールで要素が近くに来たらカウントアップさせたい

Question

ページ内にある複数の要素を、カウントアップして指定の数値に来たらストップさせるものを作ったのですが、
ページスクロールで要素が近くに来たらカウントアップをスタートさせるように書き換えたところ、
カウントアップしている数字の表示がおかしくなり、
一瞬カウントダウンしてまたカウントアップしたりするようになってしまいました。

正常にカウントアップさせるにはどのようにしたらいいのか、ご教授お願いします。

<div class="count count-slow" data-num="80">00</div>
<div class="count count-fast" data-num="20">00</div>
<div class="count count-slow" data-num="99">00</div>
<div class="count count-fast" data-num="8">0</div>
<div class="count count-slow" data-num="5">0</div>
<div class="count count-fast" data-num="40">00</div>

$(window).on('load scroll', function(){
   $('.count').each(function(){
        var self = $(this),
            thisPosition = self.offset().top,
            scroll = $(window).scrollTop(),
            windowHeight = $(window).height(),
            countMax = self.attr('data-num'),
            thisCount = self.text(),
            countTimer;


        if (scroll >= thisPosition - windowHeight + windowHeight / 5){
          if (self.hasClass('count-slow')) {
            var countSpeed = 600;
          } else if (self.hasClass('count-fast')) {
            var countSpeed = 30;
          }

          function timer(){
            countTimer = setInterval(function(){
                var countNext = thisCount++,
                    addZero = ('0' + countNext).slice(-2);

                if (countMax > 10 && countNext < 10) {
                  self.text(addZero);
                } else {
                  self.text(countNext);
                }

                if (countNext == countMax){
                  clearInterval(countTimer);
                }
            }, countSpeed);
          }
          timer();
        }
    });
});

Answer 1

質問文にあるコードでは、ページ読込直後にのみ、カウントアップを行うかどうかの判定処理を行っています。これでは、画面をスクロールしても正常に動作しません。

ページスクロールにより要素が近くに来たときに処理を行いたいのであれば、ページのスクロールを検出する必要があります。jQueryには、スクロールが行われたときの処理を登録できるscroll()というメソッドがあるので、これを使うと良いでしょう。

scroll()は、以下のように使います。

$(window).scroll(function(){
  //画面内でスクロール位置が変化したときに呼び出される
});

今回の場合、scroll()を用いた以下のコードでやりたいことが実現できます。

//$count要素のカウントアップを開始する
//$count: jQueryオブジェクト
function launchCountUp($count){

  var countMax = $count.attr('data-num');
  var thisCount = $count.text();
  var countTimer;

  if ($count.hasClass('count-slow')){
    var countSpeed = 600;
  }else if($count.hasClass('count-fast')){
    var countSpeed = 30;
  }

  function timer(){
    countTimer = setInterval(function(){
      var countNext = thisCount++;
      var addZero = ('0' + countNext).slice(-2);

      if (countMax > 10 && countNext < 10) {
        $count.text(addZero);
      } else {
        $count.text(countNext);
      }

      if (countNext == countMax){
        clearInterval(countTimer);
      }
    }, countSpeed);
  }

  timer();
}

//要素が画面にあるか確認し、あればカウントアップ処理を開始する
//$counts: jQueryオブジェクト
function detectCountElement($counts){

  $.each($counts, function(i, countElement){
    var $count = $(countElement);

    //既にカウントアップが行われている場合、何もしない
    if($count.attr("data-isLaunched")){
      return;
    }

    //スクロール量・画面の高さを取得する
    var $window = $(window);
    var scrollTop = $window.scrollTop();
    var windowHeight = $window.innerHeight();

    //要素の座標を取得する
    var offset = $count.offset();

    //要素が画面内にあるか確認する
    if(scrollTop < offset.top && offset.top < scrollTop + windowHeight){
      //要素のカウントアップを開始する
      launchCountUp($count);
      //要素にデータを付与し、カウントアップが行われたことが分かるようにする
      $count.attr("data-isLaunched", true);
    }

  });

}


$(function(){

  //<div class="count">要素を取得する
  var $counts = $(".count");

  //ページ読込直後に、画面内に<div class="count">があれば、
  //それらのカウントアップを開始する
  detectCountElement($counts);

  //スクロール後に、画面内に<div class="count">があれば、
  //それらのカウントアップを開始する
  $(window).scroll(detectCountElement.bind(window, $counts));

});

Comments

[melRel]

回答ありがとうございます！
質問に書いたjQueryの部分のスクロールの記述が抜けておりました。失礼いたしました。

回答いただいた内容で書き換えたところ正常にカウントアップするようになりました。
それぞれの処理のところに詳しくコメントも書いてくださっていて、
とても勉強になりました。ありがとうございました！

[退会済みユーザー]

当初提示していたコードは、イベントの内部部分だったのですね... 更新していただいたコードが正常に動かない理由は、スクロールイベントが起きるたびに、timer()が呼び出されているためと思われます。元々のコードでも、カウントアップが開始しているか判定し、開始されていた場合はtimer()を実行しないようにすれば、動くようになると思います。