Inspectopedia
 
2024.3

Simplifiable 'Map' operations

Warning
New
Last modified: 03 December 2024

Reports common usage patterns of java.util.Map and suggests replacing them with: getOrDefault(), computeIfAbsent(), putIfAbsent(), merge(), or replaceAll().

Example:

After the quick-fix is applied:

Example:

After the quick-fix is applied:

Example:

After the quick-fix is applied:

Example:

After the quick-fix is applied:

Note that the replacement with computeIfAbsent() or merge() might work incorrectly for some Map implementations if the code extracted to the lambda expression modifies the same Map. By default, the warning doesn't appear if this code might have side effects. If necessary, enable the Suggest replacement even if lambda may have side effects option to always show the warning.

Also, due to different handling of the null value in old methods like put() and newer methods like computeIfAbsent() or merge(), semantics might change if storing the null value into given Map is important. The inspection won't suggest the replacement when the value is statically known to be nullable, but for values with unknown nullability the replacement is still suggested. In these cases, we recommended suppressing the warning and adding an explanatory comment.

This inspection depends on the Java feature 'Lambda methods in collections', which is available since Java 8.