Express循环创建多路由异常：动态PDF手册下载链接失效求助

这是典型的闭包作用域陷阱：你用for...in循环时，循环变量element是共享的，所有路由的回调函数都引用同一个element变量。当这些回调被触发时，循环已经执行完毕，element指向的是最后一次迭代的键，所以所有路由都会错误地使用最后一个（或因循环执行时机异常指向第一个）元素的文件路径。

推荐解决方法（使用forEach遍历） #

如果manuals.list是数组，直接用forEach遍历，它的回调函数会为每个迭代项创建独立作用域，确保回调能正确捕获当前项：

const express = require('express');
const app = express();

const manuals = require('./manuals/manuals_links.json');

manuals.list.forEach(item => {
  console.log(item);
  app.get(item.permaLink, (req, res) => {
    res.download(item.file);
  });
});

其他兼容方案（针对manuals.list是对象的情况） #

1. 用let声明块级作用域变量：

const express = require('express');
const app = express();

const manuals = require('./manuals/manuals_links.json');

for (const element in manuals.list) {
  let item = manuals.list[element]; // let会在每个循环块创建独立的变量实例
  console.log(item);
  app.get(item.permaLink, function(req, res) {
    res.download(item.file);
  });
}

2. 立即执行函数（IIFE）捕获当前项：

const express = require('express');
const app = express();

const manuals = require('./manuals/manuals_links.json');

for (const element in manuals.list) {
  const item = manuals.list[element];
  console.log(item);
  // 通过IIFE将当前item传入，创建独立作用域
  (function(currentItem) {
    app.get(currentItem.permaLink, function(req, res) {
      res.download(currentItem.file);
    });
  })(item);
}

这些方案都能确保每个路由的回调函数引用的是迭代时对应的item，而非循环结束后的共享变量。

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