使用猫鼬无法向数据库添加多个对象

Question

I have a function that is supposed to add new item each time there is a match as below:

    function getTimeEntriesFromWorksnap(error, response, body) {
    //console.log(response.statusCode);
    var counter = 1;
    if (!error && response.statusCode == 200) {
        parser.parseString(body, function (err, results) {
            var json_string = JSON.stringify(results.time_entries);
            var timeEntries = JSON.parse(json_string);

            _.forEach(timeEntries, function (timeEntry) {
                _.forEach(timeEntry, function (item) {
                    Student.findOne({'worksnap.user.user_id': item.user_id[0]})
                        .populate('user')
                        .exec(function (err, student) {
                            if (err) {
                                throw err;
                            }

                            var newTimeEntry = _pushToObject(student.worksnap.timeEntries, item);

                            student.worksnap.timeEntries = {};
                            student.worksnap.timeEntries = newTimeEntry;

                            student.save(function (err) {
                                if (err) {
                                    //return res.status(400).send({
                                    //    message: errorHandler.getErrorMessage(err)
                                    //});
                                } else {
                                    //res.json(item);
                                }
                            });

                        });
                });
            });
        });
    }
}

For some reason it is only inserting once for each student that it finds. And my Student schema looks like this:

    var StudentSchema = new Schema({
    firstName: {
        type: String,
        trim: true,
        default: ''
        //validate: [validateLocalStrategyProperty, 'Please fill in your first name']
    },
    lastName: {
        type: String,
        trim: true,
        default: ''
        //validate: [validateLocalStrategyProperty, 'Please fill in your last name']
    },
    worksnap: {
        user: {
            type: Object
        },
        timeEntries: {
            type: Object
        },
     }
   });

Any solution?

Answer 1

My Guess is its always pushing the last one... closures ...

function getTimeEntriesFromWorksnap(error, response, body) {
    //console.log(response.statusCode);
    var counter = 1;
    if (!error && response.statusCode == 200) {
        parser.parseString(body, function (err, results) {
            var json_string = JSON.stringify(results.time_entries);
            var timeEntries = JSON.parse(json_string);

            _.forEach(timeEntries, function (timeEntry) {
                _.forEach(timeEntry, function (item) {
                    saveStudent(item);
               });
            });
        });
    }
}

Below is saveStudent function

function saveStudent(item) {

    Student.findOne({
            'worksnap.user.user_id': item.user_id[0]
        })
        .populate('user')
        .exec(function(err, student) {
            if (err) {
                throw err;
            }

            var newTimeEntry = _pushToObject(student.worksnap.timeEntries, item);

            student.worksnap.timeEntries = {};
            student.worksnap.timeEntries = newTimeEntry;

            student.save(function(err) {
                if (err) {
                    //return res.status(400).send({
                    //    message: errorHandler.getErrorMessage(err)
                    //});
                } else {
                    //res.json(item);
                }
            });

        });
}

OR

wrap it inside a closure...

function getTimeEntriesFromWorksnap(error, response, body) {
    //console.log(response.statusCode);
    var counter = 1;
    if (!error && response.statusCode == 200) {
        parser.parseString(body, function(err, results) {
            var json_string = JSON.stringify(results.time_entries);
            var timeEntries = JSON.parse(json_string);

            _.forEach(timeEntries, function(timeEntry) {
                _.forEach(timeEntry, function(item) {
                  (function(item){
                    Student.findOne({
                            'worksnap.user.user_id': item.user_id[0]
                        })
                        .populate('user')
                        .exec(function(err, student) {
                            if (err) {
                                throw err;
                            }

                            var newTimeEntry = _pushToObject(student.worksnap.timeEntries, item);

                            student.worksnap.timeEntries = {};
                            student.worksnap.timeEntries = newTimeEntry;

                            student.save(function(err) {
                                if (err) {
                                    //return res.status(400).send({
                                    //    message: errorHandler.getErrorMessage(err)
                                    //});
                                } else {
                                    //res.json(item);
                                }
                            });

                        });
                  }(item))
                });
            });
        });
    }
}

Can you use async library and check out...

var async = require('async');
async.map(timeEntries, function (timeEntry, next) {
   async.map(timeEntry, function (item, next) {
      //your code.
  });
});