easy-team / egg-view-react-ssr

Egg React Server Side Render (SSR) Plugin
https://easyjs.cn
MIT License
62 stars 9 forks source link

为什么取消了no state component render的功能? #5

Closed mib008 closed 4 years ago

mib008 commented 5 years ago

在git的分支中有看到,2.3.2版本是支持stateless component的渲染的,为什么后来的版本里没有支持,而且也没有相应的警告信息?

hubcarl commented 5 years ago

@mib008 这个看了提交记录,当时为了支持 promise callback 的方式,没办法判断是否是无状态组件还是callback ,因为都是 function

hubcarl commented 5 years ago

@mib008 发了 2.5.3 ,可以通过 egg-view-react-ssr@next 验证

mib008 commented 5 years ago

@mib008 这个看了提交记录,当时为了支持 promise callback 的方式,没办法判断是否是无状态组件还是callback ,因为都是 function

我认为并不需要判断 是否是无状态组件还是callback,直接丢给React就好了,出错也是反馈React内部的错误信息。

hubcarl commented 5 years ago

@mib008 你说的这个原本是没问题的。 render 时,扩展了一种 单页面 asyncData 函数调用入口方式,当初设计的时候有点缺陷,没有考虑到无状态 function与自定义区分问题,为了 支持已经在跑的项目,就没有修改

return context => {
      const { Entry, createRouter, createStore } = config;
      const url = context.state.url;
      const router = createRouter();
      const matchRoute = matchRoutes(router, url);
      const promises = matchRoute.map(({ route }) => {
        const asyncData = route.component.asyncData;
        return asyncData ? asyncData(context, route) : Promise.resolve(null)
      });
      return Promise.all(promises).then(list => {
        const data = list.reduce((item, result) => {
          return Object.assign(result, item);
        }, {});
        Object.assign(context.state, data);
        const store = createStore(context.state);
        return () => {
          return <Provider store={store}>
            <StaticRouter location={url} context={{}}>
              <Entry url={url} />
            </StaticRouter>
          </Provider>
        }
      });
    };
hubcarl commented 4 years ago

@mib008 已经发布egg-view-react-ssr@3.0.0 标准化支持了