假设我有以下代码。
function divide(numerator, denominator) {
return new Promise((resolve, reject) => {
if(denominator === 0){
reject("Cannot divide by 0");
return; //superfluous?
}
resolve(numerator / denominator);
});
}
如果我的目标是使用reject早退出，我是否也应该养成使用后return立即退出的习惯？
的return目的是拒绝后终止函数的执行，防止代码执行后它。
function divide(numerator, denominator) {
return new Promise((resolve, reject) => {
if (denominator === 0) {
reject("Cannot divide by 0");
return; // The function execution ends here
}
resolve(numerator / denominator);
});
}
在这种情况下，它会阻止resolve(numerator / denominator);执行，这不是严格需要的。但是，最好还是终止执行以防止将来出现可能的陷阱。此外，防止不必要地运行代码也是一种很好的做法。
背景
Promise可以处于以下三种状态之一：
当一个Promise被履行或被拒绝时，它将无限期地保持在这个状态（已解决）。因此，拒绝已履行的Promise或履行已拒绝的Promise，都不会产生影响。
此示例代码段表明，虽然Promise在被拒绝后得到了履行，但它仍然被拒绝。
function divide(numerator, denominator) {
return new Promise((resolve, reject) => {
if (denominator === 0) {
reject("Cannot divide by 0");
}
resolve(numerator / denominator);
});
}
divide(5,0)
.then((result) => console.log('result: ', result))
.catch((error) => console.log('error: ', error));那么我们为什么需要返回呢？
虽然我们不能改变一个已解决的Promise状态，但拒绝或解决不会停止函数其余部分的执行。该函数可能包含会产生混淆结果的代码。例如：
function divide(numerator, denominator) {
return new Promise((resolve, reject) => {
if (denominator === 0) {
reject("Cannot divide by 0");
}
console.log('operation succeeded');
resolve(numerator / denominator);
});
}
divide(5, 0)
.then((result) => console.log('result: ', result))
.catch((error) => console.log('error: ', error));即使函数现在不包含这样的代码，这也会造成未来可能的陷阱。未来的重构可能会忽略在 promise 被拒绝后代码仍在执行的事实，并且将难以调试。
解决/拒绝后停止执行：
这是标准的 JS 控制流的东西。
resolve/之后返回reject：resolve/ reject- 返回，因为回调的返回值被忽略，我们可以通过返回reject/resolve语句来保存一行：我更喜欢使用其中一个return选项，因为代码更扁平。
一个常见的习语，可能是也可能不是你的一杯茶，是将 thereturn与 the结合起来reject，同时拒绝 promise 并退出函数，这样包括 the 在内的函数的其余部分resolve不会被执行。如果你喜欢这种风格，它可以让你的代码更紧凑一些。
function divide(numerator, denominator) {
return new Promise((resolve, reject) => {
if (denominator === 0) return reject("Cannot divide by 0");
^^^^^^^^^^^^^^
resolve(numerator / denominator);
});
}
因为无极构造方法不具有任何返回值，并且在任何情况下也能正常工作resolve和reject任何回报。
相同的习语可以与另一个答案中显示的回调样式一起使用：
function divide(nom, denom, cb){
if(denom === 0) return cb(Error("Cannot divide by zero"));
^^^^^^^^^
cb(null, nom / denom);
}
同样，这工作正常，因为调用divide者不希望它返回任何内容，并且不会对返回值执行任何操作。
如果您在解决/拒绝之后没有“返回”，那么在您打算停止之后可能会发生不好的事情（例如页面重定向）。来源：我遇到了这个。
技术上在这里不需要它1 -因为Promise可以解决或者拒绝，仅仅只有一次。第一个 Promise 结果获胜，随后的每个结果都将被忽略。这是不同的节点样式的回调。
话虽如此，但在可行的情况下，确保确切地调用一个是一种很好的清洁实践，实际上在这种情况下，因为没有进一步的异步/延迟处理。“提前返回”的决定与在其工作完成后结束任何功能没有什么不同- 与继续无关或不必要的处理。
在适当的时间返回（或以其他方式使用条件来避免执行“其他”情况）减少了意外在无效状态下运行代码或执行不需要的副作用的机会；因此，它使代码不太容易“意外中断”。
1从技术上讲，这个答案也取决于这样一个事实，即在这种情况下，“返回”之后的代码如果被省略，将不会导致副作用。JavaScript 很乐意除以零并返回 +Infinity/-Infinity 或 NaN。
Ori 的回答已经解释说没有必要，return但这是一种很好的做法。请注意，promise 构造函数是抛出安全的，因此它会忽略路径中稍后传递的抛出的异常，本质上您有无法轻易观察到的副作用。
请注意，returning early 在回调中也很常见：
function divide(nom, denom, cb){
if(denom === 0){
cb(Error("Cannot divide by zero");
return; // unlike with promises, missing the return here is a mistake
}
cb(null, nom / denom); // this will divide by zero. Since it's a callback.
}
因此，虽然在 Promise 中这是一个很好的实践，但回调是必需的。关于您的代码的一些说明：