不使用回调时返回未定义的函数，使用回调时，JS说函数未定义

Question

请考虑到在 SO 上已经提出了类似的问题，我已经回答了其中的大部分问题。

我正在制作一个需要查询数据库以获取数据的 RESTful 服务。我编写了正确查询数据库但始终返回未定义的代码。代码在这里：

function returnAll(){
    ModuleDBService.find({},function(err,data){
        if(err){
            console.log('Error occured while retrieving the documents!');
        }
        return data;
    });
}

我正在使用以下方式导出模块：

module.exports = {
    getAll:returnAll
};

在挖掘了很多之后，我发现我需要使用回调来获取数据。我浏览了许多示例并尝试将类似的技术应用于我的代码，修改后的代码如下所示：

function getAllFromDatabase(callback){
    ModuleDBService.find({},function(err,data){
        if(err){
            console.log('Error occured while retrieving the documents!');
        }
        callback(returnAll(data));
    });
}


function returnAll(data){ return data;}

然后以与上述类似的方式返回它。

但现在我收到 ModuleDAO.getAll 不是函数的错误（我正在使用 var ModuleDAO = require('path to the database service')。

我尝试了许多代码变体，浏览了 YouTube 上的几个视频，所有这些视频要么导致返回未定义，要么返回上述错误。如果有人可以修复代码并阐明整个回调的事情（或者可以提供可靠的文档来理解它），那将是一个很大的帮助。

谢谢。

编辑：在所有非常有用的答案之后，这里是一个摘要：

回调不能返回数据，请传递您希望程序使用数据调用的函数（回调函数）。就我而言，是我的路由器返回数据。

以下是更正后的代码：

function returnAll(callback) {
    ModuleDBService.find({}, function (err, data) {
        if (err) {
            console.log("Error while retrieving the document!")
            callback(null);
        }
        callback(data);
    });
}

我在路由器中使用此代码：

mainAPIRouter.post('/api/module', function (req, res) {
    try {
        moduleDAO.getAll(function(data){
            res.status(200);
            res.json(data);
        });
    } catch (error) {
        res.status(500);
        return res.send("Invalid request");
    }
});

感谢所有帮助过的人！ :)

Answer 1

你很接近。您不需要returnAll() 函数，您需要导出getAllFromDatabase 并将callback 传递给它：

function getAllFromDatabase(callback){
    ModuleDBService.find({},function(err,data){
        if(err) {
            console.log('Error occured while retrieving the documents!');
        }
        callback(data);
    });
}

module.exports = {
    getAllFromDatabase: getAllFromDatabase
};

那么，当你想使用它的时候，你需要一个回调函数：

dataModule.getAllFromDatabase(callbackHandler);

function callbackHandler(dataFromDatabase) {
    // this function will be executed when ModuleDBService executes the callback
    console.log(dataFromDatabase);
}

一个小细节：如果err是Truthy，你不应该执行回调：

if(err) {
    console.log('Error occured while retrieving the documents!');
} else {
    callback(data);
}

Answer 2

您想简单地调用callback()，并将您需要的数据作为参数。通过将另一个函数传递给回调，您使事情变得更加复杂。尝试类似：

function returnAll(callback) {
  ModuleDBService.find({}, function(err, data) {
    if (err) return callback(err)
    callback(null, data);
  });
}

returnAll(function(err, data)) {
// it's customary for callbacks to take an error as their first argument
  if (err) {
    console.log('Error occured while retrieving the documents!');
  } else {
    // use data here!!
  }
}

Answer 3

如前所述，您可以使用回调。如果您愿意，也可以使用 Promise：

function returnAll(){
    return new Promise(function(resolve, reject) {
    ModuleDBService.find({},function(err,data){
        if(err){
            console.log('Error occured while retrieving the documents!');
                reject(err);
        }
          resolve(data);
    });
     });
}

然后你可以使用类似的东西来访问它：

returnAll()
  .then(data=> {console.log(data); })
  .catch(err=> { console.log(err); });

*编辑：既然你想使用回调，我想我也会在那里加上我的 0.02 美元。最简化的方法是只使用您通过ModuleDBService.find 传入的回调，而不使用临时函数。但是最好验证一下回调实际上是一个函数，如果不把它变成一个......让你的代码更容错。

function returnAll(cb){
    if(typeof cb!=='function') cb = function() {};
    ModuleDBService.find({},cb);
}