java - 逻辑运算中的条件赋值

Question

这种简写是一种不好的做法吗？或者任何具有 Java 基础知识的程序员都应该知道在下面的代码中，如果 host 等于 ，则 checkDoSResult 不会被重新分配>“本地主机”?

int checkDoSResult = 0;
if (!"localhost".equals(host) && (checkDoSResult = CheckHost.checkDoS(user+"@"+host, "query"))!=0)
...

Answer 1

“任何具有 Java 基础知识的程序员”知道某事的事实并不一定使该“某事”成为良好实践。

众所周知，条件和循环头中的赋值很难发现，因为程序员的眼睛期待的是那里的条件，而不是赋值。将赋值放在逻辑表达式的短路部分只会让事情变得更糟。

当然，任何接受过良好培训的程序员都能够准确地“破译”你想说的话，但这需要他们时间。将条件拆分为两部分，并将赋值放在 if 语句的主体中，这将有助于其他人理解您的意图，这比节省几行代码更重要。