Ant-design-pro: 这玩意好难用啊！

[guodong]

一个简单的登录复杂到要改5个文件，10分钟工作量硬生生变成48小时，登录项用map.js提出来，是打算复用还是咋地？
加个route还要改2个页面，route既然都封装了，为什么遗留如下代码的第三个参数不能兼容字符串或者function？

'/': {
      component: dynamicWrapper(app, ['user', 'login'], () => import('../layouts/BasicLayout')),
    },

还有这种丑陋的代码

Login.Tab = LoginTab;
Login.Submit = LoginSubmit;
Object.keys(LoginItem).forEach(item => {
  Login[item] = LoginItem[item];
});

[ant-design-bot]

Translation of this issue:

---

This stuff is hard to use!

A simple login is complicated to change 5 files. The 10-minute workload is hard to become 48 hours. The login item is proposed by map.js. Is it intended to be reused or smashed?
Add a route and change 2 pages, since the route is encapsulated, why is the third parameter of the following code not compatible with strings or functions?

'/': {
      Component: dynamicWrapper(app, ['user', 'login'], () => import('../layouts/BasicLayout')),
    },

There is such an ugly code

Login.Tab = LoginTab;
Login.Submit = LoginSubmit;
Object.keys(LoginItem).forEach(item => {
  Login[item] = LoginItem[item];
});

[chenshuai2144]

有兴趣可以提 pr，你可以改变掉这些现状

[AnInputForce]

如果你不知道为啥要用redux，那就是你不需要redux

同理可证：可能你目前的项目不适合引入antdesign pro

[guodong]

@AnInputForce 啥项目适合antdesign pro呢？或者说antdesign pro适合啥项目呢？

[yoyo837]

如果他能自己得出结论就不会发起这种issue了

[yoyo837]

项目要100%契合自己的实际需要最好是自己从0开始，用一个东西就受一个东西的条条框框束缚。你用pro对不好的地方可以自己改好落地到自己的项目中，也可以提给PR贡献给社区

[guodong]

好的

[chenshuai2144]

pro 更多的是一个样板代码。使用了业内的很多最佳实践。
不可否认的确实有很多脏代码，但是由于维护人员的精力和时间有限，可能任由其存在。
希望可以 社区可 pr 解决掉。让 pro 越来越好

[afc163]

我们也不是神，代码是靠社区一起维护才会越来越好。

[533001]

如果不满意,就证明给 1w+的小伙伴儿 看,什么是好代码