Map.get() optimization in ?: ternary operator

Question

Consider the following code:

java.util.Map<String, String> map = new java.util.HashMap<String, String>();
...
String key = "A";
String value = map.get(key) == null? "DEFAULT_VALUE" : map.get(key); // (1)

Would the compiler optimize the line (1) something similar to:

String tmp = map.get(key);
String value = tmp == null? "DEFAULT_VALUE" : tmp;

(or to:

String value = map.get(key);
if(value == null) value = "DEFAULT_VALUE";

) ?

Answer 1

Not sure if you are asking which corresponds to what the compiler will make of the original expression, in which case the answer is:

Neither--

In the example you are calling map.get() twice; the compiler can't know that there are no side effects, so it will call it twice when a value is found.

Probably closest to

String tmp = map.get(key);
String value;
if (tmp == null)
    value= "DEFAULT_VALUE";
else
    value = map.get(key);

Or if you are asking which alternative is most efficient, in which case the answer is:

The second alternative is probably slightly better because it does not require an additional local variable. An additional local variable imposes a slight overhead on the JVM, but it will probably amount to nothing at runtime once the JIT gets through with it.

Answer 2

The IF one is preferable.

Its more readable & self explained.

Additionally: execution time is not a big deal, at least here.

Answer 3

Your second option which is :

String value = map.get(key);
if(value == null) {
    value = "DEFAULT_VALUE";
}

is better from first one reason:

1. You dont create an extra reference "tmp"

2. You dont execute else which you do as part of your ternary comparison.