Node.js Express Router+Ajax POST报404，求问题排查与结构验证

我帮你分析下问题所在，以及优化你的路由结构：

一、404错误的核心原因 #

你的POST请求找不到路由，本质是请求路径和定义的路由不匹配：

• 在router.js里，你把login路由模块挂载到了/login路径下：

  app.use('/login', require('./routes/login'));
  

• 这意味着login.js里的所有路由都会自动带上/login前缀，所以你定义的POST /validateLogin对应的完整请求路径应该是 http://localhost:8888/login/validateLogin
• 但你的前端Ajax用的是相对路径url: 'validateLogin'，如果当前页面不是在/login路径下（比如直接访问根目录），这个相对路径会被解析成http://localhost:8888/validateLogin，自然找不到对应的路由，返回404。

另外还有个小问题：router.js里定义了router变量但完全没用到，导致里面的/logout路由根本无法访问，这个也得修正。

二、快速修复方案 #

1. 修正前端Ajax的请求路径 #

把请求URL改成绝对路径，确保无论当前页面在哪都能精准命中路由：

function login() {
  $.ajax({
    type: 'POST',
    url: '/login/validateLogin', // 改为绝对路径
    contentType: 'application/json',
    data: JSON.stringify({ username: "Foo", password: "Bar" })
  }).done(function (response) {
    if (response.isValid) {
      $(location).attr('href', 'next page');
    }
  }).fail(function () {
  });
}

2. 优化router.js的路由结构 #

把冗余的代码清理掉，同时统一管理所有路由，让结构更清晰：

module.exports = function(app){
  const mainRouter = require('express').Router();
  // 将各个子路由挂载到主路由上
  mainRouter.use('/login', require('./routes/login'));
  mainRouter.use('/route2', require('./routes/route2'));
  mainRouter.use('/route3', require('./routes/route3'));
  // 把logout路由也放到主路由里统一管理
  mainRouter.post('/logout', function (req, res) {
    var session = req.session;
    session.destroy();
    res.send({});
  });
  // 最后将主路由挂载到app的根路径
  app.use('/', mainRouter);
};

三、你的路由结构是否合理？ #

整体思路是没问题的——把不同功能的路由拆分成单独文件，通过入口文件统一挂载，这种模块化的方式非常便于后续维护和扩展。不过可以再优化几点：

• 避免冗余代码：像之前定义了router却不用的情况要及时清理
• 区分页面路由和接口路由：如果后续有更多接口，可以给接口统一加前缀（比如/api/login），和前端页面路由做区分
• 增加错误处理：可以在路由最后添加全局错误处理中间件，捕获404和其他异常请求

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