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

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

ただいまの
回答率

88.78%

varを再定義する理由が何か分からないです。

解決済

回答 3

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 1,060

Akaho

score 39

progateのjquery学習コース上級編を学習しています。
課題内容としては
画像にある 数字1〜4をおすボタンと、次へ・前へをおすボタンをおすと、それに対応する画像が出てきます。
それで、最初の画像(数字1)と最後の画像(数字4)になると、おすボタンである『前へ』と『次へ』が消えるようにするという課題です。
(最初の画像になると手本のように『前へ』が消え、最後の画像になると『次へ』が消えるようにする)
課題クリアしたのですが、今回教えてもらいたいことはvar slideIndex=$('.slide').index($('.active'));$('.index-btn').click(function() {
内で定義していれば、別に$('.change-btn').click(function() {で改めてかく必要性はないんじゃないかという疑問を解決して欲しいです。変数varのスコープはブロックの外まであるというのを理由にそう思うのですが、、、
![イメージ説明](cb5c2ef162b00e5074d9d91fc89ac027.png)

コード
<!DOCTYPE html>
<html lang="ja">
  <head>
    <meta charset="utf-8">
    <title>Progate</title>
    <link rel="stylesheet" href="stylesheet.css">
    <script src="https://ajax.googleapis.com/ajax/libs/jquery/2.1.4/jquery.min.js"></script>
  </head>
  <body>
    <div class="slide-wrapper">
      <h2 class="slide-title">にんじゃわんこの一年</h2>
        <div class="change-btn-wrapper">
          <div class="change-btn prev-btn">← 前へ</div>
          <div class="change-btn next-btn">次へ →</div>
        </div>
        <ul class="slides">
          <li class="slide active"><img src="https://s3-ap-northeast-1.amazonaws.com/progate/shared/images/lesson/jquery/advanced/spring.jpg"></li>
          <li class="slide"><img src="https://s3-ap-northeast-1.amazonaws.com/progate/shared/images/lesson/jquery/advanced/rainy.jpg"></li>
          <li class="slide"><img src="https://s3-ap-northeast-1.amazonaws.com/progate/shared/images/lesson/jquery/advanced/autumn.jpg"></li>
          <li class="slide"><img src="https://s3-ap-northeast-1.amazonaws.com/progate/shared/images/lesson/jquery/advanced/winter.jpg"></li>
        </ul>
        <div class="index-btn-wrapper">
          <div class="index-btn">1</div>
          <div class="index-btn">2</div>
          <div class="index-btn">3</div>
          <div class="index-btn">4</div>  
        </div>
      </div>
    <script type="text/javascript" src="script.js"></script>
  </body>
</html>
コード
body {
  font-family: "Hiragino Maru Gothic ProN", sans-serif;
}

ul {
  list-style: none;
}

.slide-wrapper {
  text-align: center;
  width: 1000px;
  margin: 0 auto;
  color: #5e6f84;
  padding: 60px 0;
}

.slide-title {
  font-size: 40px;
  margin-bottom: 30px;
}

.change-btn-wrapper {
  width: 500px;
  margin: 20px auto;
  font-size: 18px;
}

.change-btn-wrapper:after {
  content: "";
  clear: both;
  display: block;
}

.prev-btn {
  cursor: pointer;
  float: left;
  /* 初期状態では非表示にしてください */
  display:none;

}

.next-btn {
  cursor: pointer;
  float: right;
}

.slides {
  padding: 0;
}

.slide {
  display: none;
}

.active {
  display: block;
}

.slide img {
  width: 500px;
  height: auto;
  border-radius: 5px;
}

.index-btn-wrapper {
  font-size: 16px;
  margin-top: 20px;
}

.index-btn {
  display: inline-block;
  color: #4e90af;
  background-color: #e0f5ff;
  width: 40px;
  padding: 6px 0;
  margin: 0 5px;
  border-radius: 3px;
  cursor: pointer;
}

.index-btn:hover {
  color: #fff;
  background-color: #5cabd0;
}
コード
$(function() {
  $('.index-btn').click(function() {
    $('.active').removeClass('active');
    var clickedIndex = $('.index-btn').index(this);
    $('.slide').eq(clickedIndex).addClass('active');

    // 1. 変数slideIndexに「.active」要素のインデックス番号を代入してください
    var slideIndex=$('.slide').index($('.active'));

    // 3. change-btn要素を表示してください
    $('.change-btn').show();

    // 2. ifとelse ifを用いて、「.change-btn」の表示/非表示をおこなってください
    if(slideIndex==0){
      $('.prev-btn').hide();
    }else if(slideIndex==3){
      $('.next-btn').hide();

    }

  });

  $('.change-btn').click(function() {
    var $displaySlide = $('.active');
    $displaySlide.removeClass('active');
    if ($(this).hasClass('next-btn')) {
      $displaySlide.next().addClass('active');
    } else {
      $displaySlide.prev().addClass('active');
    }

    // index-btnのクリックイベントと同様の処理を記述してください
    var slideIndex=$('.slide').index($('.active'));
    $('.change-btn').show();
    if(slideIndex==0){
      $('.prev-btn').hide();
    }else if(slideIndex==3){
      $('.next-btn').hide();
    }

  });
});


追記
質問する経緯
$('.change-btn').click(function() {内にある
var slideIndex=$('.slide').index($('.active'));を削除してやってみるとクリアできなかったので、
理由が分からず質問しました。varのことを調べて『変数varのスコープはブロックの外まである』と知り、progateの仕様で無理だったのか、まだ知らない知識があるのかの判断ができなかったからです。なぜ無理だったのかという理由に重点をおいた質問でした。情報欠けた質問になってしまって申し訳なかったです。

リンク内容

追記 総括
『まず、変数slideIndexに入っているのはslideクラスに付いているactiveクラスのインデックス番号であること、つまり数値であることを認識。
そしてactiveクラスは.change-btnクリックイベントの中にある変数$displaySlideに入っている。
その$displaySlideを使って、if文の中でactiveクラスを動かす処理を(一番下のコード)しているから、そのまま
var slideIndex=$('.slide').index($('.active'));をいれずに
if(slideIndex==0){
$('.prev-btn').hide();
}else if(slideIndex==3){
$('.next-btn').hide();
}の処理をするとだめ。なぜなら、activeクラスが移動したために変数slideIndexの数値が初期状態からずれてしまっているため。その防止策としてvar slideIndex=$('.slide').index($('.active'));でリセッしてやらないといけない。』

コード
$displaySlide.next().addClass('active');
} else {
$displaySlide.prev().addClass('active');
}

  • 気になる質問をクリップする

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

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

  • think49

    2019/06/03 21:25

    挑発的なタイトルを付ける必要がありますか

    キャンセル

  • 1T2R3M4

    2019/06/03 21:39

    progateの見解は何だったの。

    キャンセル

  • m.ts10806

    2019/06/03 21:48

    >そう思うのですが、、、
    実際にやってみた結果を記載してください。これを実際に解決するのは周囲ではなくご自身のような気がします。
    試せばわかることなので。試したうえで思ったように動かなければデバッグして動作を確認すると良いです。

    キャンセル

  • Lhankor_Mhy

    2019/06/03 21:48

    『改めてかく必要性はないんじゃないか』とのことですが、その改変をして動作するコードをご提示ください。

    キャンセル

回答 3

checkベストアンサー

+4

var を伴って宣言した変数のスコープは実行コンテキスト (execution context)であり、これは変数を含んでいる関数、または関数の外で宣言された変数はグローバルになります。
var - JavaScript | MDN

ブロックスコープの外には出ますが、イベントリスナは関数なのでその中にスコープができます。

$(function() {
  $('.index-btn').click(function() {
//...
    slideIndex=$('.slide').index($('.active'));
//...
  });

  $('.change-btn').click(function() {
//...
    slideIndex=$('.slide').index($('.active'));
//...
});

↑のようにすれば、必要ないといえばないのですが、グローバル変数になります。あまりよいことではないと思います。

$(function() {
  $('.index-btn').click(function() {
//...
    var slideIndex=$('.slide').index($('.active'));
//...
  });

  $('.change-btn').click(function() {
//...
    slideIndex=$('.slide').index($('.active'));
//...
});

↑この場合も動きますが、$('.change-btn').clickの方はやはりグローバル変数です。

$(function() {
  var slideIndex;
  $('.index-btn').click(function() {
//...
    slideIndex=$('.slide').index($('.active'));
//...
  });

  $('.change-btn').click(function() {
//...
    slideIndex=$('.slide').index($('.active'));
//...
});

↑この場合は両方ともローカル変数ですが、スコープの範囲が広くなっていますね。不必要に変数のスコープを広げるのはいいことではないと思います。状況によってはありかと思いますが、この場合は不要でしょう。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2019/06/04 15:21

    $('.slide').index($('.active'));

    は、.slide 要素たちのなかで、.active が何番目にあるか
    を、数値で返します。

    なので、.active の場所が変わると、違う値になります。

    ですから、

    removeClass('active');
    addClass('active');

    で、.active の場所を変えた場合、

    $('.slide').index($('.active'));

    を再度取得しないと、間違った値になります。

    キャンセル

  • 2019/06/04 15:23

    繰り返しになりますが、slideIndex に入っているのはプロパティへの参照ではなく、ただの数値です。
    .active の場所が変わった時に、勝手に値が変わってくれればラクなのですが、そういうものではないです。

    キャンセル

  • 2019/06/04 19:25

    しっかり理屈を解説してくださりありがどうございました。

    キャンセル

+2

このようにslideIndexを外側に一箇所だけ記述すると
slideIndexが0となり、現在地が取得できないため
前へ次へが使用できなくなります。

そのため、スコープの関係上、click(function() {内の
2箇所に記述する必要があります。

$(function() {
  var slideIndex = $(".slide").index($(".active"));
  $(".index-btn").click(function() {
    $(".active").removeClass("active");
    var clickedIndex = $(".index-btn").index(this);
    $(".slide")
      .eq(clickedIndex)
      .addClass("active");

    // 1. 変数slideIndexに「.active」要素のインデックス番号を代入してください
    // var slideIndex = $(".slide").index($(".active"));

    // 3. change-btn要素を表示してください
    $(".change-btn").show();

    // 2. ifとelse ifを用いて、「.change-btn」の表示/非表示をおこなってください
    if (slideIndex == 0) {
      $(".prev-btn").hide();
    } else if (slideIndex == 3) {
      $(".next-btn").hide();
    }
  });

  $(".change-btn").click(function() {
    var $displaySlide = $(".active");
    $displaySlide.removeClass("active");
    if ($(this).hasClass("next-btn")) {
      $displaySlide.next().addClass("active");
    } else {
      $displaySlide.prev().addClass("active");
    }

    // index-btnのクリックイベントと同様の処理を記述してください
    // var slideIndex = $(".slide").index($(".active"));
    $(".change-btn").show();
    if (slideIndex == 0) {
      $(".prev-btn").hide();
    } else if (slideIndex == 3) {
      $(".next-btn").hide();
    }
  });
});

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2019/06/04 14:32 編集

    『slideIndexを外側に一箇所だけ記述すると』と
    『slideIndexが0となり、現在地が取得できないため』の間にある理屈がわからないです。
    教えて下さい。実際に```var slideIndex = $(".slide").index($(".active"))```をprogateの方でご指摘通りやってみると、前へボタンだけが使えなくなっています。次へはなぜか使えます。

    キャンセル

+1

変数varのスコープはブロックの外まである

たぶんグローバル変数とローカル変数を勘違いしてます。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2019/06/04 14:09

    理解していない点を指摘して下さってありがとうございました。

    キャンセル

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

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

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