views:

438

answers:

3

I've got a problem that I'm rather confused about. I have the following lines of code in my android application:

System.out.println(CurrentNode.getNodeName().toString());
if (CurrentNode.getNodeName().toString() == "start") {
    System.out.println("Yes it does!");
} else {
    System.out.println("No it doesnt");
}

When I look at the output of the first println statement it shows up in LogCat as "start" (without the quotes obviously). But then when the if statement executes it goes to the else statement and prints "No it doesn't".

I wondered if the name of the node might have some kind of non-printing character in it, so I've checked the length of the string coming from getNodeName() and it is 5 characters long, as you would expect.

Has anyone got any idea what's going on here?

Cheers,

Robin

+11  A: 

Use String's equals method to compare Strings. The == operator will just compare object references.

if ( CurrentNode.getNodeName().toString().equals("start") ) {
   ...
Bill the Lizard
+4  A: 

You need to use .equals

if ("start".equals(CurrentNode.getNodeName().toString()) { ... }
SB
+4  A: 

Use CurrentNode.getNodeName().toString().equals("start").

In Java, one of the most common mistakes newcomers meet is using == to compare Strings. You have to remember, == compares the object identity (Think memory addresses), not the content.

Xavier Ho