JavaScriptでアコーディオン開閉機能を実装したい

Question

実現したい事

アコーディオン機能をJsで実装しようと試みているのですが、開閉機能がなかなか実装出来ません。
Qを押すと、隠れているAが開くというような単純な動きです。。

ただし、クリックしたら、それに属するアコーディオン一つのみ開いている状態にしたいのです。

分からない点

nextElementSiblingとnextSiblingの違いがよく分かりません。また、ループ内での処理も正しいのか
確信を持てません。ご教授頂けますと、幸いで御座います。もしかすると、HTML構造のロジックがしっかりしていないのかもしれません、、、、、。

ソースコード

<section id =qanda>

      <div id =qa-wrapper>
      <h2>よくあるご質問</h2>

      <dl class="qa">
            <dt class="question_txt" data-hoge="q3"><span class="qicon">Q</span>矯正治療の費用を教えてください.また、学生ローンは組むことが出来ますか？</dt>
            <dd id="toggle" class="answer_txt">
              <p><span>A:</span>費用は、万円(税込)となっております</p>
            </dd>

            <dt class="question_txt" data-hoge="q3"><span class="qicon">Q</span>目安の治療期間を教えてください.過去の患者のbefore,afterも知りたいです。</dt>
            <dd id="toggle" class="answer_txt">
              <p><span>A:</span>患者様の経過状況は
              <a href="#">こちら</a>で公開しております。</p>
              </dd>

            <dt class="question_txt" data-hoge="q3"><span class="qicon">Q</span>矯正治療の費用を教えてください.また、学生ローンは組むことが出来ますか？</dt>
            <dd id="toggle" class="answer_txt">
              <p><span>A:</span>費用は、万円(税込)となっております</p>
            </dd>

            <dt class="question_txt" data-hoge="q3"><span class="qicon">Q</span>矯正治療の費用を教えてください.また、学生ローンは組むことが出来ますか？</dt>
            <dd id="toggle" class="answer_txt">
              <p><span>A:</span>費用は、万円(税込)となっております</p>
            </dd>
      </dl>

</div>
</section>

/*リセットcss*/
*,
p,
div,
ul,
li,
article,
section,
footer,
header {
  margin: 0;
  padding: 0;
}

img {
  vertical-align: bottom;
}

body {
  border-top: solid 8px #6cc6c4;
  background-color: #f6f6f6;
}

a,
p,
h1,
h2,
h3 {
  font-family: serif;
}

section h2 {
  padding: 10px;
  text-align: left;
}

header {
  width: 1050px;
  margin: 0 auto;
  margin-bottom: 10px;
}

header h1 {
  float: left;
  padding: 35px;
  padding-left: 0px;
}

.header-right {
  float: right;
  padding-top: 10px;
  margin-right: 3px;
}

p.yoyaku {
  border: solid 1px #d6d6d6;
  padding: 10px;
}

.stickey {
  position: sticky;
  top: 0;
}

.colored {
  background-color: brown;
}

.resize {
  width: 20%;
}

#header {
  width: 1050px;
  margin: 10px auto 0;
}

#header li + li:before {
  content: "<";
  margin-right: 5px;
}

#header li {
  display: inline;
}

ul {
  list-style: none;
}

li {
  text-decoration: none;
}

nav ul {
  display: flex;
  width: 1050px;
  margin: 0 auto;
}

nav ul ul {
  position: absolute;
  top: 100%;
  left: 0;
  display: none;
}

nav ul ul li {
  width: 100%;
  background-color: navy;
  border-bottom: 1px solid #fff;
  right: 0px;
  width: 190px;
  z-index: 999;
  position: relative;
}

nav ul > li {
  justify-content: center;
  background-color: #6cc6c4;
  width: 20%;
  padding: 10px;
  text-align: center;
  border-right: 1px solid white;
  position: relative;
}

nav ul li a {
  font-size: 0.85em;
}

nav ul li a {
  color: white;
  text-decoration: none;
}

nav ul > ul > li {
  background-color: #6cc6c4;
  width: 200px;
  display: block;
  padding: 10px;
  text-align: center;
  border-right: 1px solid white;
}

li.parent {
  position: relative;
}

ul.children {
  left: 100.8%;
  top: 0;
  position: absolute;
  display: none;
}

li.parent:hover ul.children {
  display: block;
  opacity: 0.87;
}

.clearfix::after {
  clear: both;
  display: block;
  content: "";
}

main,
footer {
  width: 1050px;
  margin: 0 auto;
}

main {
  border: solid 1px inherit;
}

#wrapper {
  width: 100%;
}

.left-content {
  width: 25%;
  float: left;
  margin-top: 3.2em;
}

.top h2,
.bottom h2 {
  background-color: #6cc6c4;
  color: white;
  font-size: 1em;
  text-align: left;
  padding: 10px;
  border-radius: 6px 6px 0px 0px;
}

.left-content ul {
  border-right: 1px solid #6cc6c4;
  border-left: 1px solid #6cc6c4;
  background-color: white;
}

.left-content ul li {
  background-color: white;
  margin-bottom: 10px;
  padding: 10px;
  border-bottom: 1px solid #6cc6c4;
  font-size: 0.9em;
}

.left-content li::before {
  content: "●";
  color: #6cc6c4;
  font-size: 0.8em;
}

