JavaScript代码运行结果符合预期但输出后出现undefined如何解决

• console.log() 方法本身的返回值是undefined，你的checkAir函数最后return的是console.log(...)的执行结果，所以函数实际返回值就是undefined。你调用函数的时候又在外层套了一层console.log(checkAir(...))，相当于先执行函数内部的打印输出结果，再打印函数的返回值undefined，就出现了多余的undefined提示。
• 代码中dirtyCount是计数器变量，你错误将它的初始值设为了空字符串""，虽然JS的隐式类型转换可以让自增操作正常执行，但属于不规范写法，可能引发其他潜在问题。

你可以任选一种方案修改：

方案1：仅调整调用方式 #

保留函数原有逻辑，去掉调用时外层的console.log即可：

const checkAir = function (samples, threshold) {
  let numb = samples.length 
  let dirtyCount = ""


  for (let i = 0 ; i < samples.length ; i++) {
    if (samples[i] === 'dirty'){
      dirtyCount++
    }
  }
  if (dirtyCount / numb >= threshold){
    return console.log("Polluted")
  } else {
    return console.log("Clean")
  }
}

// 去掉外层console.log，直接调用函数即可
checkAir(
  ['clean', 'clean', 'dirty', 'clean', 'dirty', 'clean', 'clean', 'dirty', 'clean', 'dirty'],
  0.3
);


checkAir(
  ['dirty', 'dirty', 'dirty', 'dirty', 'clean'],
  0.25
);

checkAir(
  ['clean', 'dirty', 'clean', 'dirty', 'clean', 'dirty', 'clean'],
  0.9
)

方案2：更规范的重构写法 #

调整函数逻辑，让函数直接返回状态字符串，同时修复计数器初始值的不规范问题：

const checkAir = function (samples, threshold) {
  const total = samples.length 
  // 计数器初始值修正为数字0
  let dirtyCount = 0

  for (let i = 0 ; i < samples.length ; i++) {
    if (samples[i] === 'dirty'){
      dirtyCount++
    }
  }
  // 直接返回结果字符串，不在return中嵌套console.log
  return dirtyCount / total >= threshold ? "Polluted" : "Clean"
}

console.log(checkAir(
  ['clean', 'clean', 'dirty', 'clean', 'dirty', 'clean', 'clean', 'dirty', 'clean', 'dirty'],
  0.3
));


console.log(checkAir(
  ['dirty', 'dirty', 'dirty', 'dirty', 'clean'],
  0.25
));

console.log(checkAir(
  ['clean', 'dirty', 'clean', 'dirty', 'clean', 'dirty', 'clean'],
  0.9
))

• 明确函数职责：工具类函数尽量只做逻辑计算返回结果，不要在内部直接执行打印操作，方便后续复用和结果二次处理。
• 调用函数前先确认函数的返回值类型，不要盲目对无返回值的函数执行打印操作。
• 数字类型的计数器、累加器变量初始化必须设为0，不要使用空字符串、null等其他类型，避免隐式类型转换带来的隐藏bug。

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