スライドショーが動かない原因が知りたいです。

Question

   <div class="slider">
      <div class="slideSet">
        <div class="slide"><img src="%E5%90%8D%E7%A7%B0%E6%9C%AA%E8%A8%AD%E5%AE%9A-1.jpg"></div>
        <div class="slide"><img src="8.jpg"></div>
        <div class="slide"><img src="7.png"></div>

      </div>


      <div class="prevBtn">pre</div>
      <div class="nextBtn">next</div>
    </div>

.slider {
  position: relative;
  overflow: hidden;
  width: 500px;
  height: 500px;
  margin: 0 auto
}

.slideSet {
  position: absolute;
}

.slide {
  display: flex;
  float: left;
  width: 500px;
  height: 500px;
  background: #333
}

.prevBtn {
  position: absolute;
  top: 50%;
  left: -1%;
  border-radius: 50%;
  width: 30px;
  height: 30px;
  background: #fff
}

.nextBtn {
  position: absolute;
  top: 50%;
  right: -1%;
  border-radius: 50%;
  width: 30px;
  height: 30px;
  background: #fff
}

    'use strict';
    $(function() {

      var dir = -1;
      var interval = 1000;
      var duration = 600;
      var timer;

      $('.slideSet').prepend($('.slide:last-child'));
      $('.slideSet').css(
        'left',
        -500);

      timer = setInterval(slideTimer, interval);

      function slideTimer() {
        if (dir == -1) {
          $('.sildeSet').animate({
              'left': '-=500px'
            },
            duration,
            function() {
              $(this).append($('.slide:first-child'));
              $(this).css('left', '-500');
            });
        } else {
          $('.sildeSet').animate({
            'left': '+=500px'
          }, duration, function() {
            $(this).prepend($('.slide:last-child'));
            $(this).css('left', '-500');

            dir = -1;

          });

        }
      }

      $('.prevBtn').on('click', function() {
        dir = 1;
        clearInterval(timer);
        timer = setInterval(slideTimer, interval);
        slideTimer();
      });
      $('.nextBtn').on('click', function() {
        dir = -1;
        clearInterval(timer);
        timer = setInterval(slideTimer, interval);
        slideTimer();
      });

コンソールでエラーチェックを見ても何も表示されていないので原因がわかりません。

Comments

[m.ts10806]

自作のものですよね。でしたら「動かない」では他者にはわかりません。
どういう思想や仕様で組んだのか、ゴールも明示した上で、起きている現象もご提示ください。
あと、どの処理が動いてるかデバッグしてみましたか？

Answer 1

意図した動きにならなかった原因は、以下の３点かと思います。

• $('.slideSet').css('left', -500); ➡　**'-500px'**とする.
• $(this).css('left', '-500'); ➡　**'-500px'**とする.
• $(''.sildeSet').animate(．．． ➡　**'.slideSet'**とする.タイプミス（iとl逆）

質問者さんの考えた作りであれば、else文は必要ないかと思いますが、どうなんでしょう。
表示が終わった要素は子要素の末尾に回してるので、if内の処理だけで実現できているかと思いますが。

私が気になったのは、
既に表示されている画像はアニメーションで画面外に消えていくことが目視できるが、画面内に入ってくる画像はアニメーション完了後にしか表示されません。
アニメーション開始時点でレンダリングされていない要素はアニメーション中に画面内に現れても、アニメーション終了後にしかレンダリングされないみたいです。
おそらく、質問者さんのスライドショーのイメージは画像が連結している状態で画像が切り替わる動きかとは思いますが。
この現象の改善方法はわかりませんでした。

以下のコードで画像が順番にループ表示されると思います。
クリックイベントは割愛します。

'use strict';
$(function() {

	var dir = -1;
	var interval = 1000;
	var duration = 600;
	var timer;

	$('.slideSet').prepend($('.slide:last-child')); //3番目の画像を先頭へ[初期状態：3 1 2]
	$('.slideSet').css('left', '-500px'); //中央（2番目）の画像が初期表示

	timer = setInterval(slideTimer, interval); //スライドショースタート

	function slideTimer() {
		if (dir == -1) {
			$('.slideSet').animate({'left': '-=500px'}, duration, function(){//画像を左へ1つずらす
				//アニメーション完了後の処理
				$(this).append($('.slide:first-child')); //先頭画像を末尾へ移動
				$(this).css('left', '-500px'); //位置修正.アニメーション完了時のleftは-1000pxとなっているので、初期位置の-500pxに戻す.
  		});
		}
	}

});

Comments

[KazuSaka]

あ、クリックイベントでどっち周りか決めてるのか！else文も必要ですね。

[退会済みユーザー]

わかりました。ありがとうございます。
あとひとつお聞きしたいのですが、
.animate({'left': '-=500px'}と.animate({'left': '-500px'だとスライドの仕方が異なり前者だと左にずれながら動くのですが、後者だと一瞬で画像が切り替わってしまいます。
なぜそうなるのか-=500pxと-500pxでの違いはどういう仕組みなのでしょうか。

[KazuSaka]

前者：移動前のLEFT値からさらに-500pxした位置までアニメーションで動く.
後者：移動前のLEFT値から-500pxまでアニメーションで動く.

今回の移動前のLEFT値は-500pxなので、
前者の場合、-500px➡-1000pxまで動きます.
後者の場合、-500px➡-500pxまで動きます（実質動いてない）

後者の場合、動いてないのに画像が切り替わるのはなぜか．．．
animate()の完了イベントで
$(this).append($('.slide:first-child'));
を実施しているから.

Answer 2

修正しないといけない箇所は何点かありますが、とりあえず・・・。

・animate内でプロパティ（今回はleft）を指定する場合は''で囲まないようにする。
・animate内でプロパティ（今回はleft）の値を入れるときは単位（px）は不要です。
・animateのdurationは速度を設定するものです。durationを入れても動かないと思います。
→こちら変数の「duration」を設定していました。失礼しました。

animateの設定方法や処理内容等をリファレンス等で見てみると良いと思います。
animate リファレンス

   'use strict';
    $(function() {

      var dir = -1;
      var interval = 3000;
      var duration = 600;
      var timer;

      $('.slideSet').prepend($('.slide:last-child'));
      $('.slideSet').css('left', -500);

      timer = setInterval(slideTimer, interval);

      function slideTimer() {
        if (dir === -1) {
          $('.slideSet').animate({
              left: '-=500'
            }, 200);
        } else {
          $('.slideSet').animate({
            left: '+=500'
          }, 200, function() {
            dir = -1;
          });
        }
      }

      $('.prevBtn').on('click', function() {
        dir = 1;
        clearInterval(timer);
        timer = setInterval(slideTimer, interval);
        slideTimer();
      });
      $('.nextBtn').on('click', function() {
        dir = -1;
        clearInterval(timer);
        timer = setInterval(slideTimer, interval);
        slideTimer();
      });
});

上記コードでとりあえず動くはずです。
ただし一定時間後に自動的にスクロールする動きになっていますが、そのまま放置すると永遠に左にスクロールする挙動のため画像が表示されなくなります。
この部分はどうしたいのかは記載いただいていないので、想定とする挙動となるよう作ってみてください。

Comments

[退会済みユーザー]

上記の内容に
function() {
$(this).append($('.slide:first-child'));
$(this).css('left', '-500');
が抜けていますが、外している理由はなんでしょうか。

[vnsa7221]

外している理由に関しては、現在のコードではそちらのコードがあってもなくても挙動に差がないためです。
おそらくスライドの一番前、一番後ろに画像を追加するように実装しようと思いますが、上で記載したコードに該当コードを追加して確認しても挙動に差はないと思われます。
入れたい場合はこちらの部分も修正が必要と思われます。

また上で「durationの設定方法が間違い」と記載しましたが、こちら変数を入れていたということで指摘に不備がありました。失礼いたしました。

[退会済みユーザー]

左に永遠にスクロールするのではなく、スライド画像が続くようにするにはどのように実装すればよろしいのでしょうか。

[vnsa7221]

おそらく実装したいものは無限ループスライダーだと思われますが、こちらのjQuery実装は調べてみると結構多くの記事があります。そちらを参照してまずは自分で進めてみてはいかがでしょうか？

自分で調べてみてうまくいかない部分が新しく出てきたら再度質問をしてみるなり、こちらの質問に追記するなどして有識者の方にお伺いすると良いのかなと思います。