Math.random不断返回相同的答案

Question

我对编码和JavaScript还是很陌生，但是我将下面的代码作为三向随机化器编写。 一切似乎都工作正常，除了无论我运行了多少次代码，返回值都是“ c”插入的。 我想知道是否有人可以给我一些有关如何解决此问题的快速建议。 谢谢。

var random = function() {
  var randomizer = function() {
    Math.random() * 100
  }

  if (randomizer <= 33) {
    var compDecision = "a"
  }
  else if (randomizer > 67) {
    var compDecision = "b"
  }
  else if (33 < randomizer <= 67) {
    var compDecision = "c"
  }

  document.write(compDecision)
}

Answer 1

一堆事情马上浮现在脑海：

1. JavaScript没有隐式返回

因此，这与您的想法不符：

var randomizer = function() {
  Math.random() * 100
}

该函数返回undefined 。 你需要：

var randomizer = function() {
  return Math.random() * 100
}

2.括号在JavaScript函数调用中不是可选的

所以这也不会做你认为：

if (randomizer <= 33) {
    var compDecision = "a"
}

您将需要：

if (randomizer() <= 33) {
    var compDecision = "a"
}

3. JavaScript没有三向比较

因此，这与您的想法不符：

else if (33 < randomizer <= 67)

您将需要：

else if (33 < randomizer() && randomizer() <= 67)

最后，就像其他人提到的那样，首先将randomizer定义为函数实际上没有任何意义。 为了让您的random函数执行您想要的操作（以大致相等的概率产生'a' ， 'b'或'c' ），您确实想在函数开始时产生一个随机值并重用它：

function random() {
  var randomizer = Math.random() * 100;

  if (randomizer <= 33) {
    return 'a';
  } else if (randomizer <= 67) {
    return 'b';
  } else {
    return 'c';
  }
}

console.log(random());

希望有帮助。

Answer 2

您没有正确调用randomizer函数。 我已将其删除，因为在这种情况下它并不是必需的：

var random = function() {
    // I assume you want the comparisons below to be run against the same number
    var randomNumber = Math.random() * 100;

    if (randomNumber <= 33) {
        var compDecision = "a";
    }
    else if (randomNumber > 67) {
        var compDecision = "b"
    }
    else {
        var compDecision = "c"
    }

    return compDecision;
}

Answer 3

您可以将代码简化为：

 var randomizer = Math.random() * 100; if (randomizer <= 33) { var compDecision = "a"; } else if (randomizer > 67) { var compDecision = "b"; } else if (33 < randomizer && randomizer <= 67) { var compDecision = "c"; } alert(compDecision); 

Answer 4

丹涛的答案很好，此外，似乎只调用一次的单行函数似乎毫无意义，因此：

var n = Math.random() * 100;

但是， document.write部分可能应该分开，因为您可能希望以始终不适合将结果写入当前文档的方式调用此函数。

最后，您只需要测试三个条件中的两个，因为如果不是前两个条件之一，那么它必须是第三个条件。 您可以使用条件运算符：

function random() {
  var n = Math.random() * 100;
  return n <= 33? 'a' : n <= 67? 'c' : 'b';
}

document.write(random()); 

Answer 5

randomizer是包含函数的变量。

要了解此问题，请尝试

alert（Math.random（））;

var value = randomizer（）; 警报（值）;

Answer 6

我认为，您可以按照以下方式做同样的事情：

function Randomize(){
    rng = Math.random() * 3;
    rng = Math.round(rng);
    switch(rng) {
    case 0:
        return "a"
        break;
    case 3:
        return "a"
        break;
    case 1:
        return "b"
        break;
    case 2:
        return "c"
        break;
    };
};
alert(Randomize());

Answer 7

JavaScript在每个语句后都需要使用分号。 但是，如果不应该在分号后面加上/然后/否则。

if (a < 10) {
  alert("Less than ten.");
} else {
  alert("Ten or more.");
}

有时，分号会自动插入它们所属的位置，但是您不应该依赖它-最好自己键入它们。

Answer 8

关键字var用于创建/声明/初始化变量。 每个变量仅执行一次。

同样好的做法是在包含变量的函数顶部声明一个变量。

function() {
  var result;

  if (condition1 === true) {
    result = "a";
  } else if (condition2 === true) {
    result = "b"
  } else {
    result = "c"
  }

  return result;
}

Answer 9

在您的代码中，“ randomizer”是一个函数。 要执行它并获取它返回的值，您必须在其后加上一对括号： randomizer()

因此， randomizer <= 33并不是将两个数字进行比较，而是将函数与数字进行比较。 randomizer > 67 。 因此，它们每个都评估为false 。

然后， 33 < randomizer <= 67计算为true 。 为什么？ 我不确定。 但是因为它被认为是“ true”， var compDecision = "c"将执行var compDecision = "c" 。

这就是为什么您不断获得“ c”的原因。

编辑： RobG在下面的评论解释了为什么33 < randomizer <= 67在JavaScript中评估为true 。