封装数组最小最大值查找功能时函数无法正常工作求助

原代码的核心问题 #

• 函数内部的smallNum和bigNum是for循环的块级局部变量，外部无法访问；调用findnum(smallNum,bigNum)时，这两个变量未定义，会直接抛出引用错误。
• 函数没有返回值，就算内部计算出结果，外部也拿不到任何数据。
• 没必要用for循环包裹Math.min/Math.max——这两个方法本身就能直接处理展开后的数组，遍历逻辑内部已实现。如果坚持用for循环（符合你提到的for+let实现需求），需要重新编写遍历逻辑。

方案一：最简实现（用Math.min/Math.max） #

修正后的代码：

const myNumbers = [1111, 245, 535 ,222, 221,12,233444];

function findnum(arr) {
  const smallNum = Math.min(...arr);
  const bigNum = Math.max(...arr);
  return { min: smallNum, max: bigNum };
}

const result = findnum(myNumbers);
console.log(result); // 输出：{ min: 12, max: 233444 }

• 函数接收数组参数，不再依赖全局变量，复用性更强。
• 返回包含最值的对象，一次性输出两个结果。
• 调用时传入目标数组，直接接收返回值即可。

方案二：for循环+let手动实现 #

如果要贴合你最初用for循环的思路，代码可以这么写：

const myNumbers = [1111, 245, 535 ,222, 221,12,233444];

function findnum(arr) {
  // 处理空数组边界情况
  if (arr.length === 0) return { min: undefined, max: undefined };
  
  let smallNum = arr[0];
  let bigNum = arr[0];
  // 从第二个元素开始遍历比较
  for (let i = 1; i < arr.length; i++) {
    if (arr[i] < smallNum) {
      smallNum = arr[i];
    }
    if (arr[i] > bigNum) {
      bigNum = arr[i];
    }
  }
  return { min: smallNum, max: bigNum };
}

const result = findnum(myNumbers);
console.log(result); // 输出：{ min: 12, max: 233444 }

• 用let声明smallNum和bigNum，方便遍历过程中修改值。
• 从第二个元素开始遍历，逐个比较更新最值，避免重复比较第一个元素。
• 加入空数组处理，防止数组为空时出现逻辑错误。

内容的提问来源于stack exchange，提问作者Pedro