我遵循伪代码在链接上实现算法,但不知道我的代码有什么问题。
这是我的代码:
/* Returns either the index of the location in the array,
or -1 if the array did not contain the targetValue */
var doSearch = function(array, targetValue) {
var min = 0;
var max = array.length - 1;
var guess;
while(min < max) {
guess = (max + min) / 2;
if (array[guess] === targetValue) {
return guess;
}
else if (array[guess] < targetValue) {
min = guess + 1;
}
else {
max = guess - 1;
}
}
return -1;
};
var primes = [2, 3, 5, 7, 11, 13, 17, 19, 23, 29, 31, 37,
41, 43, 47, 53, 59, 61, 67, 71, 73, 79, 83, 89, 97];
var result = doSearch(primes, 2);
println("Found prime at index " + result);
//Program.assertEqual(doSearch(primes, 73), 20);
答案 0 :(得分:11)
要从数组中获取值,您需要指定一个整数,如array[1]
。 <{1}}将在您的案例中返回array[1.25]
。
为了让它正常工作,我只需在你的循环内添加undefined
以确保我们获得整数。
Math.floor
。这适用于<=
和min
已变为相同的情况,在这种情况下max
。如果没有guess === max === min
,循环将无法在这些情况下运行,函数将返回<=
。
-1
您可以使用function (array, targetValue) {
var min = 0;
var max = array.length - 1;
var guess;
while(min <= max) {
guess = Math.floor((max + min) / 2);
if (array[guess] === targetValue) {
return guess;
}
else if (array[guess] < targetValue) {
min = guess + 1;
}
else {
max = guess - 1;
}
}
return -1;
}
,Math.floor
和Math.ceil
。
我希望这只是一个小小的帮助,我不是很擅长解释,但我会做我的工作。
答案 1 :(得分:2)
在min等于max的代码中,循环结束。但在这种情况下,您不会检查是否array[min] == targetValue
因此,将代码更改为最有可能解决您的问题
/* Returns either the index of the location in the array,
or -1 if the array did not contain the targetValue */
var doSearch = function(array, targetValue) {
var min = 0;
var max = array.length - 1;
var guess;
while(min <= max) {
guess = Math.floor((max + min) / 2);
if (array[guess] === targetValue) {
return guess;
}
else if (array[guess] < targetValue) {
min = guess + 1;
}
else {
max = guess - 1;
}
}
return -1;
};
JSFiddle Link:http://jsfiddle.net/7zfph6ks/
希望它有所帮助。
<强> PS:强>
此行中仅更改代码:while (min <= max)
答案 2 :(得分:1)
您只需要取消注释Program.assertEqual 像这样:
Program.assertEqual(doSearch(primes, 73), 20);
不喜欢这样:
//Program.assertEqual(doSearch(primes, 73), 20);
答案 3 :(得分:0)
如果有人仍在寻找答案,你需要制作它(最多> =分钟)
while (max >= min) {
guess = Math.floor((max + min) / 2);
if (array[guess] === targetValue) {
return guess;
}
else if (array[guess] < targetValue) {
min = guess + 1;
}
else {
max = guess - 1;
}
}
return -1;