MongoDB
 sql >> Base de données >  >> NoSQL >> MongoDB

Express Mongoose Model.find() renvoie undefined

Cela semble correct, mais vous oubliez le comportement asynchrone de Javascript :). Lorsque vous codez ceci :

module.exports.getAllTasks = function(){
        Task.find().lean().exec(function (err, docs) {
        console.log(docs); // returns json
    });
}

Vous pouvez voir la réponse json car vous utilisez un console.log instruction À L'INTÉRIEUR du rappel (la fonction anonyme que vous passez à .exec()) Cependant, lorsque vous tapez :

app.get('/get-all-tasks',function(req,res){
    res.setHeader('Content-Type', 'application/json');
    console.log(Task.getAllTasks()); //<-- You won't see any data returned
    res.json({msg:"Hej, this is a test"}); // returns object
});

Console.log exécutera getAllTasks() fonction qui ne renvoie rien (indéfini) car la chose qui renvoie vraiment les données que vous voulez est À L'INTÉRIEUR du rappel...

Donc, pour le faire fonctionner, vous aurez besoin de quelque chose comme ceci :

module.exports.getAllTasks = function(callback){ // we will pass a function :)
        Task.find().lean().exec(function (err, docs) {
        console.log(docs); // returns json
        callback(docs); // <-- call the function passed as parameter
    });
}

Et on peut écrire :

app.get('/get-all-tasks',function(req,res){
    res.setHeader('Content-Type', 'application/json');
    Task.getAllTasks(function(docs) {console.log(docs)}); // now this will execute, and when the Task.find().lean().exec(function (err, docs){...} ends it will call the console.log instruction
    res.json({msg:"Hej, this is a test"}); // this will be executed BEFORE getAllTasks() ends ;P (because getAllTasks() is asynchronous and will take time to complete)
});