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

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

ただいまの
回答率

90.12%

JavaScriptのチュートリアルが動きません

解決済

回答 5

投稿 編集

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

Ykkykk

score 95

こちらのサイトにあるJavascriptのチュートリアルを行っております。
一応スクリプトを見て書いたのですが、なぜか動きません。

var para = document.querySelector('p');
var count = 0;

// setup canvas

var canvas = document.querySelector('canvas');
var ctx = canvas.getContext('2d');

var width = canvas.width = window.innerWidth;
var height = canvas.height = window.innerHeight;

// function to generate random number

function random(min,max) {
  var num = Math.floor(Math.random()*(max-min)) + min;
  return num;
};

function Shape(x, y, velX, velY, exists) {
        this.x = x;
        this.y = y;
        this.velX = velX;
        this.velY = velY;
        this.exists = exists;
};

function Ball(x, y, velX, velY, exists, color, size) {
        Shape.call(this, x, y, velX, velY, exists);

        this.color = color;
        this.size = size;
};

Ball.prototype = Object.create(Shape.prototype);
Ball.prototype.constructor = Ball;

Ball.prototype.draw = function() {
        ctx.beginPath();
        ctx.fillStyle = this.color;
        ctx.arc(this.x, this.y, this.size, 0, 2 * Math.PI);
        ctx.fill();
};

Ball.prototype.update = function() {
        if ((this.x + this.size) >= width) {
                this.velX = -(this.velX);
        }

        if ((this.x - this.size) <= 0) {
                this.velX = -(this.velX);
        }

        if ((this.y + this.size) >= height) {
                this.velY = -(this.velY);
        }

        if ((this.y - this.size) <= 0) {
                this.velY = -(this.velY);
        }

        this.x += this.velX;
        this.y += this.velY;
};

Ball.prototype.collisionDetect = function() {
        for (var j = 0; j < balls.length; j++) {
                if (!(this === balls[j])) {
                        var dx = this.x - balls[j].x;
                        var dy = this.y - balls[j].y;
                        var distance = Math.sqrt(dx * dx + dy * dy);

                        if (distance < this.size + balls[j].size) {
                                balls[j].color = this.color = 'rgb(' + random(0, 255) + ',' + random(0, 255) + ',' + random(0, 255) + ')';
                        }
                }
        }
};

function EvilCircle(x, y, exists) {
        Shape.call(this, x, y, 20, 30, exists);

        this.color = 'white';
        this.size = 10;
}

EvilCircle.prototype = Object.create(Shape.prototype);
EvilCircle.prototype.constructor = EvilCircle;

EvilCircle.prototype.draw = function() {
        ctx.beginPath();
        ctx.strokeStyle = this.color;
        ctx.lineWidth = 3;
        ctx.arc(this.x, this.y, this.size, 0, 2 * Math.PI);
        ctx.stroke();
};

EvilCircle.prototype.checkBounds = function() {
        if ((this.x + this.size) >= width) {
                this.x -= this.size;
        }

        if ((this.x - this.size) <= 0) {
                this.x += this.size;
        }

        if ((this.y + this.size) >= height) {
                this.y -= this.size;
        }

        if ((this.y - this.size) <= 0) {
                this.y += this.size;
        }
};

EvilCircle.prototype.setControls = function() {
        var _this = this;
        window.onkeydown = function() {
                if (e.keyCode === 65) {
                        _this.x -= _this.velX;
                } else if (e.keyCode === 68) {
                        _this.x += _this.velX;
                } else if (e.keyCode === 87) {
                        _this.y -= _this.velY;
                } else if (e.keyCord === 83) {
                        _this.y += _this.velY;
                }
        };
};

EvilCircle.prototype.collisionDetect = function() {
        for (var j = 0; j < balls.length; j++) {
                if (balls[j].exists) {
                        var dx = this.x - balls[j].x;
                        var dy = this.y - balls[j].y;
                        var distance = Math.sqrt( dx * dx + dy * dy);

                        if (distance < this.size + balls[j].size) {
                                balls[j].exists = false;
                                count--;
                                para.textContent = 'Ball count: ' + count;
                        }
                }
        }
};


var balls = [];

