Question

嘿伙计们我编写了一个比较数组值并返回最小值的函数，但我想知道是否有办法使其更有效，如迭代所有数组（使用一个循环）并将结果放入一个新数组或者使单个数组的单个数组子阵列等。此外，该函数提供正确的输出，但打印答案三次：

var nums1 = [-7528819, 3927361, -6398192];
var nums2 = [1777100, -2299720, -5566643];
var nums3 = [7188445, 3724971, 7699332];
var nums4 = [-8432528, -159836, -1604959];
var nums5 = [2764889, 4681472, 701396];
var nums6 = [-5073513, 599535, 4388457];
var nums7 = [8689640, 8028586, 1022322];
var nums8 = [-1088592, 1211232, -7868192];
var nums9 = [-5848613, -4945165, 631213];
var nums10 = [3218429, -833619, -1495854];
var nums11 = [8007060, 1637562, -7568493];
var nums12 = [-8391131, -6585338, 131787];
var nums13 = [-3957775, -9396892, -6143241];
var nums14 = [-6258442, -7829421, 3696922];
var nums15 = [2136598, 4935467, -1621605];
var nums16 = [-7162005, 9861954, 8977930];
var nums17 = [7226452, 8551594, 7006517];
var nums18 = [-1751226, -2536997, -1782251];
var nums19 = [380582, 1614389, 3272584];
var nums20 = [-8988205, -5167181, -7561034];
var nums21 = [-484059, -7160121, 4076528];
var nums22 = [1947448, -5551253, 7491190];

var numsLength = nums1.length;
var i = 0;
var minNum;

function test(arr) {
    for (i; i < numsLength; i++) {
        if (arr[0] < arr[1] && arr[2]) {
            minNum = arr[0];

        } else if (arr[1] < arr[2] && arr[0]) {
            minNum = arr[1];

        } else if (arr[2] < arr[1] && arr[0]) {
            minNum = arr[2];
        }
        console.log(minNum);
    }
}

test(nums1);

Answer 1

您可以使用Math.min功能。

console.log(Math.min.apply(null, nums1));

Answer 2

查看他的代码片段并阅读内联评论：

var nums = [];

// I'm pushing only 3 sets of data, but there can be any number
// Also there can be any number of elements in each array as you can see
nums.push([-7528819, 3927361, -6398192]);
nums.push([1777100, -2299720, -5566643, 380582]);
nums.push([7188445, 3724971, 7699332, 1947448, -5551253, 7491190]);

function produceResults(nums) {
    var i,
        results = [];

    // gathering results
    for (i = 0; i < nums.length; i++) {
        results.push(Math.min.apply(null, nums[i]));
    }

    return results;
}

console.log(produceResults(nums));

所以2个建议：

使用更多动态结构（数组数组）而不是定义22个数组。
使用内置的JS函数和组件（Math.min）

Answer 3

在大多数情况下，展开循环实际上是循环的最有效实现。然而，实际上，展开循环通常是不可行的。使用一个小的，固定大小的阵列，就像你在这里一样，循环的每个排列是显而易见的，如果你的目标是原始速度，你就不会比你拥有的更有效率。话虽这么说，你的函数中的循环是没用的，正如其他人所指出的那样，因为你已经基本上展开了循环。 if语句的语法也是错误的，并且您没有处理数组中的值相等的情况。对于大小为3的固定数组，你需要的更多内容......

if (val1 <= val2 && val1 <= val3) {
    minVal = val1;
} else if (val2 <= val1 && val2 <= val3) {
    minVal = val2;
} else minVal = val3;

现在，如果你想对任意大小数组的min值进行任意搜索，你会做类似的事情，但是使用循环，比如...

var minVal = null;

for (var i = 0; i < arr.length; i++) {
    if (minVal === null || minVal > (val = arr[i]))
        minVal = val;
}

根据您实际想要完成的内容以及数组的大小，对数组进行排序并从排序数组中重新生成min（0 index）可能是有意义的。如果您选择这条路线，请先使用谷歌搜索＆＃34;排序算法＆＃34;

我怎样才能使我的功能更有效率？

3 个答案: