大家帮我看看，这代码是水平。。

[Wsdba]

刚接手的一个项目，发现这个人很喜欢这样写。

[xption]

经常遇到公司新来的同事这样写代码
习惯不好+逻辑不清晰，之前没人教过他
坐他边上手把手带他写几次就好了

[coderluan]

宝宝树，宝宝宝宝树，宝宝宝宝宝宝树

[yangzzzzzz]

给他装个 idea 按照提示优化一下代码就好了

[lrs]

这命名, 只比没有名字强一点

[lrs]

@lrs 好吧, 我看错了

[iovekkk]

由此看来，kotlin 的可空类型处理真的是太方便了

[YzSama]

看的心塞。😂

[Leviathann]

@iovekkk 然后他会写一个接受参数全是可空的方法
然后用的地方全都 double bang 而且不写注释。。

[rationa1cuzz]

像极了我之前同事写的，一个 view 6000 行

[sagaxu]

按行数算 KPI 的时候有优势

[zjsxwc]

mark, 除了命名不好外，看看大佬们会有什么写法

[Kasumi20]

服了，不会 AND OR 吗，一行代码搞定的事情，硬是拆成七八行

[kop1989]

要不提出改进方法让大家品鉴下？

[socketpeng]

@zjsxwc 我也想知道如何改进这种代码

[MrEatChicken]

想看楼主优化后的代码

[flyingyasin]

或许哪位老大哥也会这样发个帖子来嘲讽楼主写的代码

[freak118]

怎么改进啊

[DreamingCTW]

第一张图的方法...我看 if 代码块里面的返回值都是一样的.....那方法体为何不这样写.....
if ((member2 == null && member1 != null) || !member2.equals(member1)) {
return changePartPositions(member1, member2, name, org, updateTime);
}
return false;

[starsky007]

怎么改进？搜索“卫语句”。

[cstj0505]

一眼也就是缩进问题，我觉得问题不大，是正常的思维。if 判断提前返回能减少缩进，没这样做也不至于被拉出来嘲讽的地步