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

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

ただいまの
回答率

90.47%

  • jQuery

    6930questions

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

  • CSS

    6015questions

    CSSはXMLやHTMLで表現した色・レイアウト・フォントなどの要素を指示する仕様の1つです。

  • HTML5

    4180questions

    HTML5 (Hyper Text Markup Language、バージョン 5)は、マークアップ言語であるHTMLの第5版です。

jQueryによる スライドショーアプリのバグを直したい

解決済

回答 2

投稿 編集

  • 評価
  • クリップ 1
  • VIEW 957

k499778

score 501

現在jQueryを使ってスライドショーアプリを作成しています。
ドットインストールにその項目があるのでそれを見ながら作っています。

質問があるのですが、
結論から言うと、

下記のコードのバグ(「前へ」「次へ」ボタンを押したとき思い通りサムネイルが透明にならないバグ)を直すにはどうしたらいいでしょうか?


今ドットインストールの下記リンクのところを学習しており、
動画でできていることが実際やってみるとうまくできません。
http://dotinstall.com/lessons/slideshow_jquery/25406

コンソールにもエラーは出ておらず、また何度もコードを確認したのですがわかりませんでした。


状況として、
本当は
「前へ」や「次へ」ボタンをクリックすると真ん中に出ているのと同じサムネイル画像以外が透明になります。
しかし現状は、
それが透明になりません。

↓こうなってほしい
イメージ説明
コードは次の通りです。
<!DOCTYPE html>
<html>
<head>
<meta charset="UTF-8">
<title>jQueryでスライドショー</title>
<style>
body {
    margin: 10px auto;
    text-align: center;
}

.thumbnail {
    width: 300px;
    height: 300px;
    opacity: 0.5;
    cursor: pointer;
}

.current {
    opacity: 1.0;
}

#main img {
    width: 900px;
    height: 900px;
}

button {
    width: 100px;
    height: 50px;
}

.thumbnail+.thumbnail {
    margin-left: 4px;
}
</style>
</head>
<body>
    <div id="thumbnails"></div>
    <div id="main"></div>
    <div id="nav">
        <button id="prev">&laquo; 前へ</button>
        <button id="next">次へ &raquo;</button>
        <button id="play">再生</button>
        <button id="stop">停止</button>
    </div>

    <script src="http://code.jquery.com/jquery-1.11.1.min.js"></script>
    <script>
        $(function() {
            var files = [ '1b.jpg', '2b.jpg', '3b.jpg' ];

            var img;
            var timer;
            var currentNum = 0;

            for (var i = 0; i < files.length; i++) {
                img = $('<img>').attr('src', files[i]).addClass('thumbnail');
                $('#thumbnails').append(img);
            }

            $('#main').append($('<img>').attr('src', files[0]));

            $('.thumbnail:first').addClass('current');

            $('.thumbnail').click(function() {
                $('#main img').attr('src', $(this).attr('src'));
                currentNum = $(this).index();
                $(this).addClass('current').siblings().removeClass('current');
            });

            $('#prev').click(function() {
                currentNum--;
                if (currentNum < 0) {
                    currentNum = files.length - 1;
                }
                $('#main img').attr('src', files[currentNum]);
                $('.thumbnail').removeClass('current');
                $('.thumbnail').eq('currentNum').addClass('current');
            });

            $('#next').click(function() {
                currentNum++;
                if (currentNum > files.length - 1) {
                    currentNum = 0;
                }
                $('#main img').attr('src', files[currentNum]);
                $('.thumbnail').removeClass('current');
                $('.thumbnail').eq('currentNum').addClass('current');
            });
            function autoPlay() {
                $('#next').click();
                timer = setTimeout(function() {
                    autoPlay();
                }, 1000);
            }
            $('#play').click(function() {
                autoPlay();
            });
            $('#stop').click(function() {
                clearTimeout(timer);
            });
        });
    </script>
</body>

prevIDのclickメソッドやnextIDのclickメソッドの箇所が問題のポイントと思い
何度も見ているのですが解決できていない次第です。

もしわかる方がいればよろしくお願いします。
  • 気になる質問をクリップする

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 2

+1

            $('#prev').click(function() {
                currentNum--;
                if (currentNum < 0) {
                    currentNum = files.length - 1;
                }
                $('#main img').attr('src', files[currentNum]);
                $('.thumbnail').removeClass('current');
                $('.thumbnail').eq('currentNum').addClass('current');
            });

            $('#next').click(function() {
                currentNum++;
                if (currentNum > files.length - 1) {
                    currentNum = 0;
                }
                $('#main img').attr('src', files[currentNum]);
                $('.thumbnail').removeClass('current');
                $('.thumbnail').eq('currentNum').addClass('current');
            });
の部分で
                $('.thumbnail').eq('currentNum').addClass('current');
ではなく、
                $('.thumbnail').eq(currentNum).addClass('current');
ではないでしょうか?

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2015/04/28 15:20

    すみません。
    かぶってしまいましたね。
    無視してください。

    キャンセル

  • 2015/04/28 15:22

    ShunsukeIzuiさん
    ご回答ありがとうございます。その通りでした。
    今後注意したいと思います。
    少し前に同じ回答をして下さった方がいるのでベストアンサーはそちらになってしまっています。ご了承下さい。
    ありがとうございます!感謝しています!

    キャンセル

checkベストアンサー

0

$('.thumbnail').eq('currentNum').addClass('current');

$('.thumbnail').eq(currentNum).addClass('current');
に修正するとできるような気がします。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2015/04/28 15:15

    penguinshunyaさん
    ありがとうございます!その通りでした。
    やはり書き方のミスでした。jQuery勉強します。

    ありがとうございます!感謝しています。

    キャンセル

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

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

関連した質問

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

  • jQuery

    6930questions

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

  • CSS

    6015questions

    CSSはXMLやHTMLで表現した色・レイアウト・フォントなどの要素を指示する仕様の1つです。

  • HTML5

    4180questions

    HTML5 (Hyper Text Markup Language、バージョン 5)は、マークアップ言語であるHTMLの第5版です。