質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

90.50%

  • JavaScript

    16443questions

    JavaScriptは、プログラミング言語のひとつです。ネットスケープコミュニケーションズで開発されました。 開発当初はLiveScriptと呼ばれていましたが、業務提携していたサン・マイクロシステムズが開発したJavaが脚光を浴びていたことから、JavaScriptと改名されました。 動きのあるWebページを作ることを目的に開発されたもので、主要なWebブラウザのほとんどに搭載されています。

  • jQuery

    6715questions

    jQueryは、JavaScriptライブラリのひとつです。 簡単な記述で、JavaScriptコードを実行できるように設計されています。 2006年1月に、ジョン・レシグが発表しました。 jQueryは独特の記述法を用いており、機能のほとんどは「$関数」や「jQueryオブジェクト」のメソッドとして定義されています。

jQuery→JavaScriptに書き換えたスライドギャラリーを正常に動作させたい

解決済

回答 2

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 450

okame

score 50

前提・実現したいこと

タイトルの通りです。

発生している問題・エラーメッセージ

書き換え途中までは問題なくJavaScriptでも正常に動作するのですが、
5秒おきに設定している自動スライドの処理をjQueryからJavaScriptに書き換えると

  • 自動スライドが効かなくなる
  • スライド進むボタンも挙動がおかしくなる

という事象です。コンソール等にエラーメッセージは特に表示されていません。

該当のソースコード

  <div id="slideGallery">
    <ul id="slide">
      <li><a href="#"><img src="./images/photo01.jpg" alt=""></a></li>
      <li><a href="#"><img src="./images/photo02.jpg" alt=""></a></li>
      <li><a href="#"><img src="./images/photo03.jpg" alt=""></a></li>
      <li><a href="#"><img src="./images/photo04.jpg" alt=""></a></li>
    </ul>
    <div id="nav">
      <img src="./images/nav_prev.png" class="prev">
      <img src="./images/nav_next.png" class="next">
    </div>
    <ul id="paging"></ul>
  </div>
</div>
*{
  list-style-type:none;
  margin:0;
  padding:0;
}
#container{
  width:650px;
  margin:100px auto;
}
#slideGallery{
  width:650px;
  position: relative;
  overflow:hidden;
}
#slide{
  width:10000px;
}
#slide li{
  float:left;
}
#paging{
  text-align: center;
  position: absolute;
  width:100%;
  bottom:10px;
}
#paging li{
  display: inline-block;
  *display: inline;
  *zoom:1;
  width:20px;
  height:20px;
  background: url(./images/page.png);
}
#paging li.active{
  background: url(./images/page_active.png);
}
#nav {
  display: none;
}
#nav .prev{
  position: absolute;
  left:10px;
  top:50%;
  margin-top:-50px;
}
#nav .next{
  position: absolute;
  right:10px;
  top:50%;
  margin-top:-50px;
}


photo01.jpg
photo02.jpg
photo03.jpg
photo04.jpg
nav_prev.jpg
nav_next.png
page_active.png
page.png

書き換え前のjQueryのコード

// ページングの設定
$('#slide li').each(function() {
  $('#paging').append($('<li></li>').attr('data-img', $('img', this).attr('src')));
});
$('#paging li:first-child').addClass('active');

// 自動スライドの設定
var timerId = setInterval(function() {
  $('#nav .next').click();
}, 5000);

// 自動スライドの制御
$('#slideGallery').hover(function() {
  $('#nav').show();
  clearInterval(timerId);
}, function() {
  $('#nav').hide();
  timerId = setInterval(function() {
    $('#nav .next').click();
  }, 5000);
});

// 右矢印ボタンが押された際の挙動
$('#nav .next').click(function() {
  $('#slide:not(:animated)').animate({
    marginLeft: -1 * $('#slide li').width()
  }, function() {
    $('#slide').css('margin-left', '0');
    $('#slide').append($('#slide li:first-child'));
    $('#paging li.active').removeClass('active');
    $('#paging li[data-img="' + $('#slide li:first-child img').attr('src') + '"]').addClass('active');
  });
});

// 左矢印ボタンが押された際の挙動
$('#nav .prev').click(function() {
  $('#slide:not(:animated)')
    .css('margin-left', -1 * $('#slide li').width())
    .prepend($('#slide li:last-child'))
    .animate({
      marginLeft: 0
    }, function() {
      $('#paging li.active').removeClass('active');
      $('#paging li[data-img="' + $('#slide li:first-child img').attr('src') + '"]').addClass('active');
  });
});
書き換えたJavaScriptのコード

// ページングの設定
document.querySelectorAll('#slide li').forEach(function(value) {
  var li = document.createElement('li');
  li.setAttribute('data-img', value.querySelector('img').getAttribute('src'));
  document.getElementById('paging').appendChild(li);
});
document.querySelector('#paging').firstElementChild.classList.add('active');

// 自動スライドの設定
var timerId = setInterval(function() { slideNext(); }, 5000);

// 自動スライドの制御
document.getElementById('slideGallery').addEventListener('mouseover', function(event) {
  document.getElementById('nav').style.display = 'block';
  clearInterval(timerId);

  event.stopPropagation();
}, false);
document.getElementById('slideGallery').addEventListener('mouseout', function(event) {
  document.getElementById('nav').style.display = 'none';
  timerId = setInterval(function() { slideNext(); }, 5000);

  event.stopPropagation();
}, false);

