タイピングゲームJS エラー

Question

以下はタイピングゲームのJSですが、エラーが発生してしまいます。
どう直せばよいかおしえてください。

なお、teratail内に似た質問があったのですが、
game.js:22 Uncaught TypeError: Cannot read property '0' of undefinedとあり、私の質問するコード、
game.js:22 Uncaught TypeError: Cannot read property 'btn' of undefinedとは異なっていました。
OとBtnの違いなのでBtnについて教えていただければわかるかもしれません。

Cannot set property 'textContent' of nullエラーに関して
Uncaught TypeError: Cannot read property '0' of undefinedエラーに関して

エラー
Uncaught TypeError: Cannot set property 'textContent' of null
→エラーは　timer.textContent = '制限時間：' + -TIME- + '秒';　にあるようです。

game.js:22 Uncaught TypeError: Cannot read property 'btn' of undefined
→エラーは　const countdown = setInterval(function() {　にあるようです。

const subject = document.getElementById('subject');
const timer = document.getElementById('timer');
const form = document.forms.typing;
const textList = [
  'Hello World',
  'Good',
  'I love JavaScript',
  'This is MyApp',
  'Welcome'
];
 
let TIME = 20;
let count = 0;
let state = true;
 
const countdown = setInterval(function() {
  timer.textContent = '制限時間：' + -TIME- + '秒';
  if(TIME <= 0) finish();
}, 1000);
 
 
form.btn.addEventListener('click', function(e) {
  if(!state) return;
 
  if(form.input.value === subject.textContent) {
    count++;
    init();
  } else {
    subject.textContent = '間違いです！';
    setTimeout(function(){ init() },1000)
  }
});
 
init();
 
function init() {
  const rnd = Math.floor(Math.random() * textList.length);
 
  subject.textContent = textList[rnd];
  form.input.value = '';
  form.input.focus();
}
 
function finish() {
  clearInterval(countdown);
  subject.textContent = '正解数は' + count + '個でした！';
  state = false;
}

Comments

[BluOxy]

エラーの内容は読まれましたか。
それとも読んでもよく分からないから書いてあることの意味を知りたいということでしょうか。
それともエラーに興味はないからとにかく直し方を教えてほしいということでしょうか。

[退会済みユーザー]

エラーの意味が分かりません。直し方のみ教えていただいてもうれしいのですが、自分でも把握しておきたいのでエラーの意味を教えていただければ思います。よろしくお願いします。

Answer 1

form.btnで、formがundefinedとなっているためにプロパティが取得できないものと思われます。

そして、document.formsからプロパティ参照で要素を探すのは古いやり方です。特定の場所と決まっているのであれば、idを振ってdocument.getElementByIdで取得しましょう。

Comments

[maisumakun]

> Btnについて教えていただければわかるかもしれません。

これはフォームに対して参照しているプロパティですので、「name="btn"」となった要素と結びつくものです。特別な「意味」などありません。

[退会済みユーザー]

ありがとうございます！
このサイト（https://www.sejuku.net/blog/92667）から抜粋して試したのですが、うまくいかなかったのでどうにかできないものかと思っておりましたので。というのも、タイピングゲーム作成中でこのサイトのものが自分の理想に合うと思っていました。回答を参考にもう少し書き換えてみます。

[退会済みユーザー]

＞document.formsからプロパティ参照で要素を探すのは古いやり方です。特定の場所と決まっているのであれば、idを振ってdocument.getElementByIdで取得しましょう。

こちらはアドバイス通り直してみました。

＞form.btnで、formがundefinedとなっているためにプロパティが取得できないものと思われます。

こちらはどのように直したらいいか教えていただけますか？JS初心者のため、詳しくお聞かせ願いたいです。すみませんが、よろしくお願いします。

[maisumakun]

> こちらはどのように直したらいいか教えていただけますか？

「アドバイス通り直し」たのであれば、form.btnは使わないコードとなっているのではないかと思われるのですが…

[退会済みユーザー]

どういうことでしょうか。
現在、JSは以下のような感じです。
const subject = document.getElementById('subject');
const timer = document.getElementById('timer');
const form = document.getElementById.typing;
const textList = [
'Hello World',
'Good',
'I love JavaScript',
'This is MyApp',
'Welcome'
];

let TIME = 20;
let count = 0;
let state = true;

const countdown = setInterval(function() {
timer.textContent = '制限時間：' + -TIME- + '秒';
if(TIME <= 0) finish();
}, 1000);

form.btn.addEventListener('click', function(e) {
if(!state) return;

if(form.input.value === subject.textContent) {
count++;
init();
} else {
subject.textContent = '間違いです！';
setTimeout(function(){ init() },1000)
}
});

[maisumakun]

「document.getElementById.typing」という書き方がまず存在しません。

コードは見様見真似ではなく、理屈を把握した上で書き進めてください。

[退会済みユーザー]

私の早とちりだったようで、すみません。

[maisumakun]

「form.btn」に相当する要素も、getElementByIdなど別な手段で取得するのが適切、ということです。

Answer 2

前回を踏襲して・・・

<script>
var r=[
  ['00:05.00','abc'],
  ['00:08.00','xyz'],
  ['00:10.00','ijk'],
  ['00:15.00','aaa'],
  ['00:20.00','zzz'],
  ];
const qs=[
  'Hello World',
  'Good',
  'I love JavaScript',
  'This is MyApp',
  'Welcome'
  ];

window.addEventListener('DOMContentLoaded',()=>{
  document.querySelector('#start').addEventListener('click',e=>{
    const t1=e.target;
    const t2=document.querySelector('#a');
    t1.disabled=true;
    t2.disabled=false;
    const name=prompt("your name?");
    if(name=="") name="noname";
    (async()=>{
      const qStr=qs[parseInt(Math.random()*qs.length)];
      const s=new Date().getTime();
      document.querySelector('#q span').textContent=qStr;
      t2.value="";
      t2.focus();
      await new Promise(resolve=>{
        const timerId=setInterval(()=>{
          if(t2.value==qStr){
            clearInterval(timerId);
            resolve();
          }
        },10);
      });
      const e=new Date().getTime();
      const diff=e-s;
      const str=(10000+diff/1000).toString();
      const time=str.substr(1,2)+":"+
               str.substr(3,5);
      r.push([time,name]);
      r=await r.sort((x,y)=>x[0]>y[0]).splice(0,5);
      alert(r.join("\n"));
      t1.disabled=false;
      t2.disabled=true;
    })();
  });
});

</script>
<input type="button" value="start" id="start">
<div id="q">Q:<span></span></div>
A:<input type="text" id="a">

改良版

• 問題をシャッフルして順番に表示
• ランキングをいつも表示

<script>
var r=[
  ['00:30.00','abc'],
  ['01:00.00','xyz'],
  ['01:30.00','ijk'],
  ['02:30.00','aaa'],
  ['03:00.00','zzz'],
  ];
var qs=[
  'Hello World',
  'Good',
  'I love JavaScript',
  'This is MyApp',
  'Welcome'
  ];
const viewRank=()=>{
  document.querySelector('#rank span').innerHTML=r.map(x=>x.join(" ")).join("<br>");
};
window.addEventListener('DOMContentLoaded',()=>{
  viewRank();
  document.querySelector('#start').addEventListener('click',e=>{
    const t1=e.target;
    const t2=document.querySelector('#a');
    t1.disabled=true;
    t2.disabled=false;
    const name=prompt("your name?");
    if(name=="") name="noname";
    (async()=>{
      qs=qs.map(x=>[x,Math.random()]).sort((x,y)=>x[1]-y[1]).map(x=>x[0]);
      const s=new Date().getTime();
      for(var i=0;i<qs.length;i++){
        const qStr=qs[i];
        document.querySelector('#q span').textContent=qStr;
        t2.value="";
        t2.focus();
        await new Promise(resolve=>{
          const timerId=setInterval(()=>{
            if(t2.value==qStr){
              clearInterval(timerId);
              resolve();
            }
          },10);
        });
      }
      const e=new Date().getTime();
      const diff=e-s;
      const str=(10000+diff/1000).toString();
      const time=str.substr(1,2)+":"+
               str.substr(3,5);
      r.push([time,name]);
      //r=await r.sort((x,y)=>x[0]>y[0]).splice(0,5);
      //以下修正版
      r=await r.sort((x,y)=>x[0]>y[0]?1:-1).splice(0,5);
      viewRank();
      t1.disabled=false;
      t2.disabled=true;
    })();
  });
});

</script>
<input type="button" value="start" id="start">
<div id="q">Q:<span></span></div>
<div>A:<input type="text" id="a"></div>
<div id="rank">rank:<br>
<span></span></div>

Comments

[退会済みユーザー]

ありがとうございます！
しかし、追加で質問なのですが、
この状態ですと、1問しか問われないのですが、全問問うように書き換えるにはどうしたらいいか教えていただけますか？
また、順位が変化しないのですが、どのように書き換えればいいかも添えて教えていただければと思います。

[退会済みユーザー]

>この状態ですと、1問しか問われないのですが、全問問うように書き換えるにはどうしたらいいか教えていただけますか？

こちらは解決しました！１問にすべての文字列を入力します。

[yambejp]

あれ？順位変化しませんか？
分＋秒＋少数点以下なので、デフォルト順位のダミーを調整してみてください

[yambejp]

> １問にすべての文字列を入力します

入力がマッチした時点で次の文字をQに表示させる方がいいようなきがしますが・・・

[退会済みユーザー]

調整してみました。極端にわかるように最後のzzzのタイムを30:00.00にして試行していますが、順位は切り替わらないままです。

[退会済みユーザー]

>入力がマッチした時点で次の文字をQに表示させる方がいいようなきがしますが・・・

できればそちらのほうが良いですが、難しそうなのでまた別の機会に挑戦してみようと思います。
（気付くのが遅くなってしましすみません。
　Reloadの時間差で17:10のコメントだけ見えていませんでした。）

[yambejp]

ざっくり改良しときました

[退会済みユーザー]

やはりできません。サーバー上の設定など必要でしょうか。

[yambejp]

ごめんなさい
sortの挙動がブラウザによって異なっていました
修正箇所指摘してあるので見直してください

[退会済みユーザー]

できました！ここまでサポートしていただきありがとうございました！
※※※※※
詳細
・サーバー上の設定（パーミッション設定）が必要
・上記回答の改良版を使用
でうまくいきます。

[退会済みユーザー]

問題をシャッフルさせずに出題する方法ってありませんか？