File: ShortCircuitBoolean.md

package info (click to toggle)
error-prone-java 2.18.0-1
  • links: PTS, VCS
  • area: main
  • in suites: bookworm, forky, sid, trixie
  • size: 23,204 kB
  • sloc: java: 222,992; xml: 1,319; sh: 25; makefile: 7
file content (24 lines) | stat: -rw-r--r-- 550 bytes parent folder | download | duplicates (2)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
The boolean operators `&&` and `||` should almost always be used instead of `&`
and `|`.

If the right hand side is an expression that has side effects or is expensive to
compute, `&&` and `||` will short-circuit but `&` and `|` will not, which may be
surprising or cause slowness.

If evaluating both operands is necessary for side effects, consider refactoring
to make that explicit. For example, prefer this:

```java
boolean rhs = hasSideEffects();
if (lhs && rhs) {
  // ...
}
```

to this:

```java
if (lhs & hasSideEffects()) {
  // ...
}
```