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

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

ただいまの
回答率

87.61%

javascriptでクラス属性のついているテキストを変更する方法

解決済

回答 2

投稿

  • 評価
  • クリップ 0
  • VIEW 570

score 18

前提・実現したいこと

javascriptの命令で、あるidがついているdivを押した際に同じ名前のクラス属性のうちの一つのテキストを変えたいと考えています。
しかし、様々な方法を試しましたが、どのようにしたらいいのかが全くわからないです。

発生している問題・エラーメッセージ

main.js:22 Uncaught TypeError: Cannot read property 'children' of undefined
    at HTMLDivElement.

該当のソースコード

'use strict'

{
  var win = document.getElementById("win");
  var lose = document.getElementById("lose");
  var percent = document.getElementsByClassName("percent");
  var select = document.getElementsByClassName("select");
  var reset = document.getElementById("reset");
  var score = document.getElementsByClassName("score");
  var win_per = document.getElementsByClassName("win_per");


  var counter1 = 0;
  var counter2 = 0;
  var winPer = 0;

  function winnerbtn(){
    var i;
    for (i = 0 ; i < select.length ; i++){
      select[i].children[0].addEventListener("click", ()=> {
        counter1 ++;
        score[i].children[0].textContent = counter1;
        winPer = counter1 / (counter1 + counter2) * 100;
        win_per[i].children[1].textContent = `${win_per.toFixed(2)}%`;
      })
    }
  }

  winnerbtn();

  lose.addEventListener("click", () => {
    counter2 += 1;
    loser.textContent = counter2
    win_per = counter1 / (counter1 + counter2) * 100;
    percent.textContent = `${win_per.toFixed(2)}%`;
  })

  reset.addEventListener("click", () => {
    counter1 = 0;
    counter2 = 0;
    winner.textContent = 0;
    loser.textContent = 0;
    percent.textContent = "0%";
  })
}
<!DOCTYPE html>
<html lang="en" dir="ltr">
  <head>
    <meta charset="utf-8">
    <title>戦歴管理</title>
    <link rel = "stylesheet" href = "css/styles.css">
  </head>
  <body>
    <div class = "container">
      <h1><img src= "img/main_logo.png" width = 500></h1>
      <h1>戦歴管理表</h1>
      <div class = "dekki">
        <p>デッキ名:占いミネア</p>
        <a href = "" class = "btn">デッキをみる</a>
      </div>
      <div class = "characters">
        <div class = "character">
          <div class = "icon">
            <div class = "photo"><img src = "img/sword.png"></div>
            <p>テリー</p>
          </div>
          <div class = "point">
            <div class = "score">
              <div class = "box1 winner" data-fig1 = "0">0</div>
              <div class = "box1"></div>
              <div class = "box1 loser"data-fig2 = "0">0</div>
              <div class = "box1"></div>
            </div>
            <div class = "win_per">
              <div class = "box2">勝率</div>
              <div class = "box2 percent">0%</div>
            </div>
            <div class = "select">
              <div class = "S-btn" id = "win">勝ち</div>
              <div class = "S-btn" id = "lose">負け</div>
              <div class = "S-btn" id = "reset">リセット</div>
            </div>
          </div>
        </div>
        <div class = "character">
          <div class = "icon">
            <div class = "photo"><img src = "img/wizard.png"></div>
            <p>ゼシカ</p>
          </div>
          <div class = "point">
            <div class = "score">
              <div class = "box1 winner" data-fig1 = "1">0</div>
              <div class = "box1"></div>
              <div class = "box1 loser" data-fig2 = "1">0</div>
              <div class = "box1"></div>
            </div>
            <div class = "win_per">
              <div class = "box2">勝率</div>
              <div class = "box2 percent">0%</div>
            </div>
            <div class = "select">
              <div class = "S-btn" id = "win">勝ち</div>
              <div class = "S-btn" id = "lose">負け</div>
              <div class = "S-btn" id = "reset">リセット</div>
            </div>
          </div>
        </div>
      </div>

      <script src="js/main.js"></script>
  </body>
</html>

試したこと

data属性を足してみた・・・今回の形ではいらないのではないかと感じた
ドットインストールのスロットマシンが一番近いと思い、動画を視聴
・・・あまりわからなかった。

補足情報(FW/ツールのバージョンなど)

ここにより詳細な情報を記載してください。

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 2

checkベストアンサー

+3

.select.win_perの数が合わないのが原因では?

    for (i = 0 ; i < select.length ; i++){                              // select.length > win_per.length の時に
      select[i].children[0].addEventListener("click", ()=> {
        counter1 ++;
        score[i].children[0].textContent = counter1;
        winPer = counter1 / (counter1 + counter2) * 100;
        win_per[i].children[1].textContent = `${win_per.toFixed(2)}%`;  // 添え字の範囲を i が超える場合がある
訂正

すみません、よく見ると違いますね。
このループカウンタiがグローバル変数なので、イベントが呼ばれるときにはselect.lengthと同じ値になっているはずです。これはselectの添え字の範囲を超えるのでundefinedが返ります。

解決方法は、

    for (let i = 0 ; i < select.length ; i++){


のようにブロックスコープを使うのが早いかと思います。

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2019/03/27 15:13

    あ、いや、違うか。
    スコープの問題ですね。訂正します。

    キャンセル

  • 2019/03/27 17:34

    ありがとうございます!
    複雑なので、なんとなくですが、わかりました!

    一番詳しく答えてくれたので、ベストアンサーにします!

    キャンセル

+3

  function winnerbtn(){
//    var i;
    for (let i = 0 ; i < select.length ; i++){

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

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

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

関連した質問

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