JavaScript is not currently enabled, but is required for full CodeSonar manual search and browse functionality.
If you are viewing this file in your hub's Web GUI, enable JavaScript in your browser: you will also need it for GUI functionality.
If you opened this file directly from disk, your browser may be directly suppressing JavaScript functionality: certain browsers perform this suppression on local files (but not files delivered by web servers) for security reasons.
| CodeSonar® 9.2p0 | CONFIDENTIAL | CodeSecure Inc |
The construction of an object might be replaced by a literal or by a call to a factory method.
It is often convenient to use a literal or a unique object instead of explicitly constructing new instances, when the same object is used many times, since distinct objects are created at each new statement, which affects the performance of the code. Moreover, objects used instead of literals are often short-lived and consequently must be claimed back from the garbage-collector.
| Class Name | Inefficient Instantiation (Java) | |||||||||
|---|---|---|---|---|---|---|---|---|---|---|
| Significance | reliability | |||||||||
| Mnemonic | JAVA.CLASS.UI | |||||||||
| Categories |
|
|||||||||
| Availability | Available for Java and Kotlin. |
|||||||||
| Enabling | Checks for this warning class are enabled by
default. To disable them, add the following WARNING_FILTER rule to the
project configuration file.
WARNING_FILTER += discard class="Inefficient Instantiation (Java)" |
Japanese (ja)
import java.util.HashMap;
import java.util.Map;
public class InefficientConstruction {
public static void foo() {
String s = new String("hello"); /* "Inefficient Instantiation (Java)" warning issued here
* This String instantiation can be replaced by a string literal.
*/
Map<Integer, String> map = new HashMap<Integer, String>();
map.put(new Integer(13) , s); // "Inefficient Instantiation (Java)" warning issued here
* This Integer object is recreated at every execuition of foo().
*/
}
}
The programmer could modify the program to address the problem as follows.
public static void foo() {
String s = "hello";
Map<Integer, String> map = new HashMap<Integer, String>();
map.put(Integer.valueOf(13), s);
/* Integer.valueOf(13) uses a cache to yield the same object
* each time foo() is executed, at least for small values.
*/
}
The automatic boxing mechanism uses the same optimization, so we can further optimize to:
public static void foo() {
String s = "hello";
Map<Integer, String> map = new HashMap<Integer, String>();
map.put(13, s);
}
Use string literals instead of the creation of constant strings. Use factory methods or the automatic boxing mechanism instead of the instantiation of a primitive wrapper class.
The following configuration file parameters affect checks for this warning class.
To report problems with this documentation, please visit https://support.codesecure.com/.