Roll Dice JavaScript

-5

Я пытаюсь написать программу, которая позволит мне бросить кубики, но мне не повезло. Я новичок в JavaScript, и я не уверен, что я делаю неправильно.Roll Dice JavaScript

random1 = Math.floor((Math.random()*6)+1; /*holds random number from 1 to 6*/ 
    random2 = Math.floor((Math.random())*6)+1; 

    firstname = "dice" + random1 + ".jpg"; 
    secondname = "dice" + random2 + ".jpg"; 

    var first = new Image(); 
    first.src = firstname; 

    var second = new Image(); 
    second.src = secondname; 

    document.getElementById('dicea').src=first.src; 
    document.getElementById('diceb').src=second.src; 

    if (random1==0 && random2 ==0) 
    { 
     document.getElementById("sometext").innerHTML = "Doubles!You get a free turn"; 
    } 
    else 
    { 
     document.getElementById("sometext").innerHTML = "You Lose"; 
    } 
} 
</script> 
</head> 
<body> 
    <img src="dice0.jpg" id="dicea" width="100" height="100" alt="dice" /> 
    <img src="dice0.jpg" id="diceb" width="100" height="100" alt="dice" /> 
<div id="sometext">Blah blah blah</div> 

<button type="button" onclick="rolldice()">Roll Dice</button> 
</body>

источник

2014-11-24 MDI

что вы ожидаете '(random1 == 0 && random2 == 0)' делать? – njzk2

Ну, тогда, возможно, вам нужны более удачные кости. Серьезно, почему бы вам сначала не прочитать свой собственный код для глупых ошибок? –

Я думаю, вы могли бы означать

if (random1 == random2)

вместо

if (random1 == 0 && random2 == 0)

потому random1 и Random2 всегда будет находиться в диапазоне от 1 до 6, никогда 0

источник

2014-11-25 00:03:20

Линия «random1 = Math.floor ((Math.random() * 6) +1; "отсутствует закрывающая скобка перед закрывающей точкой с запятой.

Ни случайным, ни случайным2 никогда не может быть 0, так что утверждение «если» никогда не будет истинным.

источник

2014-11-25 00:05:04

Вы непарные круглые скобки здесь:

random1 = Math.floor((Math.random()*6)+1;

Удалить один:

random1 = Math.floor(Math.random()*6)+1;

Вторая линия работает круглые скобки совпадают, но вы можете удалить пару из них.

Вы создаете объекты изображения, которые вы используете только для хранения источника изображения на некоторое время, прежде чем копировать источник изображения в уже существующие изображения. Вы можете просто поместить источник в нужное изображение сразу:

document.getElementById('dicea').src = firstname; 
document.getElementById('diceb').src = secondname;

условие в этой линии никогда не будет правдой, так как ни одна из переменных никогда не будет равна нулю:

if (random1==0 && random2 ==0)

Я думаю, что вы хотели бы проверить, одинаковы ли значения в переменных:

if (random1 == random2)

источник

2014-11-25 00:08:17 Guffa

Roll Dice JavaScript

ответ

Смежные вопросы