Question

我正在编写一个React应用程序（Express服务器），它使用共享（'isomorphic'）代码库在客户端+服务器上呈现。在某些情况下，当服务器是渲染器时，我想访问请求对象，而不必遍布整个地方。由于节点一次响应一个请求，利用缓存变量是不安全的，确保在发送响应之前重置它？

这有效，但有人可以告诉我为什么这是好主意还是坏主意？...

server.js

var r = require('./request-store');
app.use(function(req, res, next) {
  r.set(req);
  next();
});

var resolve = require('./resolve');
// ...deferring to react router for client / server routing
app.use(function(req, res) {
  resolve().then(function(data) {
    r.set(null);
    res.render('index', { data: data });
  });
});

请求store.js

var r = null;
module.exports.set = function(req) { r = req; };
module.exports.get = function() { return r; };

resolve.js

var r = require('./request-store');
module.exports = function() {
  return new Promise(resolve, reject) {
    if (isServer) {
      console.log(r.get());
      resolve();
    } else if (isClient) {
      resolve();
    }
  }
};

Answer 1

但是，如果您同时收到来自不同用户的两个请求，会发生什么？第一个请求将被缓存，第二个请求将覆盖第一个用户的请求。同时假设你正在做一些IO并且一旦完成就需要访问请求，那么第一个用户的请求对象就已经被第二个用户的请求覆盖了。总的来说，这样做是非常危险的。最好的方法是使用某种内存键值存储，但你仍然需要以某种方式传递密钥。

Answer 2

鉴于Node是单线程的，并且您不会出现来执行任何I / O，那么您的代码对我来说是安全的。 Express将同步运行所有中间件，因此在您的承诺运行时（因为它也是同步的）r.get()应该返回正确的请求对象。

但是，我不确定您是否需要缓存它，您可以将其直接传递到resolve并使用闭包将其捕获到您的承诺中

var resolve = require('./resolve');
// ...deferring to react router for client / server routing
app.use(function(req, res) {
    // pass 'req' into `resolve`
    resolve(req).then(function(data) {
        r.set(null);
        res.render('index', { data: data });
    });
});

resolve.js

var r = require('./request-store');
module.exports = function(req) {
  // capture 'req' for use later
  return new Promise(resolve, reject) {
    if (isServer) {
      console.log(req);
      resolve();
    } else if (isClient) {
      resolve();
    }
  }
};

避免传递请求对象

2 个答案: