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

这代码写得好，让人有工作量

可读性还行 ，反正编译器会合并条件

没有注释，没有 log
除了这两个我不觉得有什么不妥。
可改进，但是个人感觉意义不大
接手的人不用费脑子一看就能明白

第一点 ： 两个函数高度相似能不能合并成一个
第二点 ： 可以先行判断的前提条件可以提到前面部分
第三点 ： 可以直接返回的判断是不是可以不用嵌套

我感觉简单改可以按这个改 。

又不是看不懂
要求项目代码的每一行都经得起推敲才是强人所难，屎山是在所难免的，这种算不上好代码，但好歹一眼就能看懂，没必要特地拎出来说

@starsky007 #19 是的, 我也喜欢这样写, 先逆向判断, 先处理逻辑少的条件

也不用过度考虑简化，像楼上说的，简化也要看值不值嘛。
非要说优化的话 图 1 已经很多朋友说了，图二的话我个人认为改用 try catch 也可以。
private boolean changePartyPositions(String member1, String member2, String position...) {
boolean flag = false;
try {
JSONObject member1Json = partyMemberDao.getInfoById(member1);
JSONObject memberPositionsInfo = memberPositionsDao.getInfo(member1, p...);
flag = memberPositionsDao.delete(member1, positionName, org, updateT...)
} catch (NullPointerException e) {
// do nothing.
}
return flag;
}

如果没有事情干，可以换种写法

//个人喜欢的逻辑 觉得更符合阅读习惯
if (member2 == null && member1 != null) {
return changepartypositlons();
}
if (member2 != null && !member2.equals(member1)) {
return changepartypositlons();
}
return false;

这种代码风格是把自己需要用到的场景写出来了，剩下的全是一个场景，有些考虑不到的场景就很容易走到 else 里面，造成 bug ，我更喜欢把所有场景列出来的方式来写

其实楼主第二章图没贴全，导致大家优化不彻底

https://s1.ax1x.com/2021/12/10/o5uC0P.png
第二次优化

@liuzhaowei55 你没理解我的意思，绝大部分情况下这种条件判断粗略看过就可以了，本身就已经非常清楚了，类似这样的代码过多，说明你要多阅读数倍的代码量，才能理解整体代码的意思，实际上大大降低了可读性。

我觉得挺好的啊，条理清晰的，方便阅读

append 一个代码版本的？ 图片似乎丢失了(还是我这里 load 不出来...

结构清晰，按最细粒度划分分支，对未来的变更留了空间，之后迭代只要加 else 代码不需要改 if 条件代码

@xuanbg 可以直接用 Objects.equals, 这个函数可以代替 StringUtils.equals

下面居然有评论觉得这样写挺好的 /。。。。+

咱先不说条件判断

这个 member 1 、member2 大家都能忍受吗？