提交Todo请求时遇ValidationError：user字段无法转换为ObjectId

报错信息 #

ValidationError: TodoTask validation failed: user: Cast to ObjectId failed for value
  id: 62e02a03c5f6c427c8c3ed44,
  user: {
    _id: 62e02a03c5f6c427c8c3ed44,
    name: 'abc',
    email: 'someemail@example.com',
    password: '$2a$10$C....',
    admin: true,
    date: 2022-07-26T17:53:07.645Z,
    __v: 0
  }
}"(type Object) at path "user"

错误原因 #

• 你的TodoTask Schema中，user字段被定义为mongoose.Schema.Types.ObjectId类型，要求存储关联用户文档的ObjectId值（一串字符串格式的唯一ID）。
• 但创建TodoTask实例时，你给user字段传入了嵌套对象{id: req.user._id, user: req.user}，完全不符合Schema的类型约束，导致Mongoose无法将该对象转换为ObjectId类型，触发类型验证失败错误。

解决方法 #

直接给user字段传入用户的ObjectId值即可，无需包装成对象。同时修正代码中await与回调混用、错误日志顺序错误的问题：

修改后的请求提交函数代码：

const todoTask = new TodoTask({
  content: req.body.content,
  user: req.user._id, // 直接传入用户的ObjectId
});
try {
  await todoTask.save(); // 利用await处理异步，无需嵌套回调
  console.log("item saved!");
  return res.redirect("/dashboard");
} catch (err) {
  console.log(err); // 先打印错误再跳转，确保错误信息能被捕获
  return res.redirect("/dashboard");
}

补充说明 #

原代码存在两处额外问题：

1. await todoTask.save(function (err, doc) { ... })：save()方法本身返回Promise，使用await即可处理异步结果，无需同时传入回调函数，两者混用会导致逻辑混乱。
2. catch块中先执行return res.redirect再console.log(err)：return语句会终止函数执行，导致后续日志打印永远不会触发，需调整顺序。

内容的提问来源于stack exchange，提问作者Syed Shuja Shah