0

私は基本的な修正されたCrappsゲームを作成しています。私はJavaスクリプトとHTMLを使用しています。見栄えは良くありませんが、その目的はJavaスクリプトを教えてもらうことです。

私はまだ初期段階にあり、障害にぶつかっています。ボタンをクリックすると、この機能が発生します。私の機能は発生しますが、2 6を振り続けているため、サイコロは6として表示されます。

6が繰り返し発生し続けることを除いて、すべてが機能します。

私はコードを調べましたが、よくある間違いを見つけることができませんでした-生成する必要のある数字がカバーされていること、ベースがカバーされていることを確認してください6つの数字(1-5)はすべて仕事があり、私の名前は正しいなどです。サイコロの画像が行きたいところに毎回6と表示されるので呼ばれています。

 function rolldice()
   {
     dice1=Math.floor(Math.random()*6+0);
        if(dice1=0)
            {
             document.getElementById("dice1").innerHTML="<img src='1.jpg' width='100' height='100' />";
            }
        if(dice1=1)
            {
              document.getElementById("dice1").innerHTML="<img src='2.jpg' width='100' height='100' />";

            }
        if(dice1=2)
            {
              document.getElementById("dice1").innerHTML="<img src='3.jpg' width='100' height='100' />";

            }
        if(dice1=3)
            {
              document.getElementById("dice1").innerHTML="<img src='4.jpg' width='100' height='100' />";

            }
        if(dice1=4)
            {
              document.getElementById("dice1").innerHTML="<img src='5.jpg' width='100' height='100' />";

            }
        if(dice1=5)
            {
             document.getElementById("dice1").innerHTML="<img src='6.jpg' width='100' height='100' />";

            }
dice2=Math.floor(Math.random()*6+0);
    if(dice2=0)
            {
             document.getElementById("dice2").innerHTML="<img src='1.jpg' width='100' height='100' />";
            }
        if(dice2=1)
            {
              document.getElementById("dice2").innerHTML="<img src='2.jpg' width='100' height='100' />";

            }
        if(dice2=2)
            {
              document.getElementById("dice2").innerHTML="<img src='3.jpg' width='100' height='100' />";

            }
        if(dice2=3)
            {
              document.getElementById("dice2").innerHTML="<img src='4.jpg' width='100' height='100' />";

            }
        if(dice2=4)
            {
              document.getElementById("dice2").innerHTML="<img src='5.jpg' width='100' height='100' />";

            }
        if(dice2=5)
            {
             document.getElementById("dice2").innerHTML="<img src='6.jpg' width='100' height='100' />";

            }
 }
4

1 に答える 1

4

=の代わりに使用してい==ます。これは、1 から 5 までの各ステートメントを実行することを意味します。

if (dice1 = 5){

と同じです

dice1 = 5;
if (dice1){

これは常に真実です。

varランダムな値を割り当てるときは、各変数の前にもある必要があります。参考までに。

このコードをすべて次のように短縮することを検討します。

function rolldice(){
  for (var i = 0; i < 2; i++){
    var roll = Math.floor(Math.random() * 6);
    document.getElementById('dice' + (i + 1)).innerHTML = 
      "<img src='" + (roll + 1) + ".jpg' width='100' height='100' />"
  }
}
于 2013-01-27T04:19:53.070 に答える