Thread: Hi I have a problem with my calculator

Here is the code for my simple calculator:

>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>>>>>
package calculator_1;

import java.util.Scanner;

public class apples {
public static void main(String[] args) {
Scanner scan = new Scanner(System.in);
System.out.println("Calculator");

System.out.println("Enter a number");
int number1 = scan.nextInt();


System.out.println("Enter a number");
int number2 = scan.nextInt();


// doing math with numbers
int add = number1 + number2;
int sub = number1 - number2;
int times = number1 * number2;
int divide = number1 / number2;

//choosing operation to use, if statements.
System.out.println("What operation do you want to use?: add, sub, times, divide");
Scanner opy = new Scanner(System.in);
String operator = opy.nextLine();

if (operator == "add")
{
System.out.println(" Add: " + add);
}
if (operator == "sub")
{
System.out.println(" Sub: " + sub);
}

if (operator == "times")
{
System.out.println(" Times: " + times);
}
if (operator == "divide")
{
System.out.println(" Divide: " + divide);
}



}

}

>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>>>>>

My code seems to work with allowing me to enter my two numbers but the problems exist when it asks me what order of operation I want to apply to my two numbers with the prompt working but when I enter my response like "add" for example, nothing happens, please help.


Thanks for your help
Jesse....

Welcome to the Forum! Please read this topic to learn how to post code correctly and other useful tips for newcomers.

Please post your code correctly per the above link.

Do not use '==' to compare Strings. Instead, use the equals() method: if "thisString".equals( "thatString" );

Sorry for the incorrect code convention and thanks for answering my question now that part seems to be solved I have run into another problem to do with the if statements which when I type "add" for example after the prompt ask's me what operation I want it displays all of the operations. My code is here:

package calculator_1;
 
import java.util.Scanner;
 
public class apples {
	public static void main(String[] args) {
		Scanner scan = new Scanner(System.in);
		System.out.println("Calculator");
 
		System.out.println("Enter a number");
		int number1 = scan.nextInt();
 
 
		System.out.println("Enter a number");
		int number2 = scan.nextInt();
 
 
		// doing math with numbers
				int add = number1 + number2;
				int sub = number1 - number2;
				int times = number1 * number2;
				int divide = number1 / number2;
 
				//choosing operation to use, if statements.
		System.out.println("What operation do you want to use?: add, sub, times, divide");
		Scanner opy = new Scanner(System.in);
		String operator = opy.nextLine();
 
		if (operator.equals( "add" )); 
		{
			System.out.println(" Add: " + add);
		}
		 if (operator.equals( "sub" ));
		{
		System.out.println(" Sub: " + sub);
		}
 
		 if (operator.equals( "times" ));
		{
			System.out.println(" Times: " + times);
		}
		 if (operator.equals ( "divide" ));
		{
			System.out.println(" Divide: " + divide);
		}
 
 
 
	}

Semicolons at the end of your if() statements complete the clauses so that the intended bodies of the if() statements are not connected to the if() statement conditions and will execute regardless. Remove the semicolons at the ends of your if() statements.