Why doesn't my string comparison work?

Question

Ok, this is stupid, but wtf is going on?

I have a String variable in a servlet, which takes the value of a parameter and based on that value I make a test to do something, but the if is not working. What is the problem?

 String action = request.getParameter("action");
    System.out.println("Action: " + action);
// I put 2 ifs to be sure, but not even one is working
    if(action.equals("something"))
            {
                System.out.println("hey");            
            }
    if(action.trim() == "something")
            {
                System.out.println("hey");
            }

On the console, the System.out.println shows me that the value of action is "something"

Action: something

Answer 1

Your second comparison is wrong. You should also use equals instead of ==, like this:

if (action.trim().equals("something"))

The == operator compares references of (String) objects and under normal circumstances equal strings don't automatically have the same reference, i.e. they are different objects. (Unless both are internalized, but normally you shouldn't consider it)

Other than that your example works fine and the first comparison is valid. Try fixing the second comparison. If it works, you found your problem. If not, try using a debugger and double-check everything.

PS: When comparing literal strings with dynamic string objects, it's good practice to call the equals method on the literal string:

"something".equals(action)

That way you can avoid NullPointerExceptions when the string object is null.

Answer 2

Your second condition is unlikely ever to be true - you're testing whether the string object created by trimming action is the same object as the string literal "something". This will only be true if action is set to that same literal value elsewhere. Use "something".equals( action.trim() ) instead.

Your first condition would be true the characters in the action string are the characters "something". If it's not true, then it doesn't. Assert it in a test, log it, print it or look at it in a debugger.

If you print a string for debugging, use something like System.out.println ( "String = >" + string + "<" ); so that it's obvious if there are any trailing spaces.