嵌套回调构建家族树异常：无法稳定获取5代数据求助

首先，你的代码不稳定的核心原因是异步回调的无序性和缺乏统一等待机制。虽然你用了立即执行函数解决了循环变量的作用域问题，但每次调用getChildren都是异步的——数据库查询需要时间，你的代码只是触发了这些查询，却没有等待所有查询完成就结束了。比如，当你处理到第3代节点时，它的子节点查询可能还在数据库那边跑，而你的函数已经继续执行其他逻辑，导致最终返回的家族树里某些深层节点的children还没被赋值，所以才会出现有时拿到3代、有时拿到4代的情况。

另外，你的嵌套回调写法（俗称“回调地狱”）不仅难维护，而且一旦某一层的异步请求延迟或出错，后续的节点填充就会中断，你甚至很难定位问题出在哪。

最优解决方案：用Promise + Async/Await重构 #

我们可以先把回调式的getChildren改成Promise版本，然后用递归+async/await来同步化异步流程，确保每一代的节点都等待子节点查询完成后再继续。

步骤1：把getChildren转为Promise版本

// 重构getChildren为Promise风格，方便后续用async/await
const getChildren = (person) => {
  return new Promise((resolve, reject) => {
    keystone.list('Person').model.find()
      .where('father', person.id)
      .exec((err, children) => {
        if (err) {
          // 新增错误处理，避免静默失败
          console.error(`获取${person.id}的子节点出错:`, err);
          reject(err);
        } else {
          resolve(children);
        }
      });
  });
};

步骤2：写递归的家族树构建函数

// 递归构建家族树，generationsLeft控制要获取的代际（默认5代）
async function buildFamilyTree(person, generationsLeft = 5) {
  // 终止条件：如果已经到了第5代，或者没有更多代要查，直接返回当前节点
  if (generationsLeft <= 0) return person;

  // 等待当前节点的子节点查询完成
  const children = await getChildren(person);
  person.children = children;

  // 等待所有子节点的家族树都构建完成（Promise.all确保所有异步操作都完成）
  await Promise.all(
    children.map(child => buildFamilyTree(child, generationsLeft - 1))
  );

  return person;
}

步骤3：调用函数获取完整家族树

// 使用示例：传入初始的家族成员（比如家族的第一代）
const initialPerson = { id: '第一代成员的ID' }; // 替换成你的初始person对象

buildFamilyTree(initialPerson)
  .then(completeFamilyTree => {
    console.log('完整的5代家族树:', completeFamilyTree);
    // 这里可以处理构建好的家族树，比如渲染到页面
  })
  .catch(err => {
    console.error('构建家族树时发生错误:', err);
  });

为什么这个方案能解决问题？ #

1. 异步等待可控：await关键字会暂停函数执行，直到当前的getChildren查询完成，确保每一代的子节点都被正确赋值后才继续处理下一层。
2. 批量等待异步操作：Promise.all会等待所有子节点的递归构建完成，保证整个家族树的所有节点都被填充完毕后才返回结果。
3. 错误处理清晰：新增的错误捕获可以帮你快速定位哪一次数据库查询出了问题，避免静默失败导致的家族树不完整。
4. 代码可读性高：递归+async/await完全替代了多层嵌套的回调，逻辑一目了然，后续维护也更轻松。

如果你暂时不想用Promise（不推荐） #

如果因为环境限制必须用回调，你可以用计数器来跟踪所有异步操作的完成状态，比如每完成一个getChildren调用就减计数器，直到计数器为0时再触发回调返回完整家族树。但这种写法依然会比较繁琐，还是推荐用Promise方案。

内容的提问来源于stack exchange，提问作者Phạm Quang Mẫn