获取错误未捕获的类型错误：无法读取未定义的属性“toLowerCase”

Question

出现错误未捕获的类型错误：无法读取未定义的属性“toLowerCase” 在执行此代码时。

 const map  = (array, callback) => {
    return [callback(array[0])].concat(map(array.slice(1),callback));
}
var lower = map(['A','B','C'],function (val) {
    return val.toLowerCase()
});
console.log(lower);

Answer 1

您需要检查数组length 是否为1，然后只返回元素，因为没有其他元素了。

const map  = (array, callback) => {
    if(array.length === 1) return [callback(array[0])]
    return [callback(array[0])].concat(map(array.slice(1),callback));
}
var lower = map(['A','B','C'],function (val) {
    return val.toLowerCase()
});
console.log(lower);

您似乎想创建自己的map()。认为循环比递归更好。

const map  = (array, callback) => {
    let res = []
    for(let i = 0;i<array.length;i++){
      res[i] = callback(array[i],i)
    }
    return res;
}
var lower = map(['A','B','C'],function (val) {
    return val.toLowerCase()
});
console.log(lower);

为什么 OP 的代码不起作用？

有问题的代码使用递归。每当我们使用递归方法时，我们应该有一个条件来返回函数并再次调用它。

• 现在考虑数组length 是1
• 然后array.slice(1) 将返回[]
• 一个空数组将被传递给map()。
• 对于空数组 array[0] 将 `undefined
• 然后undefined 将被传递给callback
• undefined.toLowerCase 抛出错误。

结论

当数组中只剩下一个元素时，您不需要运行 .concat(map(array.slice(1),callback)) 部分。因为concat()有更多注意事项