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

wocao, 小丑就是我自己，我也这样写

这写的挺好的, 最多就是提前返回可以优化一下.

图一有待改进，但图二真心觉得还行。。。

@mxT52CRuqR6o5 分享的这俩都挺经典, 哈哈

@JeffersonQin
flag = true
if member1 == null {
flag = false
}

if positionsInfo == null {
flag = fase
}

类似这样做个取反逻辑会更加清晰呀

@weiwenhao 我感觉图里的逻辑和你给的例子不太一样，而且嵌套 if 还有好处，可以防止深层次的 null pointer exception

比方说这两天我写 dotnet 用 opencv 的 wrapper ，判空会这么写：

if (src == null)
if (src.IsDisposed)
if (src.CvPtr == IntPtr.Zero)
src.Dispose();

诚然你也可以用 goto 的写法，不过嵌套 if 在某些情况下还是更直观的

@JeffersonQin 打错了 if 条件都反了 直接 ctrl c/v 忘记改了

@DreamingCTW Optional

楼主可不可以把重构之后的代码贴出来，学习学习

别的不说，dao 接口返回 JSONObject 是认真的吗？

看方法命名就感觉像是一个 void 的方法
member1 应该是 memberId
还有就是不至于写俩方法把
if 嵌套还好
不过我感觉既然 return 了，没必要 else 了

第二个没啥问题啊。。

尝试优化了下劝退了


第一个代码应该只需要 2 行：

if ((member2 == null && member1 != null) || (member2 != null && !member2.equals(member1))) {
return ...
}
return false

第二个代码感觉没啥问题，毕竟每一步值都是上面获取的，如果后面没有其他逻辑的话其实可以把 flag=true 改成 return true

换 kotlin

这是典型 optional 的应用场景吧

很好，简单容易理解，没有弯弯绕绕

改进方法就是判断的时候取反，多提前 return ，不要嵌套

return Objects.equals(mem1, mem2) ? false: changePartyPosition(...);

&& || 这些就硬不用呗