Boolean constructor is inefficient, consider using Boolean.valueOf instead JAVA-P0066
Performance
Major
8 months ago
—
8 months old
Boolean constructor is inefficient, use Boolean.valueOf instead 56
57 Boolean getIsLocalFromConfigData(ConfigData configData) {
58 if (configData.getParams().get("thing1") == null) return null;
59 else return new Boolean(configData.getParams().get("thing1")); 60 }
61
62 /**
Description
Creating new instances of java.lang.Boolean wastes memory, since Boolean objects are immutable and there are only two useful values of this type.
Bad Practice
Boolean a = new Boolean(true);
Recommended
Use the Boolean.valueOf method (or autoboxing since Java 1.5) to create Boolean objects instead.
Boolean a = true;
// or
Boolean b = Boolean.valueOf(true);
Note - This issue will be ignored within tests.
References
- Spotbugs - DM_BOOLEAN_CTOR