为何我的数组为空？DynamoDB数据存入数组后返回空数组问题

你遇到的其实是异步编程里非常常见的踩坑场景：DynamoDB的客户端方法（不管是get/query还是scan）都是异步执行的，你的for循环是同步跑的，但循环里的异步回调还没来得及把数据塞进myjson数组，外部可能就已经去读取这个数组了，所以才会看到空数组。

从你给出的代码片段来看，maketimetabledata里用for循环调用dynamo.docClient的方法，但没有正确等待所有异步请求完成就触发了回调，导致数组还没被填充。

我们可以把每个DynamoDB请求包装成Promise，然后用Promise.all等待所有请求完成后再处理结果，这样就能保证数组被正确填充后再返回。

修改后的完整代码示例 #

const dynamo = require('./dynamo.js');
// 不建议直接暴露全局数组，最好通过回调/返回值传递数据
exports.maketimetabledata = function(callback) {
  const table = "timetable";
  const requestPromises = [];

  // 用let声明i，避免闭包陷阱（var会导致所有异步回调共享同一个i值）
  for (let i = 1; i <= 2; i++) {
    // 替换成你实际的DynamoDB请求参数
    const params = {
      TableName: table,
      Key: {
        id: i // 假设你的主键是id，根据实际表结构调整
      }
    };

    // 把异步请求包装成Promise
    const dbRequest = new Promise((resolve, reject) => {
      dynamo.docClient.get(params, (err, data) => {
        if (err) {
          reject(err);
        } else {
          // 这里根据你实际需要的数据结构调整，比如取data.Item
          resolve(data.Item);
        }
      });
    });

    requestPromises.push(dbRequest);
  }

  // 等待所有异步请求完成
  Promise.all(requestPromises)
    .then(results => {
      // 现在results就是所有请求返回的数据组成的数组
      callback(null, results);
    })
    .catch(err => {
      // 处理错误，传递给回调
      callback(err);
    });
};

几个关键的改进细节 #

1. 解决闭包陷阱：用let声明循环变量i，如果用var的话，所有异步回调会共享同一个i值，导致最后所有请求都用的是循环结束后的i（也就是3）。
2. 统一异步流程：用Promise包装每个DynamoDB请求，再通过Promise.all等待所有请求完成，确保所有数据都拿到后再生成结果数组。
3. 避免全局数组状态：直接通过回调返回结果数组，而不是依赖全局的myjson变量，减少状态管理的混乱。

如果你的需求是批量获取多条数据，其实可以直接用DynamoDB的batchGet API，比循环调用单条get请求更高效，减少网络开销。

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