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

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

ただいまの
回答率

87.60%

初めてのクイズゲーム、処理が遅いです。

解決済

回答 1

投稿

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

score 13

前提・実現したいこと

JavaScriptで初めてクイズゲームを作りました。
動いてはくれたのですが、動作が重くて(?)10問ほどで遅くなってしまいます。

実現したい挙動は
「配列からランダムに問題文を取り出して表示→回答→答えと回答が一致しているなら次の問題へ、していないならもう一度その問題→全問答えたらもう一度シャッフルして第一問から」
です。

クイズに答える度に関数 mondai() が2回実行されている(?)と思い、関数を一度だけ実行する方法を調べて試してみたのですが、上手くいきませんでした。

どうすれば速い処理ができるでしょうか?
加えて、その他の良くない点についても指摘していたただけると幸いです。

勉強を始めて数日の初学者故、見当違いな質問をしているかもしれませんが、どうかお力添えお願い致します。

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

エラーメッセージ

該当のソースコード

<!DOCTYPE HTML>

<html lang="japa">
<head>
  <meta charset="utf-8">
  <script type="text/javascript" src="https://ajax.googleapis.com/ajax/libs/jquery/3.3.1/jquery.min.js"></script>
  <title>練習</title>
  <audio src="correct1.mp3" id="correct1"></audio>
  <audio src="incorrect1.mp3" id="incorrect1"></audio>
  <style>
  body{
    background-color: #222;
  }

  #question{
    margin:  100px auto 70px;
    background: green;
    width: 600px;
    height: 174px;
    border-radius: 0 12px 12px 12px;
    position: relative;
    box-shadow: 1px 1px 2px 0px #3f1,-1px -1px 1px 0px #3f1, 0px 0px 500px 0px #3f1;
  }

  #question_number{
    color: #fff;
    position: absolute;
    background-color: green;
    height: 45px;
    width: 150px;
    top: -45px;
    border-radius: 12px 12px 0 0;
    text-align: center;
    font-size: 23px;
    text-shadow: 0 0 50px #3f1, 0 0 30px #3f1;
    box-shadow: 1px -1px 2px 0px #3f1, -1px -1px 2px 0px #3f1;
    line-height: 45px;
    vertical-align: middle;
  }

  #sentence{
    color: #fff;
    position: absolute;
    top: 50%;
    left: 50%;
    width: 540px;
    transform: translate(-50%,-50%);
    font-size: 17.2px;
    text-align: center;
  }

  #answer{
    margin-left: 50%;
    transform: translateX(-50%);
    background: green;
    width: 400px;
    height: 80px;
    box-shadow: 0px 0px 500px 0px #3f1;
    line-height: 80px;
    vertical-align: middle;
    font-size: 18px;
    text-align: center;
  }



  #button{
    margin: 20px auto;
    background: linear-gradient(#6795fd 0%, #67ceff 100%);
    border-bottom: solid 5px #5e7fca;
    border-radius: 4px;
    width: 180px;
    height: 50px;
    line-height: 50px;
    vertical-align:middle;
    text-align: center;
    font-size: 26px;
    cursor: pointer;
    color: #222;
  }

  #button:active{
    transform: translateY(5px);
    border-bottom: none;
  }

  #button:active + #attention{
    transform: translateY(5px);
  }

  #attention{
    color: #bbb;
    font-size: 10px;
  }
  </style>
