Node.js异步方法无法返回数据库查询结果，寻求解决办法

Hey there, I totally get where you're coming from—async code in Node.js can trip up even experienced devs at first. Let's break down why your search function is returning undefined, then walk through a few solid fixes.

The Root Cause #

Your MongoDB find().toArray() call is asynchronous. That means when your search function finishes executing, the callback inside toArray hasn't run yet—so the function doesn't have any value to return, hence undefined. The result only exists inside that callback, but by then the function has already exited.

Let's look at three common ways to fix this:

1. Use a Callback Parameter (Classic Approach) #

The simplest fix is to modify your search function to accept a callback, which you'll call once the MongoDB query finishes. This matches the existing callback style of the MongoDB driver.

Modified code:

var mongo = require('../routes/mongo')

exports.search = function(userid, callback) { // Add callback parameter
  mongo.get().collection("customers").find({"userid": userid}).toArray(function(err, result) {
    if (err) {
      return callback(err); // Pass error to callback
    }
    callback(null, result); // Pass result to callback
  });
}

To use it:

// When calling the function
yourModule.search("target-user-id", function(err, data) {
  if (err) {
    console.error("Error fetching data:", err);
    return;
  }
  console.log("Got results:", data); // This will have your customers data
});

2. Return a Promise (Modern Async Pattern) #

Promises make async code easier to reason about and avoid "callback hell". You can wrap the MongoDB call in a Promise and return it from your search function.

Modified code:

var mongo = require('../routes/mongo')

exports.search = function(userid) {
  return new Promise((resolve, reject) => {
    mongo.get().collection("customers").find({"userid": userid}).toArray(function(err, result) {
      if (err) {
        return reject(err); // Reject on error
      }
      resolve(result); // Resolve with the result
    });
  });
}

To use it:

// Using .then()/.catch()
yourModule.search("target-user-id")
  .then(result => console.log("Got results:", result))
  .catch(err => console.error("Error:", err));

3. Use Async/Await (Cleanest Syntax) #

If you're using Node.js 8+, async/await lets you write async code that looks synchronous. It's built on top of Promises, so we'll use the Promise version of your function as a base.

Modified code (most modern MongoDB drivers support Promises natively, simplifying this):

var mongo = require('../routes/mongo')

exports.search = async function(userid) {
  const collection = mongo.get().collection("customers");
  const result = await collection.find({"userid": userid}).toArray();
  return result;
}

To use it:

// Inside an async function
async function fetchCustomers() {
  try {
    const data = await yourModule.search("target-user-id");
    console.log("Got results:", data);
  } catch (err) {
    console.error("Error fetching data:", err);
  }
}

fetchCustomers();

Key Takeaway #

Async operations like database queries don't block the code—so you can't just return the result directly. You need to use callbacks, Promises, or async/await to handle the result once it's ready.

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