我应该在 Javascript 中将“return”与我的回调函数一起使用吗？

Question

我在我的 JS 应用程序中使用异步函数，并用我自己的函数来接收回调输入。当我调用回调函数时，是否需要使用“return”关键字？有关系吗？有什么区别？

例如：

var getData = function(callback){
    // do some aysnc db stuff...
    return callback(results);
    // or just
    callback(results);
}

PS：我正在使用 javascript 编写一个混合移动应用程序。

Answer 1

不，回调不应与 return 一起使用。
我从不期望回调函数返回一个值。 当（异步）计算完成时，回调在传递值方面替换了返回。

您可以使用此语法 return callback(result) 作为 callback(result); return; 之类的快捷方式，但它可能会使其他团队成员感到困惑，可能会返回什么样的值回调。
这实际上是你的缩小器的任务，创建这样的代码；不是你的。

Answer 2

如果你的函数只有一条路径，那么你可以互换使用这两种形式。当然，如果没有返回，函数的返回值将是未定义的，但您的调用代码可能并没有使用它。

这是真的

return callback()

实际上等同于

callback(result); return;

后者确实会在调用堆栈上产生额外的帧，因此会使用更多资源。我想如果你有很多嵌套的回调，或者正在进行递归，你会更快地耗尽堆栈空间。

这可能是个坏主意，我不认为我在说回调之前的 return 更惯用。

当您的函数中有多个路径时，您必须小心。例如，这将按您的预期工作：

(cb)=> {
    if (something) cb('a')
    else cb('b')
}

但是，在这种情况下，两个回调都会被调用。

(cb)=> {
    if (something) cb('a');

    cb('b')
}

当您阅读以上内容时，很明显两者都会被调用。然而，编写这样的代码是一个典型的节点新手错误（尤其是在处理错误时）。如果您想要或被执行，您需要：

(cb)=> {
    if (something) return cb('a');

    cb('b')
}

Answer 3

你不必这样做，但如果你不使用'return'，你可能会遇到麻烦。例如，如果您不习惯提前返回，错误处理可能会出现问题。举个例子：

var getData = function(callback){
  dbStuff(function(err, results) {
    if (err) { callback(err, null); }
    callback(null, results);
  });
}

我通常认为完成后返回是一种好习惯......

var getData = function(callback){
  dbStuff(function(err, results) {
    if (err) { return callback(err, null); }
    return callback(null, results);
  });
}

但是你可以用 if/else 块和没有 return 语句来完成同样的事情。