代码待优化、改善

Choerodon Choerodon Framework
#1

早上坐地铁，在极客时间上突然看到这个开源项目，看了介绍之后感觉跟自己想法挺一致的，自己最近也刚好在写这一块的开源，就赶紧关注了下，本想观摩下大神的代码，但结果有点微微失望，感觉代码上有些地方还是可以优化优化的。

因为只看了oauth-server的代码，说说自己的看法：
1.可以引入lombok简化代码(Gett、Sett以及日志和构造器对象创建)
2.ConvertorI内可以引入orika对对象进行灵活转换(可以通过自定义注解的方式)
3.业务逻辑中if层次太多，try()catch()太多(可以引入全局异常处理消除try()catch()，个人觉得业务逻辑中不到万不得已一般不会出现try()catch()代码,可以全部往上一层抛，由最外层进行处理)
4.安全处理这块其实可以封到统一的模块里
5.分层和命名，无法第一时间联想出一个清晰的基础架构

最后希望Choerodon能越来越好，自己也会一直关注你们，也希望自己能和你们一起慢慢成长，加油。。。:slight_smile:

1 个赞
#2

开源项目不适合使用 lombok吧，因为这限制了所有想参与开源开发的人员，都必须装lombok的插件。
虽然lombok插件非常好用。。（本人是他的忠实fans。。。）

#3

感谢您的关注与建议,多吸收外界的发声,知道不足我们才能成长和进步.

#4

嗯，也是呢，没怎么写过开源项目，还是欠缺考虑哈。

#5

非常感谢您的建议，代码质量这块我们还有比较多的工作要做，你提到的几个工具库，我们也会去研究，在后续的开发中看以什么方式来使用。

也期待您提出更多的建议，让我们把Choerodon做的更好！

#6

代码有时间了肯定会重构优化；关于lombok要求每个人装个插件不是很方便类，自己写的项目倒是都用；ConvertorI主要解决依赖，bean拷贝我们主要用BeanUtil。如果实现一个功能需要自己写两个类实现或者引入一个jar包加个方法实现，现在可能会优先考虑自己实现。多谢支持:hugs:

#7

有思想，很高兴看到一帮都在干实事的人在一起，我们也尝试做了些模块，有涉及到 fjtnylk提到的一些点，计划后续能集成到一起，共建美好未来