Which do you prefer of these two? The goal is the same. If bar
is null then make foo
null and avoid the exception. In other languages there is the ?.
operator to help with this.
foo = bar == null ? null : bar.baz();
foo = bar != null ? bar.baz() : null;
I ask because I feel like the “English” of the first example is easier to read and has less negations so it is more straightforward, but the second one has the meat of the expression (bar.baz()
) more prominently.
The second, or early return/continue/break.
But don’t forget the third option:
This is much more readable if nontrivial; the only downside is that this inhibits the practice of ubiquitous
final
.Actually, doesn’t Java allow lazy
final
if you don’t initialize (that would require explicitelse
)? I speak too many languages …This is much less readable if non-trivial. It’s easy enough here, but now I need to search through the code to see where else foo was set.