Node.js+Express中Object is possibly null报错解决方案咨询

问题根源 #

你的代码存在两个核心问题触发TypeScript报错：

• 调用next(createError(...))后未终止函数执行，后续代码仍会尝试访问可能为null的对象（比如order.userId）
• TypeScript无法自动推断next调用后代码会停止执行，因此会提示对象可能为null的类型风险

修复方案 #

在每次调用next传递错误后，立即添加return语句终止函数执行。这样TypeScript就能确定后续代码中变量一定不为null，同时避免错误情况下继续执行业务逻辑。

修改后的API代码：

export const getOrderRaport = async (
  req: Request<{}, {}, IAuthUser>,
  res: Response,
  next: NextFunction,
) => {
  const orderId = req.query.orderId;
  const order = await Order.findOne({ orderId: orderId });
  
  if (!order) {
    next(createError("Order does not exist"));
    return; // 终止函数，避免后续代码执行
  }

  const orderUser = await User.findOne({ userId: order.userId });
  if (!orderUser) {
    next(createError("Order was not found for the specified user"));
    return;
  }

  const requestUser = await User.findOne({ userId: req.body.userId });
  if (!requestUser) {
    next(createError("You are not allowed to view this order"));
    return;
  }

  // 此时TypeScript会确认orderUser和requestUser均不为null
  if (orderUser._id.toString() !== requestUser._id.toString()) {
    // 添加权限不匹配的错误处理
    next(createError("You don't have permission to view this order"));
    return;
  }

  // 后续正常业务逻辑，比如返回订单报表
  res.status(200).json({ 
    message: "Order raport retrieved", 
    data: /* 填充报表数据 */ 
  });
};

额外优化建议 #

• 对比MongoDB的_id时，建议用toString()转换为字符串后再比较，避免Object类型直接对比的潜在问题
• 可将重复的next+return逻辑封装成工具函数，减少代码冗余：

const handleError = (next: NextFunction, message: string) => {
  next(createError(message));
  return;
};

使用时简化为：if (!order) return handleError(next, "Order does not exist");

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