toString invoked on a string value is useless JAVA-P0064
Performance
Major
a year ago
—
2 years old
String.toString is redundant and inefficient, use the string directly instead 87 break;
88 }
89 if (seat != null)
90 System.out.println(i + 10 + alphaPos + ": " + seat.name.toString()); 91 }
92 }
93 }String.toString is redundant and inefficient, use the string directly instead 52
53 }
54 if (seat != null)
55 System.out.println(j + 1 + alphaPos + ": " + seat.name.toString()); 56 }
57 }
58 } else {
Description
Calling String.toString is a redundant operation. Just use the string directly.
Bad Practice
String b = "abc".toString();
Recommended
String b = "abc";
Exceptions
There are some exceptions to this, such as within generated code where such statements are likely to appear. Consider adding these files to the exclude_files or exclude_patterns to reduce false positives.
References
- Spotbugs - DM_STRING_TOSTRING