This week's book giveaway is in the OO, Patterns, UML and Refactoring forum. We're giving away four copies of Refactoring for Software Design Smells: Managing Technical Debt and have Girish Suryanarayana, Ganesh Samarthyam & Tushar Sharma on-line! See this thread for details.
Don't use == to compare strings. Use equals() instead:
The == operator checks if two expressions refer to the exact same object. It does not compare the content of objects. Even if you have two separate String objects with the same content, comparing them with == will return false because they are two separate objects.