2

我正在尝试使用数学随机对象制作一个逼真的无偏 JavaScript 骰子。当我单击时，我希望网页上出现一个 2-12 的数字，但它没有返回。我的代码有什么问题。

<html>
<head><title>DiceBoy</title>
</head>
<body>
<script>
function getRandom(){
  var num=Math.random();
  if(num < 0.0278) return 2; 
  else if(num < 0.0834) return 3;
  else if(num < 0.1667) return 4; 
    else if(num < 0.2778) return 5; 
      else if(num < 0.4167) return 6; 
        else if(num < 0.5834) return 7; 
          else if(num < 0.7223) return 8;
            else if(num < 0.8334) return 9; 
              else if(num < 0.9167) return 10; 
                else if(num < 0.9723) return 11;
                  else return 12;
                  var x=getRandom();
                  document.write(x);
}

</script>

<input type="button" value="Click Here" onClick="getRandom();">

</body>
</html>
4

6 回答 6

4

为什么不这样写：

var RandomNumber1 = Math.floor(Math.random() * 6) + 1;
var RandomNumber2 = Math.floor(Math.random() * 6) + 1;

var DiceNumber = RandomNumber1 + RandomNumber2;
于 2013-01-31T07:25:12.910 回答
1

试试这个：

<html>
    <head>
        <title>DiceBoy</title>
    </head>
    <body>
        <script>
            function getRandom() {
                return (Math.floor(Math.random() * 6) + 1) * 2;
            }
        </script>
        <input type="button" value="Click Here" onClick="document.write(getRandom());">
    </body>
</html>

JSFiddle：http: //jsfiddle.net/kvFE5/1/

理想情况下，您应该将其重构为不显眼地处理。

虽然老实说 - 你为什么不使用Math.random ...

于 2013-01-31T07:23:11.927 回答
0

您的返回语句将在到达时退出该方法，并且其余代码不会运行。您也不想从方法中调用 getRandom 方法，否则您将有一个永无止境的循环。而不是返回数字（因为没有人在听返回值）将它存储在一个变量中，然后在最后两行中使用它。

function getRandom(){
  var num=Math.random();
  var result = 0;
  if(num < 0.0278) result = 2; 
  else if(num < 0.0834) result = 3;
    else if(num < 0.1667) result = 4; 
     .
     .
              else result =  12;
              document.write(result);
}
于 2013-01-31T07:24:37.217 回答
0

问题是..

var x=getRandom();
document.write(x);

在此之前从未被称为函数“返回”。将代码移到函数之外，以便它可以调用随机化函数，然后使用结果。

function getRandom () {
   // the long icky original code WITHOUT the following:
   // var x=getRandom();
   // document.write(x);
}
// later, OUTSIDE the function
var result = getRandom();
document.write(result);

但是，获得相同效果的更好方法是使用：

function getDiceRole() {
   return Math.floor(Math.random() * 6) + Math.floor(Math.random() * 6) + 2
}

对于读者：至于为什么需要两次掷骰，请参阅骰子角色统计- 请注意获得所有值的概率不相等。也就是说，一个简单Math.floor(Math.random() * 11) + 2的是不正确的。

..在掷两个骰子时，两个骰子的总和的不同可能性并不相同，因为获得某些数字的方法比其他方法多。

于 2013-01-31T07:26:40.233 回答
0

尝试这样的事情来生成两个数字之间的随机数

/**
 * Returns a random integer between min and max
 * Using Math.round() will give you a non-uniform distribution!
 */
function getRandomInt (min, max) {
    return Math.floor(Math.random() * (max - min + 1)) + min;
}

在你的情况下打电话

 getRandomInt (2, 12);

完整的 html 看起来像

<html>
<head>
<script type="text/javascript">
    /**
     * Returns a random integer between min and max
     * Using Math.round() will give you a non-uniform distribution!
     */
    function getRandomInt (min, max) {
        return Math.floor(Math.random() * (max - min + 1)) + min;
    }

    function generateNumber()
    {
       document.getElementById('generated-id').innerHTML = getRandomInt (2, 12);
    }

</script>
</head>
<body>
Generated number : <span id="generated-id"></span>
<input type="button" onclick="generateNumber();" value="Generate random number" />
</body>
</html>
于 2013-01-31T07:28:35.183 回答
0

您可以尝试使用类似他的方法Math.floor，将数字向下舍入到最接近的整数。

<html>
    <head>
        <title>DiceBoy</title>
    </head>
    <body>
        <script type="javascript/text">
            function getDieRoll() {
                return Math.floor(Math.random() * (12 - 1+ 1)) +1;
            }
        </script>
        <input type="button" value="Click Here" onClick="document.write(getDieRoll());">
    </body>
</html>
于 2013-01-31T07:36:44.847 回答