同事的一段代码,感觉很烂
程序猿.爱妃给朕拿点纸: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)如果抽出去在方法外面做也许会好些?