function loop() {
        ctx.fillStyle = 'rgba(0, 0, 0, 0.25)';
        ctx.fillRect(0, 0, width, height)

        while (balls.length < 25) {
                var size = random(10,20);
                var ball = new Ball(
                        random(0 + size,width - size),
                        random(0 + size,height - size),
                        random(-7,7),
                        random(-7,7),
                        true,
                        'rgb(' + random(0,255) + ',' + random(0,255) + ',' + random(0,255) + ')',
                        size
                );
                balls.push(ball);
                count++;
                para.textContent = 'Ball count: ' + count;
        }

        for (var i = 0; i < balls.length; i++) {
                if (balls[i].exists) {
                        balls[i].draw();
                        balls[i].update();
                        balls[i].collisionDetect();
                }
        }

        evil.draw();
        evil.checkBounds();
        evil.collisionDetect();

        requestAnimationFrame(loop);
}

loop();


エラーとしては、166行目で
TypeErroe:para is null.
となっています。166行目は以下のコードです。

para.textContent = 'Ball count: ' + count;


こちらを読み込んでいるHTMLは以下のものになります。

<!DOCTYPE html>
<html>
  <head>
    <meta charset="utf-8">
    <title>Bouncing balls</title>
    <link rel="stylesheet" href="style.css">
  </head>

  <body>
    <h1>bouncing balls</h1>
    <canvas></canvas>

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


こちらのHTMLはチュートリアルで用意されているものをコピペしております。
これはなぜ動かないのでしょうか?最初の変数を定義する箇所が間違っているということでしょうか?
ご教示くださいますと幸いです。

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

質問への追記・修正、ベストアンサー選択の依頼

  • m.ts10806

    2018/08/28 10:26

    console.log() で変数を引数に指定してください。「JavaScript デバッグ」で検索したら沢山出てきますよ。

    キャンセル

  • m.ts10806

    2018/08/28 10:30

    なるほど。回答の通りですね。おそらく提示されているサイトはリファレンスでもあるのでquerySelectorなどの機能を確認してみると良いです。指定情報が存在しなかったときの「返り値」がnullであればそもそも本来返るべきエレメント情報ではないわけで、textContentなどのメソッドも存在しないため使えないということになります

    キャンセル

  • Ykkykk

    2018/08/28 10:37

    いろいろとご教示いただきありがとうございます。感謝しております。

    キャンセル

回答 5

checkベストアンサー

+3

まず、

var para = document.querySelector('p');


この最初にparaを設定している部分で<p>タグの内容を取得しています。
MDN web docs
提示のHTMLにはそもそも<p>タグがないので取得できずnullが返っているので、エラーが出ているということです。
チュートリアルそのままでは
MDNチュートリアル実行結果
となるはずなので、ボールをカウント⇒ボールの数を表示をしたいなら<p>タグを追加しないといけないですね。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2018/08/28 10:35

    ご回答いただきありがとうございます。
    quertSelectorについて理解できました。

    キャンセル

+3

html側にpタグが無いからでは?

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

+3

<p>タグを追加しましたか?

1.HTMLファイルに「Ball count:」というテキストを含む<h1>要素の直下に<p>要素を追加します。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

+2

var para = document.querySelector('p');

↑この1行は「<p>」タグの要素を取得するものです。

提示されているHTMLには「<p>」タグの要素が存在しません。

よって、

TypeErroe:para is null.

↑こうなるのです。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2018/08/28 10:31

    ご回答いただきありがとうございます。
    HTMLに空欄の<p>タグだけを書き加えるということになるのでしょうか?
    166行目が<p>タグを書き換えるのかと思っていたのですが、166行目はどのような役割を果たしているのでしょうか?

    キャンセル

  • この投稿は削除されました

+2

1行目、理解していますか?
var para = document.querySelector('p');
これはdocumentから<p></p>要素を探して、最初に見つけたひとつを取得します。
Document.querySelector()
見つからない場合、nullを返します。

ご提示のHTMLを見る限り、<p></p>要素は存在しません。
つまり変数paraにはnullが代入されます。

nullはプロパティを一切持ちません。
null.textContentは実行できないということです。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2018/08/28 10:34

    ご回答いただきありがとうございます。
    理解できました。

    キャンセル

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

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

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