KeystoneJS中CD_Book与Musicians关联展示问题及优化咨询

我来帮你拆解下遇到的两个核心问题：模板无法显示数据的原因，以及查询方式的优化方向。

一、模板无法显示数据的核心原因与解决方法 #

你控制台能打印出数据，但模板里看不到，本质是异步操作的时序问题：

你的循环里调用的keystone.list("Musician").model.findOne()是异步数据库请求，当你在循环结束后立刻调用next(err)时，这些异步请求还没执行完成，locals.data.musicians此时还是空数组或者未完全填充，模板渲染时自然拿不到数据。

解决这个问题的关键是等待所有异步查询完成后再调用next()，推荐用async/await来处理异步流程，代码更直观：

// 把回调函数改成async函数
keystone.list('CD_Book').model.findOne({ slug: locals.filters.cdbook }).exec(async function(err, cdBookResult) {
  if (err) return next(err); // 错误优先处理
  locals.data.cdbook = cdBookResult;
  
  // 先判断是否有音乐人字符串，避免空值报错
  if (!cdBookResult?.musicians) {
    locals.data.musicians = [];
    return next();
  }
  
  // 分割字符串并去除每个标题的前后空格
  const musicianTitles = cdBookResult.musicians.split(',').map(title => title.trim());
  
  // 等待音乐人查询完成（这里同时优化了查询方式，后面会讲）
  const musicians = await keystone.list("Musician").model.find({ "title": { $in: musicianTitles } }).exec();
  
  locals.data.musicians = musicians;
  console.log(locals.data.musicians);
  next(); // 所有数据准备好后再触发模板渲染
});

这样修改后，next()会在音乐人数据完全填充到locals.data.musicians之后才调用，模板渲染时就能拿到完整数据了。

二、循环查询的优化方案：从N次请求到1次请求 #

你当前的循环查询方式确实不是最优解——每一个音乐人都发起一次数据库请求，当音乐人数量多的时候，会产生大量重复请求，拖慢性能。

更高效的做法是用MongoDB的$in操作符，一次查询所有匹配的音乐人，就像上面代码里写的那样：

keystone.list("Musician").model.find({ "title": { $in: musicianTitles } }).exec();

这个查询会一次性返回所有标题在musicianTitles数组里的音乐人，把数据库请求次数从N次降到1次，性能提升非常明显。

进阶优化：改用KeystoneJS的关联字段（推荐） #

其实你当前用字符串存储音乐人列表的方式，本身就不太符合数据库设计的最佳实践。如果可能的话，建议把CD_Book模型里的musicians字段改成关联字段（Relationship），这样KeystoneJS会帮你自动处理关联关系，查询起来更方便：

1. 修改CD_Book模型的字段定义

// 在CD_Book模型的fields里添加
musicians: { 
  type: Types.Relationship, 
  ref: 'Musician', // 关联到Musician模型
  many: true // 支持多个音乐人
}

2. 查询时直接关联获取音乐人

这样你查询CD_Book的时候，只需要用populate方法就能直接拿到关联的音乐人数据，完全不需要手动分割字符串和查询：

keystone.list('CD_Book').model.findOne({ slug: locals.filters.cdbook })
  .populate('musicians') // 关联查询音乐人
  .exec(function(err, cdBookResult) {
    if (err) return next(err);
    locals.data.cdbook = cdBookResult;
    locals.data.musicians = cdBookResult.musicians; // 直接拿到关联数据
    next();
  });

这种方式不仅代码更简洁，还能避免字符串分割可能带来的错误（比如音乐人标题里包含逗号的情况），是更规范的实现方式。

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