Question

我该如何解决这个问题？

问题在于：

实现一个函数“mapById”，当给定一个对象数组时，它返回一个对象，其中键是输入对象的id，值是相应的输入对象。

例如：

mapById([{id: 102, name: "Alice"},
        {id: 205, name: "Bob", title: "Dr."},
        {id: 592, name: "Claire", age: 32}]);

返回：

{
102: {id: 102, name: "Alice"},
205: {id: 205, name: "Bob", title: "Dr."},
592: {id: 592, name: "Claire", age: 32}
}

var key = callback（item）; ^ TypeError：回调不是函数

function mapById(list, callback) {
    var obj = {};
    list.forEach(function(item) {
        var key = callback(item);
        if (obj[key]) {
            obj[key] = obj[key].concat(item);
        } else {
            obj[key] = [item];
        }
    });
    return obj;
}

console.log(mapById(input));

Answer 1

你可以这样调用这个函数，

mapById(my_list, function(){
    return "my_key"
});

你得到的错误表明你已经将错误的参数作为第二个参数传递，或者你根本没有传递第二个参数。

然而，术语回调在这里没有意义。回调函数用于向调用函数发出被调用函数已完成执行的信号。

对于您的情况，我建议您使用getKey或getKeyByListItem等名称。

Answer 2

问题：

您遇到的错误是，在您定义该功能时，您给出了两个参数：list，callback

function mapById(list, callback) {}

但是，当你调用这个函数时，你只给它 ONE 参数：input

console.log(mapById(input));

因此，当您的代码执行并且它到达函数var key = callback(item);的这一部分时，它会抛出异常，因为回调是未定义，因为您没有传入第二个参数。

解决方案：

很多方法可以解决您的问题。一种解决方案是在第一次声明mapById时简单地调用内部。

然后移动代码逻辑，当 CALL mapById， NOT 时，将对象作为第二个参数返回当你宣布它时。

E.g。

function mapById(list, callback) {
    var obj = {};
    return callback(item);
}

console.log(mapById(input, function() {
    // logic goes here
}));

注意我个人不会这样构造它，因为回调是不必要的，但我告诉你如何使用回调来完成它。

Javascript TypeError：回调不是函数 - 我该如何解决这个问题？

2 个答案:

问题：

解决方案：