我需要帮助优化功能

时间:2013-02-21 12:01:35

标签: javascript loops angle

基本上我想生成一个不在许多其他角度的指定范围内的角度(0 - 360度)。我已经完成了这个功能来检查两个角度:

function check(angle1, angle2, range) {
    var diff = angle1 - angle2;

    if(Math.abs(diff % 360) <= range || (360-Math.abs(diff % 360)) <= range) {
        return true;
    } else {
        return false;
    }
}

很简单,但是我需要检查一个随机角度来对抗所有其他角度,如果它通过则继续,生成一个新角度并重新检查它是否失败,并识别任何新角度都不可能通过。

我认为这样可行:

var others = [array of objects];

...

for(var i = 0; i < 360; i++) {
    var pass = true;
    for(var n = 0; n < others.length; n++) {
        if(check(i, others[n].angle, 5)) {
            pass = false;
            break;
        }
    }
    if(pass) return i;
}

return false;

然而,这是循环的很多,我更喜欢随机角度而不是递增。有没有更快更好的方法来做到这一点?感谢。

编辑:决定做这样的事情,从@ TheBronx的回答中得到了想法。

var angles = [];

var range = 5;

function alterAngle(a, n) {
  var angle = a + n;
  if(angle < 0) angle = 360 + angle;
  if(angle > 360) angle = angle - 360;
  return angle;
}



// in the function

var angle = Math.floor(Math.random() * 360);

if(angles.indexOf(angle) == -1) {
  for(var i = -range; i <= range; i++)
    angles.push(alterAngle(angle, i));
}

3 个答案:

答案 0 :(得分:3)

观。想象一下,你的角度是卡片中的牌。生成随机角度后,从甲板上移除该角度,以及范围内的角度。 当你必须生成一个新的角度,而不是在0..360之间产生随机,你只需要选择一张卡片&#34;。这将永远有效,除非你没有更多的卡片和#34;可用。

问题是,你有很多卡吗?你有足够的时间初始化&#34;卡&#34;刚开始?

只是一个想法......不知道它是否好,但似乎很有希望。

答案 1 :(得分:1)

这一个

if(Math.abs(diff % 360) <= range || (360-Math.abs(diff % 360)) <= range) {
    return true;
  } else {
    return false;
}

你可以这样做,这将减少操作

return Math.abs(diff % 360) <= range || (360-Math.abs(diff % 360)) <= range;

答案 2 :(得分:0)

基于我之前的一个问题:Fischer Yates shuffle in coffee-script

var counter, i, permutations, shuffle, x, _i;

// Fischer Yates shuffle algorithm
shuffle = function(arr, required) {
  var i, index, randInt, _i, _ref, _ref1, _ref2;
  if (required == null) {
    required = arr.length;
  }
  randInt = function(n) {
    return Math.floor(n * Math.random());
  };
  if (required > arr.length) {
    required = arr.length;
  }
  if (required <= 1) {
    return arr[randInt(arr.length)];
  }
  for (i = _i = _ref = arr.length - 1, _ref1 = arr.length - required; _ref <= _ref1 ? _i <= _ref1 : _i >= _ref1; i = _ref <= _ref1 ? ++_i : --_i) {
    index = randInt(i + 1);
    _ref2 = [arr[i], arr[index]], arr[index] = _ref2[0], arr[i] = _ref2[1];
  }
  return arr.slice(arr.length - required);
};


// generate array of all possible angles
var angles = []
for(i=0;i<360;i++){ angles.push(i) }

// shuffle as many as you need (20 in this example)
var shuffled = shuffle(angles,20)

// check the result
console.log(shuffled)

// simply deal off values from this shuffled array as needed