同事的一段代码,感觉很烂程序猿.爱妃给朕拿点纸:m…

同事的一段代码,感觉很烂

程序猿.爱妃给朕拿点纸:m

颜良:m

菜鸟网络员工:不知道业务背景,但主观感觉逻辑有些杂糅。修改任务的前置判断(企业id)如果抽出去在方法外面做也许会好些?

程序猿.胡青牛:M

蚂蚁金服员工:m

石秀:我觉得很容易看懂啊

未来员工:还可以,不差

京东员工:我觉得这段代码还好吧~

程序猿.计无施:返回值应该是bool或者0 非0,其他都还好。

猎豹移动员工:我觉得还OK了,我是没看懂,很深奥的样子[微笑][微笑]

未来员工:楼主应该是对于返回值有意见,看不懂啥意思或者代码表达的意思有点模糊

程序猿.锤神:没懂,企业id不是和获取到的任务的企业id做比较吗?

程序猿.锤神:楼主麻烦讲解一下烂在哪里

上海游族信息技术有限公司员工:可读性可以

百度员工:我觉得没啥大毛病

程序猿.唐僧行:有啥烂的?

徐庶:这代码哪里烂了?最多没有把异常处理前置掉,也不影响啥。

武当弟子:m

程序猿.泰山弟子:这写的很清楚啊,返回值一个小瑕疵而已

程序猿.锤神:散了吧,钓鱼贴

前北京丰娱互动员工:业务代码都很烂

程序猿.二狗你变了:管你卵事

程序猿.贾雨村:有改进空间,烂不至于

同花顺员工:催业务的时候能写成这样我觉得很好了[吐血]

程序猿.收割机offer:不怕npe么

饿了么员工:M

程序猿.殷天正:楼主代码洁癖,鉴定完毕

程序猿.忽如一夜春风来:这不就是我写的么,吐槽啥呢

程序猿.阮小五:有问题?

KOP008:m

程序猿.华山弟子:m

程序猿.衡山弟子:楼主认为烂在哪里

程序猿.锤神:哪里

美团员工:还行吧,没啥大毛病

前百度员工:业务代码都这样

程序猿.衡山弟子[2]:47行如果字段值为null可能出现npe。 最后一个if条件反转一下提前抛异常,这样看起来更舒服些。

程序猿.衡山弟子[2]:枚举判断的那个if

昆仑弟子:我觉得把函数第一行的注释去掉,楼主就觉得可以了

程序猿.狐媚胡梅尔斯:不判断也没问题,这个数据应该是查出来的,要么存在,要么不存在。即使出现空属性事务外会catch住效果一样。把异常都放在上面看起来舒服点。

匿名区的韦小宝:钓鱼

前华为技术有限公司员工:返回值有点问题,别的还好

北京小米科技有限责任公司员工:知足吧

金山办公员工:你是没见过我的同事写的[坏笑]

董卓:不要用异常控制来处理正常业务逻辑。还是事务型的。

程序猿.顾九思:m

xx出行员工:没大毛病,小细节可以优化下

蚂蚁金服员工[2]:主要问题是没有用enum吧,直接把没有业务语义的int扔出去,如果没注释的话比较难懂

程序猿.郭霖:核心逻辑都有ConfigTask参与,为啥不拆分出去

少侠胡梅尔斯:tomcat源代码还if(true == 布尔表达式呢)。写代码没什么问题

京东员工:我觉得这段代码还好吧~

猎豹移动员工:我觉得还OK了,我是没看懂,很深奥的样子[微笑][微笑]

菜鸟网络员工:不知道业务背景,但主观感觉逻辑有些杂糅。修改任务的前置判断(企业id)如果抽出去在方法外面做也许会好些?