jQueryを使った画面スライド表示について

Question

html

   <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>

css

.slide {
  display: none;
}

.active {
  display: block;
}


// jQuery

$(function() {
  $('.index-btn').click(function() {
    $('.active').removeClass('active');
    var clickedIndex = $('.index-btn').index($(this));
    $('.slide').eq(clickedIndex).addClass('active');
    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 $displaySlide = $('.active');
    $displaySlide.removeClass('active');
    if ($(this).hasClass('next-btn')) {
      $displaySlide.next().addClass('active');
    } else {
      $displaySlide.prev().addClass('active');
    }
  
    var slideIndex = $('.slide').index($('.active'));
    $('.change-btn').show();
    if (slideIndex == 0) {
      $('.prev-btn').hide();
    } else if (slideIndex == 3) {
      $('.next-btn').hide();
    }
  });
});

１～４の番号ボタンを押したり、「前へ→　←次へ」ボタンを押して、画面をスライドさせたいのですが、
jQueryコードの７行目。$('.change-btn').show();　←　このコードが何故、必要なのでしょうか？
この７行目がコードがないと、一度言えた「前へ→　←次へ」ボタンは表示されません。

五行目に$('.slide').eq(clickedIndex).addClass('active');というコードがあって、activeクラスをつけているのに、
どうして７行目のコードがないと、一度消えた「前へ→　←次へ」ボタンが表示されないのでしょうか？よろしくお願いします。

Answer 1

$('.slide').eq(clickedIndex).addClass('active'); はスライドを表示。
$('.change-btn').show(); はボタンを表示。

別の物を操作しています。

Comments

[退会済みユーザー]

はっ！目から鱗です。２日間、悩んでいました・・。ありがとうございます。

Answer 2

jQueryコードの７行目。$('.change-btn').show();　←　このコードが何故、必要なのでしょうか？

このコードでchange-btnというクラスがついた要素のdisplay: noneの状態を打ち消しています。

五行目に$('.slide').eq(clickedIndex).addClass('active');というコードがあって、activeクラスをつけているのに、

どうして７行目のコードがないと、一度消えた「前へ→　←次へ」ボタンが表示されないのでしょうか？

開発者ツールで確認すると見やすいのですが、
hide()やshow()は要素に直接スタイルをつけるようになっています。
activeクラスも確かにdisplayを変更しているのですが、CSSのスタイルより、要素に直接当てたスタイルの方が優先されるので、
hide()状態でactiveクラスがついただけでは優先度の違いから表示されません。
コードの読み間違いしてました。
kei344様のおっしゃる通り、activeクラスをつけている要素と、hide()``show()をしている要素は別物です。

Comments

[退会済みユーザー]

回答ありがとうございます。コードが長くなると、何がなんだかわからなくなります。