binarySearch 找到最接近目标的数字。未定义为返回值

Question

我正在研究一个函数，该函数应该从整数列表中将最接近的较低数字返回给目标。 （即 [1,23,45,67,94,122]，目标 = 96。应该返回 94）。我已经多次查看我的代码，试图“捕获导致该函数返回“未定义”的错误，但我无法找出原因......当我通过该过程打印出我的变量时，它们都匹配我想要什么，但我的返回值一直是未定义的。我在想我的问题在于前两个条件，但我仍然不知道为什么。有什么线索吗？

这是我的代码：

function binarySearch(arr,target){
  var midpoint = Math.floor(arr.length/2);

  if (arr[midpoint] === target){
    return arr[midpoint];
  }
  if (arr.length === 1){
    return arr[0];
  }

  if (arr[midpoint] > target){
    binarySearch(arr.slice(0,midpoint),target);
  }else if (arr[midpoint] < target){
    binarySearch(arr.slice(midpoint),target);
  }
}

binarySearch([1,23,45,67,94,122],96); => 预期返回值 = 94 // 得到 = 未定义。 :/

Answer 1

执行递归调用时需要添加return语句

 function binarySearch(arr,target){
  var midpoint = Math.floor(arr.length/2);

  if (arr[midpoint] === target){
    return arr[midpoint];
  }
  if (arr.length === 1){
    return arr[0];
  }

  if (arr[midpoint] > target){
    return binarySearch(arr.slice(0,midpoint),target);
  }else if (arr[midpoint] < target){
    return binarySearch(arr.slice(midpoint),target);
  }
}

Answer 2

所以原来的算法似乎是错误的，选择小于目标的最大值，而不是数值上最接近它的那个。

这是另一个受a java version 启发的版本，但它是为 ES6 编写的，并且像问题的代码一样是递归的。

function binarySearch(arr, target, lo = 0, hi = arr.length - 1) {
   if (target < arr[lo]) {return arr[0]}
   if (target > arr[hi]) {return arr[hi]}
   
   const mid = Math.floor((hi + lo) / 2);

   return hi - lo < 2 
     ? (target - arr[lo]) < (arr[hi] - target) ? arr[lo] : arr[hi]
     : target < arr[mid]
       ? binarySearch(arr, target, lo, mid)
       : target > arr[mid] 
         ? binarySearch(arr, target, mid, hi)
         : arr[mid]
}

console.log(binarySearch([1, 23, 45, 67, 94, 122], 96))  //=> 94
console.log(binarySearch([1, 23, 45, 67, 94, 122], 47))  //=> 45
console.log(binarySearch([1, 23, 45, 67, 94, 122], 207)) //=> 122
console.log(binarySearch([1, 23, 45, 67, 94, 122], 0))   //=> 1

Answer 3

我认为您的算法不会给出正确的结果。 您需要首先找到应该插入元素的位置。 这是堆栈溢出链接： O(log n) algorithm to find best insert position in sorted array

  function binarySearch(arr,target, l, h){
  var midpoint = Math.floor(l+h/2);

  if (arr[midpoint] === target){
    return midpoint;
  }
  if (l === h){
    return l;
  }

  if (arr[midpoint] > target){
    return binarySearch(arr,target, l,midpoint);
  }else if (arr[midpoint] < target){
    return binarySearch(arr, target, midpoint,h);
  }
}
 function binaryclosest(arr, i){
 x= abs(target - arr[i]) 
 y = arr[i+1] - target
 z = target - arr[i-1]
 if(x<y && x<z){
     return arr[i]
 }
 else{
     if(y<z)
        return arr[i+1]
     else
        return arr[i-1]
 }
}
i = binarySearch(arr, target, l, h) 
value = binaryclosest(arr, i)

然后，你应该减去 i-1 和 i+1 取一个 mod 然后返回较低的那个