</head>
<body>

  <div id="question">
    <div id="question_number"><span id="num"></span></div>
    <div id="sentence">
    </div>
  </div>
  <input type="text" id="answer">
  <div id="button">OK</div>

  <div id="attention">漢字は正しく入力してください。</div>

  <script>
  //問題文と答えの2次元配列↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓
  var que = [
    ["「天地創造」「最後の審判」の作者を答えよ。","ミケランジェロ",],
    ["マキァヴェリの著作において、「君主論」の他に何があるか。","ローマ史論"],
    ["「聖フランチェスコの生涯」の作者を答えよ。","ジョット"],
    ["サンタ=マリア大聖堂を完成させた人物を答えよ。","ブルネレスキ"],
    ["「太陽の都」の作者を答えよ。","カンパネラ"],
    ["トマス=モアの代表作を答えよ。","ユートピア"],
    ["「カンタベリ物語」の著者を答えよ。","チョーサー"],
    ["チョーサーの代表作を答えよ。","カンタベリ物語"],
    ["ドン・キホーテの作者が参加した戦争をなんというか。","レパントの海戦"],
    ["西欧人にとっての伝説のキリスト教国を支配していたとされるのは誰か。","プレスター=ジョン"],
    ["1498年カリカットに到達したポルトガル人を答えよ。","ヴァスコ=ダ=ガマ"],
    ["ニューファンドランドを探検したイタリアの探検家を答えよ。","カボット"],
    ["パナマ地峡を横断したスペインの探検家を答えよ。","バルボア"],
    ["史上2番目に世界周航を達成した人物を答えよ","ドレーク"],
    ["スペインのフィリピン経営の拠点としてマニラを建設した人物を答えよ。","レガスピ"],
    ["大航海時代、スペイン領とポルトガル領を分けた条約を答えよ","トルデシリャス条約"],
    ["トルデシリャス条約締結の一年前、教皇子午線を設置した人物を答えよ。","アレクサンデル6世"],
    ["大航海時代、西欧諸国が主に用いた船舶の種類を答えよ。","ガレオン船"],
    ["プランテーションはラテンアメリカでは何という名で普及したか。","アシエンダ制"],
    ["新大陸の発見後に栄えたスペインの開港都市を答えよ。","セビリャ"],
    ["宗教改革を引き起こした免罪符の発行をした人物を答えよ。","レオ10世"],
    ["ルターの代表的著作を答えよ。","キリスト者の自由"],
  ];
  //↑↑↑↑↑↑↑↑↑↑↑↑↑↑↑↑↑↑↑↑↑↑↑↑↑↑↑↑↑↑↑↑↑↑↑↑↑↑↑↑↑


  //シャッフルの関数↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓
  function shuffle(array) {
    var n = array.length, t, i;

    while (n) {
      i = Math.floor(Math.random() * n--);
      t = array[n];
      array[n] = array[i];
      array[i] = t;
    }

    return array;
  }
  //↑↑↑↑↑↑↑↑↑↑↑↑↑↑↑↑↑↑↑↑↑↑↑↑↑↑↑↑↑↑↑↑↑↑↑↑↑↑↑↑


  //メインの処理↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓
  $(function(){

    var count = 1;
    var mondai = function(){
      if (count < que.length+1) {

        console.log(count);

        $("#answer").focus();
        $("#num").text(count);
        $("#sentence").text(que[count-1][0]);

        $("#answer").keypress(function(){
          var enter = event.keyCode;
          if(enter === 13){
            var $a = $("#answer").val();
            $("#answer").val("");
            if ($a === que[count-1][1]) {
              $("#correct1")[0].play();
              count += 1;
            }
              mondai();
          }
        });

        $("#button").click(function(){
          var $a = $("#answer").val();
          $("#answer").val("");
          if ($a === que[count-1][1]) {
            $("#correct1")[0].play();
            count += 1;
          }
          mondai();
        });

      } else {
        location.reload();
      }
    }
    shuffle(que);
      mondai();
  });

  //↑↑↑↑↑↑↑↑↑↑↑↑↑↑↑↑↑↑↑↑↑↑↑↑↑↑↑↑↑↑↑↑↑↑↑↑↑↑↑↑
  </script>
</body>
</html>

試したこと

関数の位置を変えたり、$(document).ready(function){}で囲ってみたりしました。

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

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

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 1

checkベストアンサー

+3

何度も使う関数mondai内でクリックイベントなどイベントを登録しているからです。同じ要素にイベントが複数重ねて登録されることになり、処理が遅くなります。

毎回イベント解除するか、$("#answer").keypress(/*略*/); $("#button").click(/*略*/); を関数mondaiの外に置くようにしてみてください。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2019/04/01 11:03

    返信が遅れてしまい、申し訳ありません。

    おかげ様で、解決致しました。
    これほどまで単純明快に導いてくださり、感動すら覚えています。

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

    キャンセル

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

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

関連した質問

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