if else not working

[silverglade]

Hi, my if else statement is not working. If I enter the name "Pawel", the program is supposed to output "Hello Pawel, I hope you are having a nice day!".
But when I enter Pawel as the name, it never goes to that line, it skips and outputs what is in the else block. Any help greatly appreciated. Thanks. Derek

here is the code.

//name and random number by Derek Van Derven


import java.util.Scanner; //import scanner class specifically
import java.util.Random;//import random class


public class nameRandom
	{
	
		public static void main (String[] args)
		{
			Random generator = new Random();
			int num1;
			num1 = generator.nextInt(100);//create random number between 0 and 99
			
			String firstName;
			String lastName;
			
			Scanner scan = new Scanner (System.in); //create scanner obj, System.in is the keyboard.
			System.out.println ("Enter your first name.");
			firstName = scan.nextLine();
			System.out.println ("Enter your last name");
			lastName = scan.nextLine();
			
			if (firstName == "Pawel" || firstName == "pawel")
					{
					System.out.println("Hello Pawel, I hope you are having a nice day!");
					System.out.println("Your name is: " + firstName + " " + lastName + ". Thanks Pawel");
					System.out.println("Here is a random number 0-99: " + num1);
					}
				else 
					{
					System.out.println("Your name is: " + firstName + " " + lastName + ".");
					System.out.println("Here is a random number 0-99: " + num1);
					}
				
		}
	}

[Fubarable]

Don't use == to compare Strings since this checks to see if one String object is the same as another, something you're not interested in. Instead use String's equals(...) or equalsIgnoreCase(...) method which will check if the character sequences of the String objects are the same -- something you are interested in.

[KevinWorkman]

Don't use == with Strings (or any Object unless you're testing whether the references are the same). Use the equals() method instead.

Edit- Too slow!

[Fubarable]

Don't use == with Strings (or any Object unless you're testing whether the references are the same). Use the equals() method instead.

Edit- Too slow!

Ha! :)

[Dark]

You also might want to use .equalsIgnoreCase() that way you don't have to have a seperate check for Pawel, pAwel, paWel, etc...

[silverglade]

THANK YOU SO MUCH!!!! GOD I LOVE THIS FORUM. This is the best forum with the nicest people for any programming forum I have ever been on. And I receive the most help. Thank you all and fubarable I NEVER would have caught that. here is the new code snippet.

if (firstName.equalsIgnoreCase("Pawel"))
					{
					System.out.println("Hello Pawel, I hope you are having a nice day!");
					System.out.println("Your name is: " + firstName + " " + lastName + ". Thanks Pawel");
					System.out.println("Here is a random number 0-99: " + num1);
					}