为什么我得到 undefined 而不是 null？

Question

我写了一个函数，它返回一个给定数字（1 = 星期一，2 = 星期二等）的星期几，我应该创建一个警告，如果数字小于 1 或大于 7，函数应该返回 null。为什么它会为 0 和 8 返回 undefined？

daysOfWeek = ['Monday', 'Tuesday', 'Wednesday', 'Thursday', 'Friday', 'Saturday', 'Sunday']

function returnDay(day) {
  day = daysOfWeek[day - 1];
  if (day > 7 || day < 1) {
    return null
  } else {
    return day
  }
}

console.log(returnDay(8))
console.log(returnDay(0))

Answer 1

您正在覆盖您的论点，这被认为是不好的做法。

当您访问daysOfWeek[0 -1] => daysOfWeek[-1] => undefined 时，您评估的是undefined < 1，即falsy。所以你输入 else，它返回 day => undefined

以下代码有效

daysOfWeek = ['Monday', 'Tuesday', 'Wednesday', 'Thursday', 'Friday', 'Saturday', 'Sunday']

function returnDay(day) {
  if (day <= 7 && day >= 1) {
    return daysOfWeek[day - 1]
  }
  return null
}

console.log(returnDay(8))
console.log(returnDay(0))

或者，甚至更密集

daysOfWeek = ['Monday', 'Tuesday', 'Wednesday', 'Thursday', 'Friday', 'Saturday', 'Sunday']

function returnDay = d => daysOfWeek[d - 1] || null

console.log(returnDay(8))
console.log(returnDay(0))

Answer 2

基本上，您的代码中现在有两个范围检查，这是最大的问题。

这是第一个检查：

day = daysOfWeek[day - 1];

... 以undefined 结尾，除非day 具有有效值（因此daysOfWeek 中有一个元素对应于该值）。

这是第二次检查：

if (day > 7 || day < 1) { ... }

... 最终会做错事，因为 day 变量在第一次检查的那一行中获得了不同的值。当然，您可以通过引入另一个名称来解决此问题...

const dayTitle = daysOfWeek[day - 1];
if (day > 7 || day < 1) { ... }

...但这意味着您仍在检查相同的条件两次。重复自己永远不是一个好主意。

下面是如何做到的：

function returnDay(day) {
  return daysOfWeek[day - 1] || null;
}

... 是的，就这么简单：边界范围检查现在由 JS 完成，而不是您和您的代码。换句话说，如果day 值不能用作该dayOfWeek 数组的正确索引，那么您只会得到undefined（它将由|| null 合并到，嗯，null）。

是的，在这种情况下将null 作为默认值对我来说似乎很奇怪。

Answer 3

所以我改变了论点，所以那一天不再被覆盖，它看起来像这样：

function returnDay(n){
    day = daysOfWeek[n-1];
    if (n  > 7 || n  < 1){
        return null
    } else {
        return day
    }
}
console.log(returnDay(8))
console.log(returnDay(0))

我现在成功地为 returnDay(0) 和 returnDay(8) 获取 null