4 年 React 经验 写出这样的业务代码，请问代码质量如何，有什么值得改进的地方吗

[rookie2luochao]

github 仓库地址为：openapi-ui

求各位大佬 review 代码，并指导一二

[rookie2luochao]

@chanChristin 你说的建议很中肯，比如多个 routes 、components 我都改了，但是对组件文件夹名字叫 AppTitle 我提出了疑问，你看下[antd-design 命名？]( https://github.com/ant-design/ant-design/tree/master/components/auto-complete)
我们也属于交流是吧？

[rookie2luochao]

@yagamil 凑合用一下😭😭，略懂一点，入门菜鸡

[rookie2luochao]

@h1104350235 我也觉得大家都是人，说话不能太难听吧

[rookie2luochao]

@hahawode 你发的图看不到？

[ColdBird]

像是放空大脑随便写写的，感觉没什么好 review 的。。。。。。。

[crocoBaby]

我完了,完全看不懂了

[chanChristin]

@rookie2luochao #61
https://github.com/ant-design/ant-design-pro/tree/master/src/components
antd pro 用的全都是大驼峰和文件夹的形式写的组件，而且他们的页面也是大驼峰的组件。
你想用哪种无所谓，关键是要统一格式，格式混乱看着难受，用着也难受。

[mumuwen]

1 、行内样式太多了，看起来 js 代码一坨（视图+逻辑+样式） 不够整洁。cssModule+less 。
2 、vite 配置下 alias ，引入路径太长了，很多 '../.../../'这种，配置 src 目录下的 @/
3 、 @chanChristin 说的没错，纯组件放在 components 里面，业务组件和通用组件需要分开，路由写的确实不太好维护，每个文件都有个 route ，写在一个路由文件里，统一维护查找比较契合 react 的写法，不能因为之前 vue 这样写 react 也这么写。
4 、项目文件有点...，可以优化一下，src 目录下可以有一个 component/util/type.ts/route/config ，作为根目录下的通用模块，可以将一些业务文件里的抽离出来，比如函数、类型声明...，每个文件下都有这些 js 很杂乱。
----- 这些都是偏基础和规范问题，可以优化，具体的项目我也没跑，只是简单看了看。
最直观的感觉就是...不是专攻前端？或者没进过中大厂？感觉没有在中大厂的项目中历练过，项目规范性、架构差很多...
这里面很多点如果在实际工作中，会被领导喷的

[ruoxie]

哪些是页面，哪些是组件，看起来有点乱

[rookie2luochao]

@mumuwen 好的，谢谢建议，接下来抽时间改造一下，整的规范见解一点

[rookie2luochao]

@bojackhorseman 我确实不太保持风格，或者就是别人口中说的，以自己为规范🥹😭😭

[K120]

https://github.com/xjh22222228/nav 和我一起学多一门 Angular ，吃多一口饭，嘿嘿

[rookie2luochao]

@ruoxie 后面加个 pages 文件夹把页面搞进去

[chill777]

感觉一般吧。但是比我们公司的那个十年前端钉子户好太多了。

[rookie2luochao]

@K120 可是国内的 angular 实在太少，出海又没有渠道😭

[tuomasi]

一小时能送几个外卖，一小时能抢几个网约单