Promise拒绝时出现Cannot read property 'then' of undefined错误求助

这个问题我太熟了！本质上是你service层的函数没有始终返回一个Promise实例，导致路由层调用它的时候拿到的是undefined，自然就没法调用.then()了。咱们一步步拆解：

最常见的错误场景 #

你大概率是在service层的业务逻辑里，某个分支（比如检测到用户存在的分支）没有返回Promise，或者忘记返回整个Promise链。举个典型的错误写法：

// 错误的service层函数示例
function createUser(userData) {
  db.find({ email: userData.email })
    .then(existingUser => {
      if (existingUser) {
        // 这里抛错，但外层函数没返回Promise链！
        throw new Error('用户已存在');
      }
      return db.insert(userData);
    })
  // 注意：这里没有return任何东西，函数默认返回undefined
}

当路由层调用createUser(req.body).then(...)时，createUser返回的是undefined，调用.then()自然就会抛出你看到的错误。

两种正确的写法 #

根据你习惯的异步风格，选下面一种即可：

1. Async/Await写法（推荐，逻辑更清晰） #

用async函数的话，不管是正常返回还是抛出错误，都会自动包装成Promise实例，从根源避免返回undefined的问题：

// 正确的service层函数（async/await版）
async function createUser(userData) {
  const existingUser = await db.find({ email: userData.email });
  
  if (existingUser) {
    // 抛错会被async自动转为rejected状态的Promise
    throw new Error('用户已存在');
  }
  
  // 正常插入，返回的结果会被包装为resolved状态的Promise
  return await db.insert(userData);
}

// 路由层调用（没问题）
router.post('/users', async (req, res) => {
  try {
    const user = await createUser(req.body);
    res.status(201).json(user);
  } catch (err) {
    res.status(400).json({ message: err.message });
  }
});

2. Promise链式写法 #

如果习惯用传统的Promise链式调用，一定要记得return整个Promise链：

// 正确的service层函数（Promise链式版）
function createUser(userData) {
  // 关键：return整个Promise链！
  return db.find({ email: userData.email })
    .then(existingUser => {
      if (existingUser) {
        // 直接返回rejected的Promise，或者throw（then回调里的throw会自动转为reject）
        return Promise.reject(new Error('用户已存在'));
      }
      return db.insert(userData);
    })
    // 可选：在这里统一处理数据库底层错误，再向上抛出
    .catch(dbError => {
      throw new Error(`数据库操作失败：${dbError.message}`);
    });
}

// 路由层调用（没问题）
router.post('/users', (req, res) => {
  createUser(req.body)
    .then(user => res.status(201).json(user))
    .catch(err => res.status(400).json({ message: err.message }));
});

避坑总结 #

• 不管用哪种异步写法，service层的函数必须始终返回Promise，不能在任何分支里返回undefined。
• 用Promise链式写法时，千万不要忘记return开头的db.find(...)整个链条。
• 用async函数时，只要确保每个分支要么return值，要么throw错误，就不会出现返回undefined的情况。

内容的提问来源于stack exchange，提问作者francesco.venica