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

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

ただいまの
回答率

87.77%

innerHTMLで表示されるテキストをvar txt1,var txt2 …のように定義してonclickで切り替えたい

解決済

回答 2

投稿

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

score 9

前提・実現したいこと

全くの初学者です。
学習歴はjavascript、html5、css関連の入門書を5冊ほど
通読、載っているコードを全て写経した程度です。
現在ブラウザ上で動く、onclickで画像とテキストが切り変わっていく
プログラムを構成しようとしているのですが
innerhtmlテキストの切り替えが出来ず、頓挫しております。
お手数ですが、どなたかご教授願えませんでしょうか。

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

Uncaught TypeError: Cannot set property 'onclick' of null
at scenario.js:10
Uncaught ReferenceError: txt is not defined
at onload (scenario.js:7)

該当のソースコード

<!DOCTYPE html>
<html lang="ja" dir="ltr">
<head>
    <meta charset="utf-8">
    <title>sample</title>
    <link rel="stylesheet" href="canvas.css">
    <script src="scenario.js" charset="UTF-8"></script>
    <script src="clickpage.js" charset="UTF-8"></script>
</head>
<body>
    <!--ビジュアル表示枠を配置します-->
    <div id="imagebox"></div>
    <!--メッセージ表示枠を配置します-->
    <div id="messagebox"></div>
    <!--最初に戻る用のボタンです-->
    <button id="init">最初に戻る</button>
</body>
</html>
#imagebox{
    width:480px;
    height:270px;
    border-style:solid;
    border-color:#696969;
    border-width:2px;
    background-color:#DCDCDC;
  }

#messagebox{
    /*文字間隔です*/
    letter-spacing:1px;
    /*行の高さです*/
    line-height:27px;
    width:480px;
    height:135px;
    border-style:solid;
    border-color:#696969;
    border-width:2px;
    /*フォントの太さです*/
    font-weight:bold;
    font-size:26px;
    color:#ffffff;
    background-color:#DCDCDC;
  }
var txt0="最初に表示するテキスト";
var txt1="2番目に表示するテキスト"
var txt2="―了―"

window.onload = function onload(){
    target = document.getElementById("messagebox");
    target.innerHTML = txt++;
    }

document.getElementById("messagebox").onclick = function(){
    document.getElementById("messagebox").innerHTML = txt++;
} 

試したこと

target.innerHTML = テスト1; のように
1行だけの表示であれば問題無くmessagebox内に表示されます。

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

環境はvidual Studio Code でコードを記載後、Chrome上で確認を行っています。
上記コードはvidual Studio Code 上では警告は出ていません。

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 2

+3

Uncaught TypeError: Cannot set property 'onclick' of nullのほうは、onloadの外側でイベント定義をやろうとした結果、まだエレメントが存在しない状態で実行されてしまった結果です。

Uncaught ReferenceError: txt is not definedはメッセージのとおりで、txtという変数が存在しません。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2019/12/09 15:07

    エラー内容に関するご回答を頂き、誠にありがとうございます!

    キャンセル

checkベストアンサー

+2

Sample

<!DOCTYPE html>
<html lang="ja" dir="ltr">
<head>
    <meta charset="utf-8">
    <title>sample</title>
    <style>
        #messagebox {
            /*文字間隔です*/
            letter-spacing: 1px;
            /*行の高さです*/
            line-height: 27px;
            width: 480px;
            height: 135px;
            border-style: solid;
            border-color: #696969;
            border-width: 2px;
            /*フォントの太さです*/
            font-weight: bold;
            font-size: 26px;
            color: #ffffff;
            background-color: #DCDCDC;
        }

    </style>
</head>
<body>
<!--メッセージ表示枠を配置します-->
<div id="messagebox"></div>
<!--最初に戻る用のボタンです-->
<button id="init">最初に戻る</button>
<script>

    let num = -1;
    let messages = [
        "最初に表示するテキスト",
        "2番目に表示するテキスト",
        "―了―"
    ];

    window.onload = function onload() {
        let target = document.getElementById("messagebox")
        target.addEventListener('click', function () {
            num = (num + 1) % messages.length;
            this.innerText = messages[num];
        });
        target.click();

        let init = document.getElementById('init');
        init.addEventListener('click', function () {
            target.innerText = messages[0];
        })
    };
</script>
</body>
</html>

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2019/12/09 15:12

    ご回答頂き、誠にありがとうございます!
    いただきましたHTMLをコピーさせていただいたところ
    希望通りの動作が実現できました。
    これから頂きましたコードを1行ずつ紐解き
    自分の知識と出来るよう、勉強させていただきます。
    このたびは誠にありがとうございました。

    キャンセル

  • 2019/12/09 18:11

    Kosuke_Shibuya様
    大変申し訳ございません。
    いただきましたサンプルコードの内

    num = (num + 1) % messages.length;

    こちらの行が全体の中でどのように作用しているのか
    理解できませんでした。
    % は剰余演算子で messages.length は messages の文字数でしょうか。
    重ね重ね申し訳ございませんが
    ご教授いただけましたら非常に助かります。
    よろしくお願いいたします。

    キャンセル

  • 2019/12/09 18:13 編集

    messagesは配列です。
    messages.lengthは配列のアイテム数

    num = (num + 1) % messages.length;
    アイテム数が3個なので、4回目にボタンを押した時に、値を0に戻す処理。

    キャンセル

  • 2019/12/09 18:27

    messagesは配列でしたか・・・・・・お恥ずかしい限りです。
    ご説明頂きまして、ありがとうございます。
    連続してクリックしたときに最初へ戻る処理と
    最初へ戻るボタンを押してのinitで
    最初へ戻る処理の2つが実装されているのですね。
    理解できました。
    手元の書籍でもう一度配列の箇所を読み込もうと思います。

    何度もご返答いただきまして、感謝いたします!

    キャンセル

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

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

関連した質問

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