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

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

ただいまの
回答率

87.49%

jqueryでのルーレット。配列以外で同じ挙動になるようにしたい。

解決済

回答 3

投稿 編集

  • 評価
  • クリップ 1
  • VIEW 1,162

score 9

jqueryでルーレットを作成しまして、以下の要件を満たすものを作成しました。

1.ルーレットの目を16個作成する
2.スタートボタンを押すとルーレットが回転する
3.回転中はスタートボタンを押せなくする(ストップボタンで再び押せるようになる)
4.ルーレットの回転にランダム要素を組み込む
5.ルーレットが回っているときに、ストップボタンを押すとルーレットが止まる
6.ストップボタンを押した目が、ユーザに分かるようにする(目の色が変わるなど)
7.一度ストップボタンで停止した目は次回以降は停止しない
8.スタートとストップを16回の繰り返しで、全ての目に停止する
9.16回停止後はスタートボタンが押せなくなる(リセットボタンで押せるようになる)
10.過去に停止した目が何か、ユーザに分かるようにする(目に色がつくなど)
11.リセットボタンを押すと初期表示の状態に戻る(もしルーレットが回っている場合はルーレットの回転が止まり、初期表示の状態になる)
12.(ソースコード)比較演算子は、「===」や「!==」を利用すること

そして、以下の通り実装ができました。

<!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-1.11.0.min.js"></script>
        <script>
            $(function(){
               var number = 0;
               var numbers =[1,2,3,4,5,6,7,8,9,10,11,12,13,14,15,16];
               var interval_id,tr,td;
               for (var j = 0; j <= 12; j+=4){
               tr = $("<tr>");
               for (var i = 1; i <= 4; i++){
               td = $('<td class="num' + (i+j) +'">');
               td.html(i+j);
               tr.append(td);
               }
               $("#matrix").append(tr); 

               }  


                $("#start").on("click",function(){
                    interval = setInterval(count,100);//ルーレット動かす
                    $("#start").prop("disabled",true);//ルーレット動作中はスタートボタン押せないようにする
                    $("#stop").prop("disabled",false);//ルーレット動作中はストップボタンを押せるようにする
                });

                $("#stop").on("click",function(){
                    clearInterval(interval);//1-16の数字の選択を止める
                    numbers.splice(index,1);//配列のキー+1が値。配列の中から、ストップ時に選択した配列の中の値を消す
                    console.log(numbers);
                    if (numbers.length === 0){// 配列の中身が0になったら
                        $("#start").prop("disabled",true);//スタートボタンを押せなくする
                    } else if(numbers.length !== 0){// 配列の中身が0以外の場合、
                        $("#start").prop("disabled",false);//スタートボタンを押せなくする
                    }
                    $(".num" + number).addClass("stop");//ストップの時に選択したマスの背景をピンクにする。
                    $("#stop").prop("disabled",true);//ストップの二度押し禁止
                });

                 $("#reset").on("click",function(){
                    clearInterval(interval);//ルーレット動作を止める
                    $("td").removeClass("count stop");//すべてのマスを白に戻す
                    $("#start").prop("disabled",false);//スタートボタンを押せるようにする
                    //削除した値を戻す方法が分からない。。。!!!
                    numbers =[1,2,3,4,5,6,7,8,9,10,11,12,13,14,15,16];
                    $("#stop").prop("disabled",true);
                 });

                 function count(){
                  $(".num" + number).removeClass("count");//まずは背景白
                  index = Math.floor(Math.random() * numbers.length);//0-15のランダムな数字をindexで取得
                  number = numbers[index];//配列の中から1-16の数字を変数numberで取得
                  $(".num" + number).addClass("count");//選択されたマスが赤色
                 }

    });

    </script>

    </head>
    <body>
    <div id="rouolette">
     <table id="matrix">

        </table>
       <div id="button_erea">
            <button id ="start">スタート</button>
            <button id ="stop">ストップ</button>
        <button id ="reset">リセット</button>
        </div>
    </div>
    </body>
</html>

今度は配列を使わず、背景が赤色以外のマスを選択するようにといったコードを書いて同じことを実装したいと
思っているのですが、うまくいきません。

以下が配列を使わわずに同じことを実現しようとして書いたコードです。

<!DOCTYPE html>
<html lang="ja">
    <head>
        <meta charset="utf-8">
        <title>配列使わないルーレット</title>
        <link rel="stylesheet" href="roulette2.css">
        <script src="http://code.jquery.com/jquery-1.11.0.min.js"></script>
    <script>
     $(function(){
    //配列を使わない場合のルーレットは可能???
    var number = 0;
    var interval_id,tr,td;
    for (var j = 0; j <= 12; j+=4){
        tr = $("<tr>");
        for (var i = 1; i <= 4; i++){
        td = $('<td class="num' + (i+j) +'">');
        td.html(i+j);
        tr.append(td);
        }
        $("#matrix").append(tr); 

    }

    $("#start").on("click",function(){
        interval = setInterval(count,100);//ルーレットを実行
        $("#start").prop("disabled",true);//スタートボタン二度押し禁止
        $("#stop").prop("disabled",false);//ルーレット動作中はストップボタンを押せるようにする
     });

     $("#stop").on("click",function(){
        clearInterval(interval);//ルーレットを止める
        $("#start").prop("disabled",false);//スタートボタンを押せるようにする
        $(".num" + number).addClass("stop");
        $("#stop").prop("disabled",true);//ストップの二度押し禁止
     });

     $("#reset").on("click",function(){
        clearInterval(interval);//ルーレットを止める
        $("td").removeClass("count stop");//すべてのマスを白に戻す
        $("#start").prop("disabled",false);//スタートボタンを押せるようにする
        $("#stop").prop("disabled",true);//リセット押している時はストップを押せない
     });

     function count(){
      $(".num" + number).removeClass("count");//最初はマスの背景白
      number = 1 + Math.floor(Math.random()*16);//1-16の数字をランダムに変数numberに代入
      if (!$(".num" + number).hasclass("stop")){//class名stopがなければ
      $(".num" + number).addClass("count");//マスの色を赤くする
      }

     }

    });

    </script>

    </head>
    <body>
    <div id="rouolette">
     <table id="matrix">

        </table>
       <div id="button_erea">
            <button id ="start">スタート</button>
            <button id ="stop">ストップ</button>
        <button id ="reset">リセット</button>
        </div>
    </div>
    </body>
</html>

試したこと

以下の部分です。

 function count(){
      $(".num" + number).removeClass("count");//最初はマスの背景白
      number = 1 + Math.floor(Math.random()*16);//1-16の数字をランダムに変数numberに代入
      if (!$(".num" + number).hasclass("stop")){//class名stopがなければ
      $(".num" + number).addClass("count");//マスの色を赤くする
      }


ただ、これだと1度ストップが押されたマスも通過して、通過してから色がついてるかついてないかを判断してしまうし、そもそも上記コードを入力するとルーレット自体繰り返し処理ができなくなってしまいました。

実現するためにどのような方法が考えられるでしょうか?

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 3

check解決した方法

0

以下のように解決いたしました。

<!DOCTYPE html>
<html lang="ja">
    <head>
        <meta charset="utf-8">
        <title>配列使わないルーレット</title>
        <link rel="stylesheet" href="roulette2.css">
        <script src="http://code.jquery.com/jquery-1.11.0.min.js"></script>
        <script>


        $(function(){
            var number = 0;
            var interval_id,tr,td;
            for (var j = 0; j <= 12; j+=4){
                tr = $("<tr>");
                for (var i = 1; i <= 4; i++){
                td = $('<td class="num' + (i+j) +'">');
                td.html(i+j);
                tr.append(td);
                }
                $("#matrix").append(tr); 

            }

            $("#start").on("click",function(){
                interval = setInterval(count,100);//ルーレットを実行
                $("#start").prop("disabled",true);//スタートボタン二度押し禁止
                $("#stop").prop("disabled",false);//ルーレット動作中はストップボタンを押せるようにする
             });

            $("#stop").on("click",function(){
                clearInterval(interval);//ルーレットを止める
                $("#start").prop("disabled",false);//スタートボタンを押せるようにする
                $(".count").addClass("stop")
                $("#stop").prop("disabled",true);//ストップの二度押し禁止
             });

            $("#reset").on("click",function(){
                clearInterval(interval);//ルーレットを止める
                $("td").removeClass("count stop");//すべてのマスを白に戻す
                $("#start").prop("disabled",false);//スタートボタンを押せるようにする
                $("#stop").prop("disabled",true);//リセット押している時はストップを押せない
             });

            function count(){
                $("td").removeClass("count");//最初はマスの背景白
                number = Math.floor(Math.random()*$("td").not(".stop").length);//class名stopを持っていないtdの個数
                $("td").not(".stop").eq(number).addClass("count")//tdにclass名stopがなければ、ランダムで選んだ数字に止まって赤にする。

             }



        });

        </script>

    </head>
    <body>
     <div id="rouolette">
      <table id="matrix">

        </table>
       <div id="button_erea">
            <button id ="start">スタート</button>
            <button id ="stop">ストップ</button>
        <button id ="reset">リセット</button>
        </div>
     </div>
    </body>
</html>

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

0

Jqueryオブジェクトじゃだめ?
オブジェクトは配列じゃないらしい

      var R=$("td").not(".stop");//配列ではなくJqueryオブジェクトらしい
      $(".num" + number).removeClass("count");//最初はマスの背景白
      //number = 1 + Math.floor(Math.random()*R.length);//1-16の数字をランダムに変数numberに代入
      var M=Math.floor(Math.random()*R.length);
      number = R[M].innerText;


@コード間違ってる所がある
動かないのはそれかも

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2020/02/11 23:07

    ご回答ありがとうございます!
    hasClassのところ、hasclassになっていましたね…。
    自己解決しまして、そちらの解決方法を掲載させていただきます。
    ヒントを頂きありがとうございました!!!

    キャンセル

0

とりあえずhasclass→hasClass

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2020/02/10 10:19

    .numの使い方が変ですね。
    クラスなんだから数字と組み合わせる必要はないでしょう
    .num:not(.stop)とかでフィルタすればよいでしょう

    キャンセル

  • 2020/02/11 23:03

    ご回答ありがとうございました!
    hasClassでとりあえず動くようになりました。notを使って自己解決いたしました。
    ヒントをいただきありがとうございました。

    キャンセル

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

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

関連した質問

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