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

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

ただいまの
回答率

89.97%

ボタンをクリックすると画像を変更できるスライドショーを作りたい

解決済

回答 2

投稿

  • 評価
  • クリップ 1
  • VIEW 3,028

vnsa7221

score 105

前提・実現したいこと

(例)jQueryでボタンをクリックするとスライドできスライドショーをを作っています。  
自分なりにサイト等を参照して作ってみたのですがうまくいきません。
jQueryのコード内で、どこがおかしいかアドバイスをお願いします。

ソースコード

<!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="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>
</div>
<script type="text/javascript" src="js/jquery.js"></script>
<script type="text/javascript">
$(function(){
    var $slideWidth = $('.slider-list').find('img').width(),    // slider-list内にあるimgの幅を習得して代入
        $slideLength = $('.slider-list').find('img').length,    // slider-list内にあるimgの数を習得して代入
        $slideMain = [],                                        // 空の配列指定                            
        currentNum = 0;                                          // 現在表示している画像の番号

    for(var i = 0 ; i < $slideLength ; i++){
        $slideMain[i] = -i * $slideWidth;
    }


    var sliding = function(){
        $('slider-list').animate({
            'left': $slideMain[currentNum]
        },500);

        if(currentNum < 0){
            currentNum = $slideLength - 1;
        }

       else if(currentNum > $slideLength - 1){
            currentNum = 0;
        }
    };


    $('.prev').click(function(){
        currentNum--;

        sliding();
    });

    $('.next').click(function(){
        currentNum++;

        sliding();
    });
});
  • 気になる質問をクリップする

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

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

  • orange0190

    2016/02/22 09:06

    どううまくいかないのか教えてください。

    キャンセル

  • vnsa7221

    2016/02/22 15:06

    詳細な情報がなくて申し訳ございません。
    ボタンをクリックしても画像がスライドさせることができない状態です。

    キャンセル

回答 2

checkベストアンサー

+2

とりあえず、

$('slider-list').animate({


↑を↓に変更。ドット打ち忘れです。

$('.slider-list').animate({



あと、currentNum = 0の時に.prevを押すとcurrentNum = -1になり$slideMain[-1]を参照しようとするはずなので、そこは何とかした方がいいのでは。逆方向も同様です。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2016/02/22 15:32

    回答ありがとうございます。
    classがきちんと指定できていませんでした・・・。
    ご指摘ありがとうございます。

    $slideMain[-1]を参照しようとする問題に関しては、変数slidingでうまくできていると思っていたのですが、その点も問題があるみたいです。
    修正してみたいと思います。

    キャンセル

0

プラグインを使っていいのであれば、使ってみてはどうでしょうか?
http://www.jquerystudy.info/plugins/carou/fred1.html

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2016/02/22 15:17

    回答ありがとうございます。
    今回プラグインを使用せずに作成しようと思っているため、プラグインなしでコードを作成しています。

    ですが、後々プラグインを使用して作成したいと考えています。
    ご丁寧にリンクまでつけていただきありがとうございます。

    キャンセル

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

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