Express/Node.js应用实现随机背景遇到问题求助

嘿，我看了你这段用来实现随机背景的Node.js/Express代码，先帮你梳理下几个容易踩的坑，同时给出优化后的实现方案，这样你不管遇到什么具体问题，都能先把基础逻辑理顺：

首先，当前代码的核心问题 #

• 随机背景值不会动态更新：你现在的bg变量是在脚本加载时就生成了固定值，服务器启动后所有用户的请求都会拿到同一个bg，完全达不到“随机”的效果！得把随机数生成逻辑放到每次请求的处理流程里。
• 路由逻辑覆盖不全：你的router.get('/*', ...)只处理了根路径（slug为空）的情况，其他路径的请求会直接走到next()，导致这些页面拿不到bg值，甚至可能出现404。
• 缺少错误处理：数据库查询没有错误捕获，一旦查询失败会直接抛出异常，导致服务器崩溃。

优化后的代码示例 #

方案1：用中间件给所有页面注入随机背景（推荐） #

这样所有路由渲染的模板都能直接使用随机背景值，不用重复写逻辑：

function getRandomInt(max) { 
  return Math.floor(Math.random() * Math.floor(max)); 
}

// 全局中间件：每次请求都生成新的随机背景值
router.use((req, res, next) => {
  // 如果你的背景是固定的几个，也可以存成数组直接取，更直观
  // const bgList = ['bg-1.jpg', 'bg-2.jpg', 'bg-3.jpg'];
  // res.locals.bg = bgList[getRandomInt(bgList.length)];
  res.locals.bg = getRandomInt(3);
  next();
});

// 根路径路由
router.get('/', function(req, res, next) {
  connection.query('SELECT * FROM `pages`', function (error, results, fields) {
    if (error) {
      // 处理数据库查询错误，交给Express的错误处理中间件
      return next(error);
    }
    // 模板里直接用{{bg}}就行，不用再传这个参数
    res.render('index', { title: title, year: year, pages: results });
  });
});

// 其他页面路由示例
router.get('/contact', (req, res) => {
  res.render('contact', { title: 'Contact', year: year });
});

方案2：单独在需要的路由里生成随机值 #

如果只有部分页面需要随机背景，就把随机数生成逻辑放到对应路由里：

function getRandomInt(max) { 
  return Math.floor(Math.random() * Math.floor(max)); 
}

router.get('/', function(req, res, next) {
  // 每次请求都生成新的随机值
  const bg = getRandomInt(3);
  connection.query('SELECT * FROM `pages`', function (error, results, fields) {
    if (error) return next(error);
    res.render('index', { title: title, year: year, bg: bg, pages: results });
  });
});

额外建议 #

• 尽量避免用router.get('/*', ...)这种全局匹配路由，它会覆盖所有更具体的路由（比如/about），导致这些路由无法生效。
• 如果背景是图片资源，建议把图片路径存在数组里，直接随机取路径，模板里可以直接用<body style="background-image: url({{bg}})">，不用在模板里做数值到路径的映射。
• 一定要处理数据库查询的错误，否则一旦数据库连接出问题，服务器会直接崩溃。

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