有没有大佬可以帮忙优化的,刚毕业的小白,项目组长说我的代码有点冗余

Tory1 发布于 07/11 11:06
阅读 1K+
收藏 0

加载中
1
Kingviker
Kingviker

大概看了一下 由于不懂业务逻辑,先说两个可以优化的地方,

1.info日志输出,建议把日志的输出封装到对应的model内部,在balanceModel和updateAmountModel中封装两个logMsg()的类似toString方法类输出需要的对象日志,方便复用

2.中间大段的if self判断 可以在if开始前定义四个变量保存临时值,在判断结束后统一赋值给balanceModel(这条只是逻辑清晰一些,代码量没减少多少)

3.大量的bigDecimal转换函数重复的出现,可以考虑在对应的model增加方法来节省代码量.

0
蓝水晶飞机
蓝水晶飞机

把现在的代码逻辑抽象一点,抽象简化重复的逻辑。

0
初入虎穴

emmmmmm,确实,重新写吧

0
玫瑰少年在旅行
玫瑰少年在旅行

想我一个不用java三年多的人竟然还能大体看懂

你不懂的吧
你不懂的吧
让人看得懂的都是好代码:no_mouth:
0
草庐过客
草庐过客

看下去两页纸都是一样的代码

0
cakeralter
cakeralter

重复的逻辑抽出来

0
落舞者
落舞者

适当的控制一下长度

0
ffqcode
ffqcode

代码发过来 我给你改

0
Joyzhou
Joyzhou

看了下,逻辑还算清晰。我觉得有两个优化点:

1. transactionType最好用swtich来判断

2. add的逻辑可以放在model中实现

返回顶部
顶部