Estoy tratando de guardar los valores del sensor en mi base de datos mongodb. Estoy usando la promesa de hacer un manejo adecuado de errores. Estoy usando then() y catch() pero supongo que no lo estoy haciendo bien. Envío una identificación de grupo para que verifique si la identificación de grupo existe y luego solo almacena información del sensor. Aquí está mi código.
router.post('/data/:group_id', function(req,res,next){ var group_id = req.params.group_id; User.find({group_id : group_id}).then(function(user){ var object_id = user[0]._id; var datas = req.body; var data = new Data({ user : object_id, value: datas.value, valueString : datas.valueString, sensorStatus : datas.sensorStatus, timeStamp : new Date().toJSON() }); data.save().then(function(data){ res.send('Data Saved'); }); }).catch(function(e){ if(e.code === undefined){ res.send('Group id does not exist'); }else{ res.send(e); } }); });si envío una identificación de grupo que no existe, dice que la identificación de grupo no existe, pero si envío datos json incorrectos para guardar la información de mi sensor, no hay error, pero mi aplicación de cartero que estoy usando para enviar información obtiene pegado. Aquí hay un error que generé deliberadamente con una ID de grupo válida.
Soy nuevo en esto, así que necesito entender cómo la promesa maneja el error primero si no se puede encontrar el group_id y segundo si la información del sensor json es incorrecta.
Enviado a http://localhost:3000/api/data/3
{ "value1" : "79" , "valueString" : "Small data", "sensorStatus" : "false" }Información extra Usuario y modelo de Datos. Nota (no agregue la parte requerida y exporte)
var schema = new Schema({ group_id : {type : Number, required: true}, password : {type : String, default: null}, project : {type : String , default : null} }); var schema = new Schema({ user: {type: Schema.Types.ObjectId, ref: 'User'}, value :{type : Number , required : true} , valueString : {type : String, required : true}, sensorStatus : {type : Boolean , default : 0}, timeStamp : {type :String , required : true} });Correcto, está cayendo en una trampa común: no anide las cláusulas .then() , en su lugar, devuelva Promesas y encadene:
no hagas
foo().then(() => { bar().then(() => { baz(); } } Devuelve la Promesa desde el interior de .then() y la cadena, así:
foo() .then(() => { return bar(); }) .then(() => { return baz(); });O la versión más corta
foo() .then(() => bar()) .then(() => baz());En tu caso:
router.post('/data/:group_id', function(req,res,next){ var group_id = req.params.group_id; User.find({group_id : group_id}) .then(function(user){ // bla bla bla return data.save(); }) .then(function(data){ return res.send('Data Saved'); }); .catch(function(e){ if(e.code === undefined){ res.send('Group id does not exist'); }else{ res.send(e); } }); }); Como regla común: siempre devuelve o lanza desde un controlador .then() .