As I study the source code of some open source products, I find code like:
if (a=="cluser")
a
is a String variable. Can a String equals
operation be replaced with ==
?
You should almost never use ==
and almost always use equals()
. It will only work with ==
if both strings reference the same object. There's an intern()
method on String
to return the same reference for a given string value. String literals are implicitly interned. Only if you have a very good reason should you use ==
for string comparison, and even then you need to be very careful.
The only good reason is performance, and very rarely will it matter. Only optimize once you know for sure that you need to do so. It's otherwise normally not worth the hassle. If you're looking at some open source code, they might have a case whereby the comparison is in a very tight loop or called very frequently and the optimization can help. Or it was just prematurely optimized and maybe seemed safe.
You can use == if the two strings are interned. String literals are defined to be interned, so if I, say, assign the literal "fred" to a String variable and pass it as a parameter to a method in another class where it's compared to "fred", the == test will be true.
So the specimen code is not certainly defective, just probably so.
If you love us? You can donate to us via Paypal or buy me a coffee so we can maintain and grow! Thank you!
Donate Us With