Node.js MongoDB调试：查询结果正常但回调参数为undefined问题排查

嘿，这个问题其实是回调参数顺序不符合Node.js错误优先回调规范导致的，咱们一步步理清楚：

问题根源 #

你看你的dadosCliente方法里，调用回调的时候是这么写的：

callback(result);

但控制器里的回调函数定义是：

function(err, dadosCliente){ ... }

Node.js里的标准回调规范是第一个参数传错误对象（无错误则传null），第二个参数才是结果数据。你现在直接把查询结果传给了第一个参数，就导致：

• 控制器里的err变量其实拿到的是查询结果（非null，所以if(err)判断为true，进入错误分支）
• dadosCliente因为没有传第二个参数，自然就是undefined

这就完美解释了日志里的矛盾：明明数据库返回了结果，但控制器里dadosCliente是undefined，还触发了错误分支。

修复步骤 #

1. 修正dadosCliente方法的回调调用，同时完善全链路错误处理

把各个环节的错误都正确传递，并且按规范传递结果：

ClientesDAO.prototype.dadosCliente = function(clienteid, callback){ 
  console.log('clienteid em dadosCliente', clienteid) 
  var client_id = new ObjectId.ObjectID(clienteid); 
  this._connection.open(function(err, mongoClient){ 
    // 处理数据库连接错误
    if (err) {
      return callback(err);
    }
    mongoClient.collection('clientes', function(err,collection){ 
      // 处理集合获取错误，记得先关闭连接
      if (err) {
        mongoClient.close();
        return callback(err);
      }
      collection.find({_id:client_id}).toArray(function(err,result){ 
        // 无论查询成功失败，先关闭数据库连接
        mongoClient.close();
        // 处理查询错误
        if (err) {
          return callback(err);
        }
        console.log('dadosCliente result', result) 
        // 正确传递参数：第一个为null（无错误），第二个为查询结果
        callback(null, result); 
      }) 
    }) 
  }) 
}

2. 完善控制器的错误响应逻辑（可选但重要）

原来的控制器里只是return (err);，这不会给客户端返回任何有效响应，建议补充响应逻辑：

module.exports.cliente = function(application, req, res){ 
  var cliente_id = req.query.id; 
  console.log('cliente_id em cliente', req.query.id) 
  var connection = application.config.dbConnection; 
  var ClientesDAO = new application.app.models.backOffice.ClientesDAO(connection); 
  ClientesDAO.dadosCliente(cliente_id, function(err, dadosCliente){ 
    console.log('dadosCliente', dadosCliente) 
    if (err) { 
      console.log('entrou err getDadosCliente') 
      // 返回500错误信息给客户端
      return res.status(500).json({ error: err.message }); 
    } 
    console.log('entrou resolve getDadosCliente') 
    console.log('dadosClienteE', dadosCliente)
    // 这里可以添加渲染页面或返回数据的逻辑，比如：
    res.render('cliente', { cliente: dadosCliente[0] });
  }) 
}

补充小提示 #

Node.js生态里几乎所有异步API都遵循错误优先回调规范，记住这个规则：回调第一个参数永远是错误对象，成功时传null，结果放在后面的参数里，这样就不会再出现这种参数错位的低级问题啦！

内容的提问来源于stack exchange，提问作者EFO