// 右矢印ボタンが押された際の挙動
slideNext();
/**
 * スライド進む
 */
function slideNext() {
  document.querySelector('#nav .next').addEventListener('click', function() {
    // TODO: スライドアニメーションを追加...
    var slide = document.getElementById('slide');
    slide.appendChild(slide.firstElementChild);
    document.querySelector('#paging li.active').classList.remove('active');
    document.querySelector('#paging li[data-img="' + slide.firstElementChild.querySelector('img').getAttribute('src') + '"]').classList.add('active');
  }, false);
}

// 左矢印ボタンが押された際の挙動
document.querySelector('#nav .prev').addEventListener('click', function() {
  // TODO: スライドアニメーションを追加...
  var slide = document.getElementById('slide');
  slide.insertBefore(slide.lastElementChild, slide.firstElementChild);
  document.querySelector('#paging li.active').classList.remove('active');
    document.querySelector('#paging li[data-img="' + slide.firstElementChild.querySelector('img').getAttribute('src') + '"]').classList.add('active');
}, false);

テスト環境

https://jsbin.com/fesaneg/edit?html,js,output
※画像リンク貼れないので意味ないとは思いますが...。

以上です。
皆様のお知恵を拝借できれば幸いです。宜しくお願い致します。

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • Lhankor_Mhy

    2017/09/27 18:37

    『挙動がおかしくなる 』とは具体的には何が起こりますか?

    キャンセル

回答 2

checkベストアンサー

+1

function slideNext() {
  document.querySelector('#nav .next').addEventListener('click', function() {
    // TODO: スライドアニメーションを追加...
    var slide = document.getElementById('slide');
    slide.appendChild(slide.firstElementChild);
    document.querySelector('#paging li.active').classList.remove('active');
    document.querySelector('#paging li[data-img="' + slide.firstElementChild.querySelector('img').getAttribute('src') + '"]').classList.add('active');
  }, false);
}

↑ここがおかしいですね。
setIntervalで呼ばれていますが、そのたびにイベントリスナが増えます。50秒待てば、1回クリックするたびに10回クリックしたことになるかと。
slideNext関数は普通に定義して、イベントで呼べばいいかと思います。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2017/09/28 06:50

    回答ありがとうございます!試してみます。

    キャンセル

  • 2017/09/28 06:56

    正常に動作するようになりました!感謝感謝です☆

    キャンセル

+1

sampleがうまく動かせなくてちょっと仕様でわからないところがあるので
以下適当に書いてみました

 仕様

  • .prevを押すと#slideの末尾のliが先頭にくる
  • .nextを押すと#slideの先頭のliが末尾にくる
  • #slideGalleryにmouseoverすると1秒置きに.nextを押す
  • #slideGalleryにmouseoutすると.nextを押すのをやめる
var timerId;
document.addEventListener('click',function(e){
  var t=e.target;
  var firstLi=document.querySelector("#slide li:first-child");
  var lastLi=document.querySelector("#slide li:last-child");
  if(t.nodeName=="IMG"){
    if(t.classList.contains("prev")){
      lastLi.parentNode.insertBefore(lastLi.cloneNode(true),firstLi);
      lastLi.parentNode.removeChild(lastLi);
    }else if(t.classList.contains("next")){
      firstLi.parentNode.appendChild(firstLi.cloneNode(true));
      firstLi.parentNode.removeChild(firstLi);
    }
  }
});
window.onload=function(){
document.querySelector('#slideGallery').addEventListener('mouseover',function(e){
  timerId = setInterval(function() {
    document.querySelector('#nav .next').click();
  }, 1000);
});
document.querySelector('#slideGallery').addEventListener('mouseout',function(e){
   clearInterval(timerId);
});
}
<div>
<div id="slideGallery">
<ul id="slide">
<li><a href="#"><img src="1.jpg" alt="1"></a></li>
<li><a href="#"><img src="2.jpg" alt="2"></a></li>
<li><a href="#"><img src="3.jpg" alt="3"></a></li>
<li><a href="#"><img src="4.jpg" alt="4"></a></li>
</ul>
</div>
<div id="nav">
<img src="prev.png" class="prev" alt="prev">
<img src="next.png" class="next" alt="next">
</div>
<ul id="paging"></ul>
</div>

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2017/09/28 06:50

    回答ありがとうございます!試してみます。

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 90.50%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る

  • JavaScript

    16443questions

    JavaScriptは、プログラミング言語のひとつです。ネットスケープコミュニケーションズで開発されました。 開発当初はLiveScriptと呼ばれていましたが、業務提携していたサン・マイクロシステムズが開発したJavaが脚光を浴びていたことから、JavaScriptと改名されました。 動きのあるWebページを作ることを目的に開発されたもので、主要なWebブラウザのほとんどに搭載されています。

  • jQuery

    6715questions

    jQueryは、JavaScriptライブラリのひとつです。 簡単な記述で、JavaScriptコードを実行できるように設計されています。 2006年1月に、ジョン・レシグが発表しました。 jQueryは独特の記述法を用いており、機能のほとんどは「$関数」や「jQueryオブジェクト」のメソッドとして定義されています。