是否可以返回 false，通过异步函数传递 ESLint 一致返回错误？

Question

我正在使用 ExpressJS，需要在我的代码上运行和传递 ESLint 规则。有一个规则“consistent-return”，会抛出这样的代码：

function getUsers( req, res, next ){
  if( req.userIds.length === 0 ){
    return next();
  }

  collection.find({"_id": {$in: req.userIds}}, function( err, doc ){
    if( err ){
      return next();
    }

    req.users = doc;
    return next();
  });//find()
}//getUsers()

现在，此代码给出“一致返回”错误。似乎有 2 个可能的更改可以消除此错误，我不确定哪个是正确的。

function getUsers( req, res, next ){
  if( req.userIds.length === 0 ){
    return next();
  }

  collection.find({"_id": {$in: req.userIds}}, function( err, doc ){
    if( err ){
      return next();
    }

    req.users = doc;
    return next();
  });//find()

  return false; // adding return false passes the ESLint error
}//getUsers()

或

 function getUsers( req, res, next ){
  if( req.userIds.length === 0 ){
    return next();
  }

  // adding return in below line passes the ESLint error.
  return collection.find({"_id": {$in: req.userIds}}, function( err, doc ){
    if( err ){
      return next();
    }

    req.users = doc;
    return next();
  });//find()
}//getUsers()

请告知哪种方法是正确的。谢谢。

Answer 1

这里的规则指向一个真正的一致性问题：有时您似乎返回了一个值，而有时您却没有。它只起作用，因为返回的值被忽略了。您的return 语句的唯一要点是从执行函数的其余部分分支出来。

您的代码具有误导性，会损害可读性。

替换

if( err ){
  return next();
}

与

if( err ){
  next();
  return;
}

整个不混乱的代码：

function getUsers( req, res, next ){
    if( req.userIds.length === 0 ){
        next();
        return
    }
    collection.find({"_id": {$in: req.userIds}}, function( err, doc ){
        if( !err ){
            req.users = doc;
        }
        next();
    });//find()
}//getUsers()

Answer 2

让我们稍微分析一下你的代码，去掉一些细节：

function getUsers( req, res, next ){

  if( req.userIds.length === 0 ){
    return next();
  }

  collection.find({"_id": {$in: req.userIds}}, function( err, doc ){
    // lotsa code here
  });

}

这里，在函数getUsers() 内，可以有两个代码路径：错误条件（当req.userIds 为空时）或不。

根据是否满足错误条件，函数getUsers() 要么返回一些东西（next()），要么什么都不返回（实际上隐式返回undefined）。请注意，collection.find() 在第二种情况下执行，它本身可能会在其中返回一些内容，但getUsers() 不会知道。

现在，作为 ESLint docs 状态，函数可以返回某些内容或不返回任何内容（隐式 undefined），但不能同时返回（这是不一致的行为）；否则会抛出 consistent-return 错误。

因此，要解决该错误，Denys 的代码和您自己的第二个可能的更改都将起作用。在 Denys 的代码中，两个代码路径均不返回任何内容（隐式 undefined），而在您的代码中，它们都返回一些内容。