.left-content ul li a {
  margin-left: 5px;
}

.first-last {
  border-bottom: 0px;
}

.top {
  margin-bottom: 15px;
}

.right-content {
  width: 73%;
  float: right;
}

.right-content p {
  width: auto;
}

#imformation {
  width: 99.5%;
  background-color: white;
  padding: 0;
}

.imfor-dl {
  margin-bottom: 50px;
}

dl {
  padding: 5px;
}

dd {
  width: 30%;
  padding: 10px;
  padding-left: 0px;
  padding-bottom: 15px;
  float: left;
}

dt {
  border-bottom: 1px dashed #6cc6c4;
  padding-bottom: 6px;
  padding-top: 6px;
}

/*ここから質問と回答コーナー*/
#qa-wrapper {
  text-align: center;
  margin-top: 2.3em;
}

.qablock {
  position: relative;
  margin-bottom: 20px;
  background: #eee9e6;
  border: 1px solid #eee9e6;
}

.question_txt {
  cursor: pointer;
  padding: 5px;
  margin: 10px !important;
  border-bottom: 1px dashed #6f5436;
}

.qicon {
  color: #6f5436;
  font-weight: bold;
  margin-right: 15px;
}

.answer_txt {
  margin: 0px;
  padding: 20px 15px;
  width: 100%;
  display: none;
}

.question_txt::after {
  content: "▼" !important;
  width: 43px;
  height: 43px;
  line-height: 43px;
  top: 0;
  left: 0;
  margin-left: 5px;
  margin-top: 0px;
  color: #b4ada9;
}

.question_txt2::after {
  content: "△" !important;
  color: #b4ada9;
}

.qa {
  background: #fff;
  overflow: hidden;
}

.show {
  display: block;
}

.page-top {
  display: none;
  position: fixed;
  right: 10px;
  bottom: 10px;
}

.open {
  display: block;
}

/*ここからfooter*/
footer {
  background-color: #6cc6c4;
  border-radius: 5px;
  color: white;
}

footer ul {
  display: flex;
}

footer li {
  justify-content: space-around;
  margin-right: auto;
  margin-left: auto;
}

//アクセスページのアコーディオン
window.addEventListener("DOMContentLoaded", () => {
  const qa = document.querySelectorAll(".question_txt");

  for (let i = 0; i < qa.length; i++) {
    qa[i].addEventListener("click", Slide());
  }
  function Slide() {
    const toggle = document.querySelectorAll("#toggle");
    /*ここがエラーになってしまう */
    this.nextElementSibling.classList.toggle("show");
    this.classlist.toggle("question_txt2");
    const next_dd = this.nextSibling(toggle);

    if (next_dd.style.height !== "0px") {
      next_dd.style.height = "0px";
      next_dd.classlist.remove("question_txt2");
    }
  }
});

Comments

[kuma_kuma_]

大至急　企業、システムが判別できる文字はすぐ削除して別の文字に変更して下さい。

[ah6BD2gZo5GnjnT]

ご指摘ありがとうございます。

[kuma_kuma_]

いや直してくださいよ。

[ah6BD2gZo5GnjnT]

は？どこの部分なんですか？

[ah6BD2gZo5GnjnT]

企業名の含まれているリンクが問題だと思ってそれを消したんですが違うんですかね

[kuma_kuma_]

文章で色々かかれてるでしょ(<dd>タグの中)？それで検索したらどこのシステム(HP)か判断つきますよね？
それに質問として内部処理について書かれることは「システムの情報漏えい」になります。
ここの内容は数時間もしない内にGoogleに拾われます。
質問者様が質問したサイトはここでこんな仕組みで動いている。と公表しているのと同じ。
さらにシステム内部の情報まで漏らしているので「悪意」がある人が知ったら
HPの改変等行われかねません。

質問するなら「...のみの対応となります...」等を「xxxxxxxxx」とかにして下さい。
これお客様に知れたら大変なことになりますよ？

[ah6BD2gZo5GnjnT]

これ全て、私が考案したダミーテキストなんですが、アウトなんですね。
丁寧にありがとう御座います。

[kuma_kuma_]

(いやもう特定したけど...)

Answer 1

まずHTMLコードの間違い。

<section id =qanda>

      <div id =qa-wrapper>

idの名前は引用符(" または ')で囲む。上記は下記のように修正。

<section id="qanda">

      <div id="qa-wrapper">

id="toggle"が複数あるが、id名に重複は許されない。これは不要なので削除する。

JavaScriptコードの間違い。

    qa[i].addEventListener("click", Slide());

Slide() の括弧は不要。()があると関数の戻り値を渡すことになる。

    qa[i].addEventListener("click", Slide);

function Slide()内のコードは意味不明。
自分が書くなら下記のようにします。

//アクセスページのアコーディオン
  function Slide() {
    for (let i = 0; i < qa.length; i++) {
      if(this == qa[i]){
        qa[i].classList.toggle("question_txt2");
      }else{
        qa[i].classList.remove("question_txt2");  
      }
    }  
  }

CSSに下記を追加すれば、上記のコードで開閉できます。

.question_txt2 + dd {
  display: block;  
}

CodePenサンプル

Comments

[ah6BD2gZo5GnjnT]

ありがとうございました。