Node.js Express项目中Cannot POST错误修复求助（MongoDB场景）

1. 直接报错原因：路由路径不匹配 #

你的请求路径是 /api/v1/users/:userid/subscribe/:businessid，但路由定义的路径是 /users/subscribe/:userid/:businessid，参数顺序与结构完全不符，导致Express无法匹配到对应的处理函数。

修复路由代码：
调整路由路径结构，与请求路径匹配：

import { Router } from 'express';
import { index, deactivate, activate, subscribe } from '../../controllers/users';

const userRouter = Router();

// 修正路径，匹配请求的/:userid/subscribe/:businessid结构
userRouter.post('/users/:userid/subscribe/:businessid', subscribe);

export default userRouter;

同时确认主应用（如app.ts）中挂载路由的基础路径为/api/v1：

import userRouter from './routes/userRouter';
app.use('/api/v1', userRouter);

2. 控制器逻辑错误 #

当前控制器存在两处核心逻辑错误：

• 权限判断倒置：需验证被订阅的商家（businessid对应用户）是商家账号，而非订阅者（userid对应用户）
• 数据操作存在并发风险：先查询再修改再保存的方式，在高并发场景下可能导致数据不一致

修复后的控制器代码：

import { Request, Response } from 'express';
import UserModel from '../models/User';

export const subscribe = async (req: Request, res: Response) => {
  const { userid, businessid } = req.params;

  try {
    // 验证商家存在且为商家账号
    const business = await UserModel.findById(businessid);
    if (!business) {
      return res.status(404).json({ error: '商家不存在' });
    }
    if (!business.is_business) {
      return res.status(403).json({ message: '该用户不是商家账号' });
    }

    // 原子操作更新订阅者的following数组
    const updatedUser = await UserModel.findByIdAndUpdate(
      userid,
      { $addToSet: { following: businessid } },
      { new: true, runValidators: true }
    ).exec();

    if (!updatedUser) {
      return res.status(404).json({ error: '用户不存在' });
    }

    // 原子操作更新商家的follower数组
    await UserModel.findByIdAndUpdate(
      businessid,
      { $addToSet: { follower: userid } },
      { runValidators: true }
    ).exec();

    return res.status(201).json({ user: updatedUser });
  } catch (err) {
    return res.status(500).json({ error: '服务器内部错误', details: err });
  }
};

3. 额外优化建议 #

• 使用MongoDB的$addToSet原子操作替代先查后改的方式，避免并发数据不一致问题
• 添加try/catch捕获数据库操作异常，返回更友好的错误信息
• 可引入Joi/Zod等工具，验证userid和businessid是否为合法的MongoDB ObjectId格式

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