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

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

ただいまの
回答率

87.37%

スライドショーをjQuery/switchで実装したい

解決済

回答 1

投稿

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

score 6

前提・実現したいこと

スライドショーをjQueryで実装したいです。
自力で実装してみたものの、 prevボタンとそれ以外、という処理になってしまいました。elseは例外処理用に使うべきだと思うですが、nextの時の動作を正常系の挙動として実装する場合は、どう実装したら良いでしょうか・・・
また、今回は、switchで書くよりも、if文で書くほうが使い方として合ってるのかな、と考えていますが、実際どうでしょうか。ご意見お聞かせください。

該当のソースコード

   <div class="slider-wrap">
        <div class="slider-area">
            <ul class="slider-list clearfix">
                <li><img src="img/1.jpg" alt=""></li>
                <li><img src="img/2.jpg" alt=""></li>
                <li><img src="img/3.jpg" alt=""></li>
                <li><img src="img/4.jpg" 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>
.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; }
$(function() {
    "use strict"

    var $slider = $(".slider-list"), 
        photos = $slider.find("li").length, 
        slideAnimation = [], 
        listIndex = 0; 

    for(var i = 0; i < photos; i++) { 
     slideAnimation [i] = - i * $slider.find("li").width() + "px"; 
    }


    $(".slider-ctrl-btn").on("click", function() {
        slidePhoto($(this).data("ctrl"));
    });


    function slidePhoto (_direction) {
        if(_direction === "prev") {
            if(listIndex === 0) {
                listIndex = photos - 1;

            } else {
                listIndex--;
            }

        } else {
            if(listIndex === photos - 1) {
                listIndex = 0;

            } else {
                listIndex++;
            }
        }

        $slider.animate({left: slideAnimation[listIndex]}, 500, "swing");
    }
});
  • 気になる質問をクリップする

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 1

checkベストアンサー

0

以下、全て私見です。


普通にif文を並べるのではダメなのですか?
条件分岐のコストが気になるならば、早期リターンで関数を離脱するという手もあると思います。
(その場合、状態変更だけの関数として、描画部分を別関数に切り分ける必要がありますが)

あるいは、早期チェックで例外スローしてしまえば、心置きなく else で処理できるような気もしますが、意図が伝わりにくいように思いますし、_direction の状態が増えることがある場合、バグの温床になりそうな気もしますね。

switch と if についてはどちらでもいいと思いますが、if の羅列より switch の方が意図が伝わりやすいのでしょうね。


私ならば、辞書を作って、ループ処理は剰余で書き、if文をやめます。

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2021/03/26 07:16

    遅くなってすみません!
    ありがとうございます。

    キャンセル

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

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

関連した質問

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