String equals not working correctly

Question

This is my debugging code:

log.error(u.getName() + " - " + u.getName().length() + " - host_node" + " - " + u.equals("host_node"));

My log:

11:27:16 [main] ERROR com.google - host_node - 9 - host_node - false

Why it doesn't equal, I am even checking length, to see if first contains whitespaces. But it does not.

Are you overriding equals? Or perhaps you want u.getName().equals("host_node") — Lews Therin, Nov 07 '12 at 09:33
GAHHH JUST WAS TYPO. I wish I could delete this question. Most of answers are correct. — Jaanus, Nov 07 '12 at 09:35

score 8 · Accepted Answer · answered Nov 07 '12 at 09:33

8

I would use following as I suspect u is not a String.

u.getName().equals("host_node")

or better you can use the following to get false if the name is null.

"host_node".equals(u.getName())

answered Nov 07 '12 at 09:33

Peter Lawrey

I will always go with second option. Safe check to save the code from most irritating NullPointerException. :) – Vishal Nov 07 '12 at 10:50

score 1 · Answer 2 · edited Nov 07 '12 at 09:41

1

You should compare u.getName() with the text, thus:

u.getName().equals("host_node")

edited Nov 07 '12 at 09:41

sp00m

answered Nov 07 '12 at 09:33

iOS

score 0 · Answer 3 · answered Nov 07 '12 at 09:34

0

This

 u.equals("host_node")

compares the string with the object u, not its name! i.e. it's calling u.equals()

answered Nov 07 '12 at 09:34

Brian Agnew

score 0 · Answer 4 · answered Nov 07 '12 at 09:35

0

I guess

u.equals()

is not what you wanted to compare. Maybe you want the name of u?

u.getName().equals() would be a better choice then.

Anyhow, your Post lacks some information about u and other context to answer this properly.

answered Nov 07 '12 at 09:35

Stefan

4 Answers4