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

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

ただいまの
回答率

87.49%

【jQuery質問】スライドショーを実装する方法を教えてください

解決済

回答 1

投稿 編集

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

退会済みユーザー

グローバル変数でなく関数スコープ(ローカル変数)を用いてスライドショーを実装したいと考えてます。

var pages = [],
        num = 0,
        pagesNum = 0;


上の3行をグローバルに宣言した変数にすれば動作できるのですが、下のindex.htmlのようにローカルで閉じ込めると以下のエラーがでてしまいます。

Uncaught TypeError: pages.push is not a function
    at slide (index.html:49)
    at HTMLButtonElement.<anonymous> (index.html:58)
    at HTMLButtonElement.dispatch (jquery.js:3058)
    at HTMLButtonElement.eventHandle (jquery.js:2676)
slide @ index.html:49
(anonymous) @ index.html:58
dispatch @ jquery.js:3058
eventHandle @ jquery.js:2676


どんなコードを書けば良いか教えていただけると非常に嬉しいです。
参考にした記事です。
以下に私が書いたindex.htmlとstyle.cssになります。
グローバル変数を用いないという条件であれば、コードをこう書いたほうが良いというアドバイスをいただけたら非常に嬉しいです。

<!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/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">
"use strict";
$(function() {
  (function(){
    var pages = [],
        num = 0,
        pagesNum = 0;
  })();

  function slide(pages, num, pagesNum){
    for(var i = 0; i < $('.slider-list li').length; i++){
      if (i == 0) {
        pages.push(0);
      } else {
        num += - $('.slider-list li').eq(i).width();
        pages.push(num);
      }
    }
  }

  $('.slider-ctrl-btn').on('click', function(){
    slide(pages, num, pagesNum);

    if ($(this).data('ctrl') === 'next') {
      var step = 1;
      pagesNum = pagesNum + step;

    if (pagesNum === pages.length) {
      pagesNum = 0;
    }

    $(".slider-list").animate({left: pages[pagesNum]},500)}

    else if ($(this).data('ctrl') === 'prev') {
      var back = -1;
      pagesNum = pagesNum + back;

    if (pagesNum === -1) {
      pagesNum = pages.length - 1;
    }
      $(".slider-list").animate({left: pages[pagesNum]},500)}
  });
});
</script>
</body>
</html>

style.css

.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/11/19 18:40 編集

    おそらくスタイルシートがそれなりに影響をしていますよね?
    命題のソースだと判断つかないです
    画像もplacehold.jpとか使って明示してもらったほうがよいでしょう

    キャンセル

  • 退会済みユーザー

    退会済みユーザー

    2019/11/19 18:53

    ありがとうございます。
    画像はアドバイスの通りにplacehold.jpを使いました。

    キャンセル

  • kei344

    2019/11/19 20:24

    エラー文も質問文に追記してください。

    キャンセル

  • 退会済みユーザー

    退会済みユーザー

    2019/11/19 20:34

    ありがとうございます。
    エラー文とエラーが出たファイルの現在のコードを書きました。

    キャンセル

回答 1

checkベストアンサー

0

  // ここはグローバルスコープではなく、$(function() {});のローカルスコープです。
  /* 削除 *//* (function(){ */
    var pages = [],
        num = 0,
        pagesNum = 0;
  /* 削除 *//* })(); */

【JavaScriptのスコープ総まとめ - スコープの種類とその基本 | CodeGrid】
https://app.codegrid.net/entry/2017-js-scope-1

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

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

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

関連した質問

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