Слишком много утверждений "или" (JavaScript) - PullRequest
2 голосов
/ 14 февраля 2012

Я создаю простую математическую программу с 6 различными типами задач. Я хочу, чтобы программа случайно отображала один из 6 типов, но некоторые проблемы должны появляться чаще. Я использую взвешенный массив, но после того, как тип проблемы выбран из взвешенного массива, у меня возникают проблемы с выяснением, какой это тип проблемы, без использования 10 или более команд «или» внутри оператора if. Вот упрощенная версия:

//shuffle array of 10 integers to get a random value
var rand_10 = [0,1,2,3,4,5,6,7,8,9];
fisherYates(rand_10);

//Set weightedProb
weightedProb[0] = probType[0];
weightedProb[1] = probType[0];
weightedProb[2] = probType[0];
.
.
.
weightedProb[8] = probType[0];
weightedProb[9] = probType[1];

theProblem = weightedProb[rand_10[0]];
if(rand_10[0] == 0 || rand_10[0] == 1 || rand_10[0] == 2 || rand_10[0] == 3 ||rand_10[0] == 4||rand_10[0] == 5||rand_10[0] == 6||rand_10[0] == 7||rand_10[0] == 8){
  //do something
}else if(rand_10[0] ==9){
  //do something else
}

Ответы [ 6 ]

5 голосов
/ 14 февраля 2012

Будет ли простое сравнение диапазонов (> и <) тем, что вы ищете?

например

if(rand_10[0] >= 0 && rand_10[0] <= 8){
  //do something
}else if(rand_10[0] == 9){
  //do something else
}
3 голосов
/ 14 февраля 2012

Вы можете сделать что-то вроде этого (обратите внимание, indexOf не поддерживается в некоторых версиях IE):

if ([0,1,2,3,4,5,6,7,8].indexOf(rand_10[0]) > -1) {
}

Предполагается, что значение не всегда является диапазоном, и вам необходим детальный контроль.

1 голос
/ 14 февраля 2012

Вы также можете использовать оператор switch:

 switch(rand_10[0]) {
   case 0:
   case 1:
   case 2:
   case 3:
   case 4:
   case 5:
   case 6:
   case 7:
   case 8:
     // do something
     break;
   case 9:
     // do something
     break;
 }
1 голос
/ 14 февраля 2012

arr = [ 1, 2, 3, 4, 5, 6, 7, 8]

if (rand_10[0] in arr) {
//...
}
1 голос
/ 14 февраля 2012

почему ты не хочешь так писать?

if(rand_10[0] >= 0 && rand_10[0] <= 8){
  //do something
}else if(rand_10[0] ==9){
  //do something else
}
0 голосов
/ 14 февраля 2012

Лучше использовать функцию, которая генерирует взвешенный случайный индекс, например:

function weightedRandomIndex(weights) {
    var sum = 0, i, x, n=weights.length, rnd;
    for (i=0; i<n; i++) {
        sum += weights[i];
    }
    rnd = Math.floor(Math.random()*sum);
    for (i=0, x=0; i<n; i++) {
        x += weights[i];
        if (x > rnd) return i;
    }
}

Тогда вам просто нужно указать веса, чтобы выбрать индекс массива:

var idx = weightedRandomIndex([9,1]);  // 9 to 1 chance for index 0 over index 1
...