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

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

ただいまの
回答率

87.61%

このエラーについて教えてください。

解決済

回答 4

投稿

  • 評価
  • クリップ 2
  • VIEW 826

score 17

let controllers = document.querySelectorAll(".controller");
controllers[0].addEventListener(
  "click",
  function() {
    controllers.classList.add(".current");  
// error   Cannot read property 'add' of undefined
  },
  controllers[1].addEventListener(
    "click",
    function() {
      console.log("hghg2");
    },
    controllers[2].addEventListener("click", function() {
      console.log("hghg3");
    })
  )
);

controllers[0].addEventListener(
"click",
function() {
controllers.classList.add(".current");  
// error   Cannot read property 'add' of undefined
},
この記述でクラスを付与しようとした時に、このようなエラーがでました。
これの解決の仕方がわからないのでご教授お願いします。

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 4

checkベストアンサー

+5

controllersに入っているのはquerySelectorAllの戻り値なので、ElementではなくNodeListです。

Element の querySelectorAll() メソッドは、与えられた CSS セレクターに一致する文書中の要素のリストを示す静的な (生きていない) NodeList を返します。
Document.querySelectorAll() | MDN

ですから、controllers[0].classList.add("current")といった感じで、配列のように添え字をつけるなどして扱ってください。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2019/01/29 15:18

    回答いただきありがとうございます!
    無事、クラス付与することができました。

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

    キャンセル

+4

controllers.classList.add(".current");

controllersは配列なので、classListというものはありません。
そのため、無いものにaddなんて無いぞってエラーになるわけです。

Elementを指定する必要があるので、添字をつけるなどが必要です。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

0

https://qiita.com/takeharu/items/d75f96f81ff83680013f

addなんてしらんぞこのやろーって怒られてます。

Listに値を追加するのはJavaならaddですが、
javascriptならpushじゃなかったですっけ?


あ~なんか違いますね、普通のListじゃないんですね~よく読んでませんでした。

https://developer.mozilla.org/ja/docs/Web/API/Element/classList

例をみると".クラス名" じゃなくて "クラス名" として入れてますが
それでもエラーは出たままですか?


他の方も回答してくれてますね。
確かに配列から.classListをとってそこに値を追加してくれ~なんて無理な話なので、
for文回したり、値を付けたい要素を指定して.add()を使わないといけないっぽいですね
console.log()などで中身を見てみるとわかりやすいですよ

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2019/01/29 15:12

    回答いただきありがとうございます!
    一応参考にしたサイトのURLを貼らせていただきます。
    ▽URL
    http://shanabrian.com/web/javascript/add-class.php

    javascriptもaddでいけるはずらしいのですが、、、
    初心者なので知識がまだついてないのでお願いします。

    キャンセル

  • 2019/01/29 15:14

    最初”クラス名”でエラーがでてしまい、”.クラス名”でやり直したところ同じエラーで怒られました。

    キャンセル

  • 2019/01/29 15:21

    教えていただきありがとうございます。
    今回から、活用してみます!

    キャンセル

  • 2019/01/29 15:22

    解決したようで何よりです

    キャンセル

0

> controllers[0].addEventListener(
"click",
function() {
controllers.classList.add(".current");  
// error   Cannot read property 'add' of undefined
},

controllersはNodeListですよね?
そもそもclassListを持っていないのでは?
それとクラスを追加するときにピリオド付きのクラス名は微妙・・・

<script>
window.addEventListener('DOMContentLoaded', function(e){
  var controllers = document.querySelectorAll(".controller");
  controllers[0].addEventListener("click",function() {
    [].forEach.call(controllers,function(x){
      x.classList.add("current");
    });
  });
});
</script>

<div class="controller">c1</div>
<div class="controller">c2</div>
<div class="controller">c3</div>

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

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

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

関連した質問

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