1
JavaScriptを初めて使用しています。 私はLearning JavaScript（o'reilly）を通して仕事をしましたが、最初のJavaScriptを作成しようとしています。初心者用JavaScript
私が興味を持っているものに取り組むのが最善だと思っていましたが、それはかなり複雑です。
私は基本的に、ジェーンステイラーが彼とスペースマリンの間に12歩のスペースハルク（ボードゲーム）の状況をシミュレートしようとしています。 最初のステップでは、いずれかのダイスでGenestealerを殺し、その後5または6を殺す。 ダイスの数字が同じ場合、ガンが詰まる。
ここでは最初のステップをエミュレートしようとしています。私は問題がjamCheckであると思う。
基本的には、これを変更しても常に出力されます！=常に詰まっていることを示しています。
私は変数を別のローカル変数に渡す必要があるのだろうかと思ったが、これをしなくてもkillCheckのために働いていた。 （私は間違っているかもしれませんが、試しました）
本当にここで間違っていることは完全に可能です。
私はあなたが助けてくれるといいか、正しい方向に向けることを願っています。
多くの感謝！
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.1//EN" "http://www.w3.org/TR/xhtml11/DTD/xhtml11.dtd">
<html xmlns="http://www.w3.org/1999/xhtml" xml:lang="en">
<head>
<title>SH</title>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8" />
<script type="text/javascript">
//<![CDATA[
function diceRoll1() {
iValue = Math.random(); // random number between 0 and 1
iValue *= 6; // multiply by 6 to move the decimal
iValue = Math.floor(iValue)+1; // round to nearest integer. +1 to 1-6.
var roll1 = iValue;
document.getElementById('result1').innerHTML = 'Dice roll 1 : ' + roll1;
killCheck (roll1);
jamCheck (roll1);
return;
}
function diceRoll2() {
iValue = Math.random(); // random number between 0 and 1
iValue *= 6; // multiply by 6 to move the decimal
iValue = Math.floor(iValue)+1; // round to nearest integer. +1 to 1-6.
var roll2 = iValue;
document.getElementById('result2').innerHTML = 'Dice roll 2 : ' + roll2;
killCheck (roll2);
jamCheck (roll2);
return;
}
function killCheck(roll1,roll2){
if (roll1==6 || roll2==6)
{
document.getElementById('kill').innerHTML = 'GS KILLED';
}
return;
}
function jamCheck(roll1,roll2){
if (roll1 == roll2)
{
document.getElementById('jam').innerHTML = 'GUN JAMMED';
}
return;
}
//]]>
</script>
</head>
<body onload="diceRoll1();diceRoll2();killCheck();jamCheck();">
<p id="result1">Dice roll 1</p>
<p id="result2">Dice roll 2</p>
<p id="kill">GS ALIVE</p>
<p id="jam">GUN FINE</p>
</body>
</html>
EDIT：私は最終的に友人から多くの援助でそこに着きました。ここでは、現在のコードです：あなたは本当にあなたが数学を使用する必要があり、「最も近い」整数に丸めたい場合
...
function getDiceValue() {
var diceValue = Math.random();
diceValue *= 6;
diceValue = Math.floor(diceValue) + 1;
return diceValue;
}
function killCheck(roll1, roll2) {
if (roll1 === 6 || roll2 === 6) {
document.getElementById('kill').innerHTML = 'GS KILLED';
}
return;
}
function jamCheck(roll1, roll2){
if (roll1 === roll2) {
document.getElementById('jam').innerHTML = 'GUN JAMMED';
}
return;
}
function rollDice() {
var roll1 = getDiceValue(),
roll2 = getDiceValue();
document.getElementById('result1').innerHTML = 'Dice roll 1 : ' + roll1;
document.getElementById('result2').innerHTML = 'Dice roll 2 : ' + roll2;
killCheck (roll1, roll2);
jamCheck (roll1, roll2);
}
//]]>
...
<body onload="rollDice();">
ちょうど関係のないメモ、関数の最後に空の戻り値を使用しないでください（何も返されません）。空のリターンを使用する唯一の理由は、関数を停止することです（ループの中断に似ています）。 – ajax333221
ちょっとしたものをきれいにしたい場合、以下は便利なダイロール関数です：function rollDie（min、max ）{ return Math.floor（Math.random（）* max）+ min; } 6面の例：rollDie（1,6）; – pdizz
@ ajax333221 killCheckとJamCheckに返すと読みましたが、まったく必要ないのですか？ – snoopyj