my else statement problem

[Aquos]

Write your question here.
Hi all, im newbie that need your pro help, right now im trying to do a simple calculator that do simple math like, addition,substraction,multiplication and division, the problem is i when my program get the answer, it also showing the else statement that supposedly for user that put wrong input when my program asking for math symbol, so can anybody help me, how to correct this coding? your help much be appreciate, this code only for school process, so i dont need the advance coding, thanks for your help in advance~ :)

  #include <iostream>
#include <string>
#include <math.h>
#include <Windows.h>

using namespace std;

void main(){
	HANDLE hConsole;
	hConsole = GetStdHandle (STD_OUTPUT_HANDLE);

	int input1, input2;
	char math = ('+','-','/','*');
SetConsoleTextAttribute (hConsole, 6);

	cout<<"                             Simple Calculator 3.5v.\n";
	cout<<"                               Develop by Mercier\n\n";

SetConsoleTextAttribute (hConsole, 7);
	cout<<"What type of mathematic operation do you like to use? (+,-,*,/)\n\n";
	cin>>math;cout<<endl;
	
	
	if (math == '+'){

SetConsoleTextAttribute (hConsole, 2);cout <<"                        You have choosen addition operation.\n\n";SetConsoleTextAttribute (hConsole, 7);
		cout <<"What is your first number(s)?\n";
		cin>>input1;cout<<"\n";
		cout<<"What is your second number(s)?\n\n";
		cin>>input2;cout<<"\n";
		cout<<"Your total addition for both numbers is "<<input1 + input2<<endl<<endl;
	}

		if (math == '-'){

SetConsoleTextAttribute (hConsole, 4);cout <<"                        You have choosen substraction operation.\n\n";SetConsoleTextAttribute (hConsole, 7);
		cout <<"What is your first number(s)?\n\n";
		cin>>input1;cout<<"\n";
		cout<<"What is your second number(s)?\n";
		cin>>input2;cout<<"\n";
		cout<<"Your total substraction for both numbers is "<<input1 - input2<<endl<<endl;
	}

			if (math == '*'){

SetConsoleTextAttribute (hConsole, 6);		cout <<"                        You have choose multiplication operation\n\n";SetConsoleTextAttribute (hConsole, 7);
		cout <<"What is your first number(s)?\n";
		cin>>input1;cout<<"\n";
		cout<<"What is your second number(s)?\n";
		cin>>input2;cout<<"\n";
		cout<<"Your total multiplication for both numbers is "<<input1 * input2<<endl<<endl;
	}
	if (math == '/'){

SetConsoleTextAttribute (hConsole, 3);		cout <<"                        You have choosen devision operation\n\n";SetConsoleTextAttribute (hConsole, 7);
		cout <<"What is your first number(s)?\n";
		cin>>input1;cout<<"\n";
		cout<<"What is your second number(s)?\n";
		cin>>input2;cout<<"\n";
		cout<<"Your total devision for both numbers is "<<input1 / input2<<endl<<endl;
	}
	
	else if (math == 'x');{
		cout<<"You can only put these four math symbol:\n";
		cout<<"(+) for addition.\n";
		cout<<"(-) for substraction.\n";
		cout<<"(*) for multiplication.\n";
		cout<<"(/) for devision."<<endl<<endl;
	}

	//unfinish calculator kalau user type lain, error keluar "You not put right input".
	system ("pause");
	
}

[AbstractionAnon]

Line 63, you have a ; after the if statement. That terminates the if statement.
Lines 64-69 are not considered part of the if statement and will be executed unconditionally.

[Aquos]

owh thanks, i try it out :)

edit:

Thanks AbstractionAnon!! Its really work~owh my, thats was my mistake, just very simple mistake can make my program running crazy, i need to be more careful, thanks again!

[Aquos]

but how my program want to cout the "You can only put these four math symbol:\n" if user put other than the 4 symbol?

[nedo]

i suggest you use a switch and you put in the default the "You can only put these 4 math symbols"
it should look something like this:

switch(math)
{
   case '+':
   {
           code for +
   }
   case '-':
   {
          code for -
   }
   ......
   default:
   {
        cout << "You can only put these four math symobls:\n";
   }
}

[Aquos]

thanks for suggestion :)

[Aquos]

any other suggestions would much appreacite~ to enhance my skill~