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

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

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

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

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

@lrs 好吧, 我看错了

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

看的心塞。😂

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

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

按行数算 KPI 的时候有优势

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

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

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

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

想看楼主优化后的代码

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

怎么改进啊

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

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

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