javascript - 返回异步回调作为最佳实践？

Question

到目前为止，我从未返回过任何异步回调，我只是调用了它们。我不知道(愚蠢的我!)回调调用之后的代码也会被执行。

这表明了我的意思:

var asyncFunctionNoReturn = function(callback) {
    alert('Inside asyncFunctionNoReturn');
    callback();
    alert('Still inside asyncFunctionNoReturn');
};

var asyncFunctionReturn = function(callback) {
    alert('Inside asyncFunctionReturn');
    return callback();
    alert('You will never see me! :(');
};

asyncFunctionNoReturn(function() {
    alert('I am called back!');
});

asyncFunctionReturn(function() {
    alert('I am called back!');
});

对我来说，只有在回调调用后有代码时，这才有意义:

版本 1:

...
if(err) {
    callback(err);
} else {
    callback(null, data);
}

版本 2:

...
if(err) return callback(err);
return callback(null, data);

我见过这两个版本都被使用过。哪一项已被确立为“最佳实践”？

我的猜测是版本 2:

• 更少的代码和更少的嵌套(-> 更好的可读性？)
• 可以更好地扩展

我希望这不是太基于观点。

Answer 1

首先，我并不总是同意 Douglas Crockford 的观点(天啊，我说什么？;-))，但我认为他关于 if 语句的说法是正确的:它应该始终用括号( http://javascript.crockford.com/code.html )。

我不同意你的观点:更少的代码并不意味着更好的可读性，对我来说，第一个版本对于代码的意图更加明显。让我们现在拥有的(令人惊叹的)工具来担心缩小/丑化生产代码。

不过，您仍然可以(应该？)返回回调的结果。

所以我会写:

if (err) {
    return callback(err);
} else {
    return callback(null, data);
}