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

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

ただいまの
回答率

88.78%

javascriptでルーレットを作っているのですがスタートを押さずにストップを押すと押した数だけ止まらないマス目が出てしまいます。

解決済

回答 3

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 1,238

osusi

score 15

<!DOCTYPE html>
<html lang="ja">
<head>
    <meta charset ="UTF-8">
    <title>ルーレット</title>
    <link rel = "stylesheet" href = "roulette.css">
    <script src="http://code.jquery.com/jquery-3.3.1.min.js"></script>
    <script>
     var roulette_id;
    var start_flag = false;
    var cell = [1,2,3,4,5,6,7,8,9,10,11,12,13,14,15,16];
    var num =1;
    var random;
    function random_start() {

        if ( start_flag === false) {
             roulette_id = setInterval(roulette_start, 100);
             start_flag =true;
        }
    }


    function roulette_start() {   
      $('#td' + num).css('backgroundColor', 'white');
        random = Math.floor(Math.random() * cell.length);
        num = cell[random];
      $('#td' + num).css('backgroundColor', 'red');
    }

    function random_stop() {
        clearInterval(roulette_id);
        start_flag = false;
        $('#td' + num).css('backgroundColor', 'blue');
        cell.splice(random, 1);
        num=0;
    }

    function random_reset() {
        start_flag =false;
        clearInterval(roulette_id);
        cell = [1,2,3,4,5,6,7,8,9,10,11,12,13,14,15,16];
        $("td").css('backgroundColor', 'white');
    }

    $(function() {
             $('#start').click(random_start);
            $('#stop').click(random_stop);
            $('#reset').click(random_reset);
});
    </script>
</head>
<body>

    <table>
        <tr>
            <td id="td1">1</td>
            <td id="td2">2</td>
            <td id="td3">3</td>
            <td id="td4">4</td>
        </tr>
        <tr>
            <td id="td5">5</td>
            <td id="td6">6</td>
            <td id="td7">7</td>
            <td id="td8">8</td>
        </tr>
        <tr>
            <td id="td9">9</td>
            <td id="td10">10</td>
            <td id="td11">11</td>
            <td id="td12">12</td>
        </tr>
        <tr>
            <td id="td13">13</td>
            <td id="td14">14</td>
            <td id="td15">15</td>
            <td id="td16">16</td>
        </tr>
    </table>
    <button id ="start">スタート</button>
    <button id ="stop">ストップ</button>
    <button id ="reset">リセット</button>
</body>

</html>

 前提・実現したいこと

プログラミング初心者です。16桁の数字でルーレットを作っているのですが、
スタートボタンを押さずに、ストップボタンを押すと、押した数だけ、止まらないマス目が残ります。
スタートボタンを押下後のストップ以外は反応しないようにしたいです。
できるだけソースコードはそのままでどこを追加すればいいのでしょうか。

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

該当のソースコード

試したこと

どこにどう追加したらいいのか意味がわからないです。どなたか
教えてください!よろしくお願いします。

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

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

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

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

  • kei344

    2019/05/25 11:35

    (質問文は編集できます)質問文のコードはコードブロックで囲んでいただけませんか? ```(バッククオート3つ)で囲み、前後に改行をいれるか、コードを選択して「<code>」ボタンを押すとコードブロックになります。

    キャンセル

  • osusi

    2019/05/25 11:51

    すいません今追加してみました!教えてくれてありがとうございます!

    キャンセル

回答 3

checkベストアンサー

+2

スタートボタンを押下後のストップ以外は反応しないようにしたいです。
できるだけソースコードはそのままでどこを追加すればいいのでしょうか。

いろんな手がありますが、「ルーレットが動いている」状態を「roulette_idに値が入っている」という状態で表すのが簡単だと思います。

clearIntervalを行うたびにroulette_idをnullやundefinedに初期化し、それを判定することで停止処理をスキップするというような方法です。

function isRouletteStarted() {
    return roulette_id == null
}

function random_stop() {
    if (!isRouletteStarted()) return;
    // ...

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2019/05/25 12:54

    無事解決しました!ご丁寧にありがとうございました(><)

    キャンセル

+2

スタートしていないとストップを押せないようにするなら以下のようにifでflagを判定したらどうでしょうか?

function random_stop() {
if ( start_flag === true) {
clearInterval(roulette_id);
start_flag = false;
$('#td' + num).css('backgroundColor', 'blue');
cell.splice(random, 1);
num=0;
}
}


もしくは
>できるだけソースコードはそのままで
とのことなので
var num =0;
にするとかどうでしょう

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2019/05/25 12:54

    無事解決しました!ご丁寧にありがとうございました(><)

    キャンセル

+2

こんにちは

いくつかやり方がありそうですが、

  • start_flag の値が変わるたびに、3つのボタンそれぞれが disabled かどうかを設定する

という方法が考えつきます。
その上で、ご質問にあるコードをなるべく生かすことにすると、まずは以下のような関数を追加します。

function buttons_reset() {
  $('#start').prop('disabled', start_flag);
  $('#stop').prop('disabled', !start_flag);
  $('#reset').prop('disabled', start_flag);
}

上記の関数によって、

  • ルーレットが止まっているとき: 「ストップ」はクリックできない。他の2つはクリックできる。
  • ルーレットが動いているとき: 「ストップ」だけクリックできる。他の2つはクリックできない。

ようにできます。

上記の buttons_reset() を以下の4カ所で使います。

まず、初期表示のときです。

      $(function() {
        $('#start').click(random_start);
        $('#stop').click(random_stop);
        $('#reset').click(random_reset);

        buttons_reset(); // この行を追加
      });

さらに、各ボタンがクリックされたときの start_flag に true か false を代入した直後に呼びます。

      function random_start() {

        if ( start_flag === false) {
          roulette_id = setInterval(roulette_start, 100);
          start_flag =true;
          buttons_reset();  // この行を追加
        }
      }
      function random_stop() {
        clearInterval(roulette_id);
        start_flag = false;
        buttons_reset();  // この行を追加
        $('#td' + num).css('backgroundColor', 'blue');
        cell.splice(random, 1);
        num=0;
      }
      function random_reset() {
        start_flag =false;
        buttons_reset();  // この行を追加
        clearInterval(roulette_id);
        cell = [1,2,3,4,5,6,7,8,9,10,11,12,13,14,15,16];
        $("td").css('backgroundColor', 'white');
      }

以下に上記の修正後のHTML(ファイル名は index.html) を上げていますので、参考にしてください。

先に説明した修正は以下です。

以下のような感じでボタンの disabled 状態が変わります。

イメージ説明

この後の、さらなるリファクタリングとしては、フラグ変数はプログラムの見通しを悪くさせるので、

  • start_flag を無くして roulette_id に有効な値が入っているかどうかで、ルーレットが動作中かどうかを判定する。

といったリファクタをしていってもよいかもしれません。

参考になれば幸いです。

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2019/05/25 14:12

    とても分かりやすい解説ありがとうございました!助かりました(><)

    キャンセル

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

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

関連した質問

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

  • トップ
  • JavaScriptに関する質問
  • javascriptでルーレットを作っているのですがスタートを押さずにストップを押すと押した数だけ止まらないマス目が出てしまいます。