Question

我想创建一个带有一些修改的二进制搜索算法。所以我从谷歌的封闭库中抓取代码并开始进行这些修改。我修改后的版本似乎比它应该慢，所以我慢慢拿出任何我认为可能影响速度的东西。我剩下的是二进制搜索算法的SIMPLER版本，它在Chrome或Firefox中的速度仍然要慢几倍。可能是什么导致了这个？看看这个测试页面。检查来源，看看我在说什么。

https://dl.dropboxusercontent.com/s/4hhuq4biznv1jfd/SortedArrayTest.html

这是谷歌的版本。

goog.array.binarySearch_ = function(arr, compareFn, isEvaluator, opt_target,
    opt_selfObj) {
  var left = 0;  // inclusive
  var right = arr.length;  // exclusive
  var found;
  while (left < right) {
    var middle = (left + right) >> 1;
    var compareResult;
    if (isEvaluator) {
      compareResult = compareFn.call(opt_selfObj, arr[middle], middle, arr);
    } else {
      compareResult = compareFn(opt_target, arr[middle]);
    }
    if (compareResult > 0) {
      left = middle + 1;
    } else {
      right = middle;
      // We are looking for the lowest index so we can't return immediately.
      found = !compareResult;
    }
  }
  // left is the index if found, or the insertion point otherwise.
  // ~left is a shorthand for -left - 1.
  return found ? left : ~left;
};

这是我的版本：

        var search = function(array, num){
            var left = 0;  // inclusive
            var right = array.length;  // exclusive
            while (left < right) {
                var middle = (left + right) >> 1;
                var midValue = array[midValue];
                if (num > midValue) {
                    left = middle + 1;
                } else {
                    right = middle;
                }
            }
            return left;
        };

因为人们似乎用comparefn函数来思考它...当你没有为binarySearch方法提供比较器函数时，它使用以下默认比较函数：

    goog.array.defaultCompare = function(a, b) {
      return a > b ? 1 : a < b ? -1 : 0;
    };

    goog.array.binarySearch = function(arr, target, opt_compareFn) {
  return goog.array.binarySearch_(arr,
      opt_compareFn || goog.array.defaultCompare, false /* isEvaluator */,
      target);
};

请不要在不查看代码的情况下回复。猜测不是很有帮助。

Answer 1

您的实施包含错误。它包含：

var midValue = array[midValue]

应该是

var midValue = array[middle]

代替。

显然，你不幸的是你的数据集不会将错误暴露为不正确的结果，但这只是一个性能问题。

相同的代码比JS闭包库等效的慢

1 个答案: