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

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

ただいまの
回答率

89.85%

【jQuery】イベントバインドと処理を分けて実装したいです。

解決済

回答 1

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 101
退会済みユーザー

退会済みユーザー

jQueryのスライドショーのイベントバインドと処理を分けて実装しています。

slider関数の中に変数とfor文を入れるとコンソールでエラーは出なかったのですが、右にクリックしたときに1つしかスライドしませんでした。
なぜ変数とfor文をslider関数の中に入れてしまうと動作できないのかを教えていただきたいです。

具体的にやりたいことは、ボタンをクリックしたときにslider関数を読み込むようにしたいです。

<!doctype html>
<html>
<head>
<meta charset="UTF-8">
<meta name="viewport" content="width=device-width, initial-scale=1.0, maximum-scale=1.0, user-scalable=no">
<meta name="format-detection" content="telephone=no">
<title></title>
<link rel="stylesheet" href="css/reset.css">
<link rel="stylesheet" href="css/base.css">
<link rel="stylesheet" href="css/style.css">
</head>
<body>
<div class="wrapper">
    <div class="slider-wrap">
        <div class="slider-area">
            <ul class="slider-list clearfix">
                <li><img src="http://placehold.jp/150x150.png" alt=""></li>
                <li><img src="http://placehold.jp/3d4070/ffffff/150x150.png" alt=""></li>
                <li><img src="http://placehold.jp/150x150.png" alt=""></li>
                <li><img src="http://placehold.jp/3d4070/ffffff/150x150.png" alt=""></li>
            </ul>
            <button type="button" class="slider-ctrl-btn prev" data-ctrl="prev"></button>
            <button type="button" class="slider-ctrl-btn next" data-ctrl="next"></button>
        </div>
    </div>
</div>
<script type="text/javascript" src="js/jquery.js"></script>
<script type="text/javascript">
/* slider-list→空の配列
*  sliderClickCount→クリックしたときの初期の値
*  btn→ボタンの引数
*/
$(function(){
  $(".slider-wrap").find(".slider-ctrl-btn").on("click",function(){
        sliderDirection=$(this).data("ctrl");
        slider(sliderDirection);
    });

    function slider(btn){
        var sliderList=[];
        var sliderClickCount=0;

        for(i=0; i < $(".slider-list").find("img").length; i++){
          sliderList[i]=-i*$(".slider-list").find("img").width()
        }

        switch(btn){
            case"next":if(sliderClickCount<$(".slider-list").find("img").length-1){
                sliderClickCount++
            }else{
                sliderClickCount=0
            }
            break;
            case"prev":if(sliderClickCount!=0){
                sliderClickCount--}else{
                    sliderClickCount=$(".slider-list").find("img").length-1
                }
            break;
            default:break
        }
        $(".slider-wrap").find(".slider-list").animate({left: sliderList[sliderClickCount]},500)
    }
});
</script>
</body>
</html>
.slider-wrap {
    width: 600px;
    padding: 10px;
    margin: 0 auto;
    border-radius: 10px;
    background-color: #DDD;
    box-shadow: 0 2px 5px rgba(50, 50, 50, 0.4);
}
.slider-area {
    position: relative;
    width: 600px;
    height: 300px;
    background-color: #FFF;
    overflow: hidden;
}
.slider-list {
    position: absolute;
    top: 0;
    left: 0;
    width: 2400px;
    height: 300px;
}
.slider-list > li { float: left; }
.slider-ctrl-btn {
    position: absolute;
    top: 50%;
    width: 30px;
    height: 30px;
    margin-top: -15px;
    cursor: pointer;
    border-radius: 15px;
    background-color: rgba(255, 255, 255, 0.5);
}
.slider-ctrl-btn.prev { left: 30px; }
.slider-ctrl-btn.next { right: 30px; }
  • 気になる質問をクリップする

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

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

  • yambejp

    2019/12/03 10:29

    CSSが提示されていないので挙動が確認できません

    キャンセル

  • 退会済みユーザー

    退会済みユーザー

    2019/12/03 10:32

    CSS追記しました。
    大変失礼いたしました。

    キャンセル

