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

[Wsdba]

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

[liuzhaowei55]

有啥问题吗？判断逻辑清晰，没有复杂逻辑判断。
说实话代码降低心智负担才是真的重要

[zjsxwc]

@DreamingCTW
但是我反倒觉得合并条件到一个 if 中去后，反而更加烦躁，逃。。

[EggplantLover]

我觉得还好吧，能怎么精简呢，哪位大佬给个示范

[DreamingCTW]

@zjsxwc 我一眼看去挺清晰的....||左边一个条件，右边一个条件，也不算太复杂

[vanton]

if 套 if 的问题么？

这里套的也还好，不算太长。

不过最好不要这样多个 if 深层嵌套。

[DreamingCTW]

我也想看看有没有大佬来优化，因为我也经常写 if 非空判断，毕竟 Java 这个空指针异常挺恼火的

[rming]

if A and B:
return
if C:
return
return

[ww940521]

我觉得这样写挺好的逻辑一目了然，把判断条件合并起来反而看起来费劲，要去想有没有把所有的可能包含进去。一层一层判断不容易出现疏漏。我也想看看大佬优化。

[gps949]

还好吧？没看出有啥值得批判的问题。现代编译器 /解释器会对多级判断有自己的优化吧，不过是一个写 web 应用 crud 的，又不是开发编译器或操作系统，只要人读得感觉清晰，有必要非要炫技“人工优化”到一行跟 js 代码混淆一样吗？

[zhouyg]

if 套 if 其实没啥问题，跟业务逻辑对应就行

[ToDyZHu]

虽然我自己不太会写这种代码 但是我很喜欢改这种代码

[ianEros]

代码水平高应该是让应届生也能很快理解，而不是为了好看导致可读性差

[arthas2234]

if 判断逻辑简单越好，判断逻辑过长可以先把结果赋值给临时变量，变量名语义要清晰
包括里面的逻辑，也是越短越好，实在不能精简，就拆分成小的函数，方便阅读
善用 if-return：if(member != null) {...} => if(member == null) return;
变量名：flag ，a1 ，a2 之类的语义不清晰的就不要用了

[pengtdyd]

写代码的不厉害，会改别人的代码才是大佬。

[selfcreditgiving]

@starsky007 一直这么写的，这还有一个说法的啊

[starsky007]

@selfcreditgiving
一起这么写就好；《重构：改善既有代码的设计》这本书有提到这个概念，交流起来也方便些。

[sue0917]

最后他代码行数多，拿了个比你高的绩效

[SheHuannn]

这变量命名真是绝了，太直接了，像是机器人干的

[chengkai1853]

@SheHuannn 变量名并没什么问题吧？！一看函数就知道是更改成员位置信息的代码。

[Tink]

两层 if 还好吧