How do I get this if else statement to work with a String variable? (Java)

Question

The script below is a test script and I can get the if else to run when the variable is an int. But I can't seem to get it to run the if else statement when a string is entered.

import java.util.Scanner;

public class ifelse
{
   public static void main (String[] args)
   {
      Scanner keyboard = new Scanner (System.in);

      String money;

      System.out.println("Enter how much money you have.");
      money = keyboard.nextLine();

      if (money == "fifty"){

      System.out.println("money");
      }
      else if(money == "seventy"){
      System.out.println("Nothing");
      }
   }
}

The code below works and it's using int for the variable.

import java.util.Scanner;

public class ifelse
{
   public static void main (String[] args)
   {
      Scanner keyboard = new Scanner (System.in);

      int money;

      System.out.println("Enter how much money you have.");
      money = keyboard.nextInt();

      if (money == 50){

      System.out.println("money");
      }
      else if(money == 70){
      System.out.println("Nothing");
      }
   }
}

Any help is appreciated. Thank you!

Francisco Romero · Accepted Answer · 2015-09-08T00:07:15.083

1

You have to compare Strings with .equals function, not with ==.

if(money.equals("fifty")){
 //Code
}

edited Sep 08 '15 at 00:07

answered Sep 07 '15 at 23:55

Francisco Romero

12,787
22
92
167

Beat me to it. +1 Better yet, mention that you use .equals for any Objects. – Gamma2626 Sep 07 '15 at 23:57
@Gamma2626 I guess he is a totally beginner so I think it is better simplify it for not confuse him. – Francisco Romero Sep 08 '15 at 00:01
Great! worked perfect! I had to add an extra ")" at the end of the statement. Appreciate it, could not find a clear answer online. – Matt Sep 08 '15 at 00:03
Missing a closing `)` in your answer – domdomcodecode Sep 08 '15 at 00:04
@d_ominic For sure. Just a mistake. Thanks for the point! – Francisco Romero Sep 08 '15 at 00:07
This answer adds nothing compared to the much more informative answers in the canonical post. If you have new information to add on the subject, add it there in order to avoid fragmentation. – Jeroen Vannevel Sep 08 '15 at 00:07
@JeroenVannevel Oh, it wasn't my purpose to fragment the information. I really don't have new information about that but I adapted the function to the OP question. Maybe it could help a little bit more to him. – Francisco Romero Sep 08 '15 at 00:09

How do I get this if else statement to work with a String variable? (Java)

1 Answers1