回答 1

checkベストアンサー

+1

最終型がよくわからないのでprev/nextで左右移動だけ

<style>
.slider-wrap {
    width: 600px;
    padding: 10px;
    margin: 0 auto;
    border-radius: 10px;
    background-color: #DDD;
    box-shadow: 0 2px 5px rgba(50, 50, 50, 0.4);
}
.slider-area {
    position: relative;
    width: 600px;
    height: 300px;
    background-color: #FFF;
    overflow: hidden;
}
.slider-list {
    position: absolute;
    top: 0;
    left: 0;
    width: 2400px;
    height: 300px;
}
.slider-list > li { float: left; }
.slider-ctrl-btn {
    position: absolute;
    top: 50%;
    width: 30px;
    height: 30px;
    margin-top: -15px;
    cursor: pointer;
    border-radius: 15px;
    background-color: rgba(255, 255, 255, 0.5);
}
.slider-ctrl-btn.prev { left: 30px; }
.slider-ctrl-btn.next { right: 30px; }
</style>
<script type="text/javascript">
$(function(){
  var left=0;
  var current=0;
  $('.next').on('click',function(){
    var w=$('.slider-list img').eq(current).width();
    current++;
    var len=$('.slider-list img').length;
    if(current<len){
      left-=w;
      $('.slider-list').animate({left:left});
    }else{
      current=len-1;
    }
  });
  $('.prev').on('click',function(){
    var w=$('.slider-list img').eq(current).width();
    current--;
    if(current>=0){
      left+=w;
      $('.slider-list').animate({left:left});
    }else{
      current=0;
    }
  });
});
</script>
<div class="wrapper">
    <div class="slider-wrap">
        <div class="slider-area">
            <ul class="slider-list clearfix">
                <li><img src="https://placehold.jp/ff0000/00ffff/150x150.png?text=1"></li>
                <li><img src="https://placehold.jp/00ff00/ff00ff/150x150.png?text=2"></li>
                <li><img src="https://placehold.jp/0000ff/ffff00/150x150.png?text=3"></li>
                <li><img src="https://placehold.jp/000000/ffffff/150x150.png?text=4"></li>
            </ul>
            <button type="button" class="slider-ctrl-btn prev" data-ctrl="prev"></button>
            <button type="button" class="slider-ctrl-btn next" data-ctrl="next"></button>
        </div>
    </div>
</div>

最後の次は最初、最初の前は最後

$(function(){
  var left=0;
  var current=0;
  $('.next').on('click',function(){
    var w=$('.slider-list img').eq(current).width();
    var len=$('.slider-list img').length;
    current++;
    if(current<len){
      left-=w;
    }else{
      left =0;
      current=0;
    }
    $('.slider-list').animate({left:left});
  });
  $('.prev').on('click',function(){
    var w=$('.slider-list img').eq(current).width();
    var len=$('.slider-list img').length;
    current--;
    if(current>=0){
      left+=w;
    }else{
      left=-$('.slider-list img:not(:last)').map(function(){
        return $(this).width();
      }).get().reduce((x,y)=>x+y);
      current=len-1;
    }
    $('.slider-list').animate({left:left});
  });
});

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2019/12/03 11:28

    ありがとうございます。
    最終型は4つ目の画像を右にクリックしたら1つ目の画像にスライドさせて、1つ目の画像を左にクリックしたら4つ目の画像にスライドする動作になります。
    左右の移動だけでもかなり大きいので、勉強になりました。

    キャンセル

  • 2019/12/03 12:32 編集

    一応追記しておきました
    画像をクリックするのでしょうか?prev/nextのクリックではいけないのでしょうか?

    キャンセル

  • 2019/12/03 12:37

    prev/nextのクリックで大丈夫です。
    ありがとうございます!
    動作も仕様書通りで、バッチリです。
    勉強になりました。

    キャンセル

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

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