Node.js多图上传异常：循环重复创建条目，移出循环字段未定义

我看到你遇到的问题了——核心原因是Cloudinary的upload方法是异步的，你的代码逻辑没处理好异步流程的时序问题：

• 当你在for循环里的upload回调中调用Campground.create时，每完成一张图片的上传，就会创建一条新的Campground记录，自然会生成多条条目。
• 如果你把Campground.create移到循环外面，此时所有upload的回调还没执行，req.body.ca.video1还没被填充，所以会变成undefined。

正确的解决方案：用Promise.all等待所有上传完成再创建条目 #

我们可以把每个Cloudinary上传操作包装成Promise，然后用Promise.all等待所有上传完成，收集完所有图片的URL和格式后，再一次性创建单条Campground记录。

修改后的POST路由代码如下：

router.post("/n/v/n", upload.array('images'), middleware.isloggedin, async function(req, res) {
  try {
    // 1. 把Cloudinary的upload方法包装成Promise，方便异步流程控制
    const uploadToCloudinary = (filePath) => {
      return new Promise((resolve, reject) => {
        cloudinary.v2.uploader.upload(filePath, { resource_type: "auto" }, (error, result) => {
          if (error) reject(error);
          else resolve({ url: result.secure_url, format: result.format });
        });
      });
    };

    // 2. 用Promise.all等待所有图片上传完成，批量获取上传结果
    const imageList = await Promise.all(
      req.files.map(file => uploadToCloudinary(file.path))
    );

    // 3. 构建完整的Campground数据对象
    const campgroundData = {
      ...req.body.ca,
      video1: imageList, // 将所有上传后的图片数据存入video1数组
      author: {
        id: req.user._id,
        username: req.user.username
      },
      created: new Date()
    };

    // 4. 仅执行一次创建操作，生成包含所有图片的单条记录
    await Campground.create(campgroundData);

    req.flash("success", "it is uploading");
    res.redirect("/c");
  } catch (err) {
    req.flash("error", err.message);
    res.redirect("back");
  }
});

关键改动说明： #

• 异步流程同步化：通过Promise.all确保所有图片上传完成后，才会执行Campground的创建操作，彻底解决回调时序问题。
• 单条记录创建：只调用一次Campground.create，保证最终只会生成一条包含所有图片的数据库条目。
• 更可靠的错误处理：所有异步操作的错误都会被外层try/catch捕获，避免了原代码中回调内错误无法被捕获的问题。

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