Need HELP!! PLZ

[ShehabKKhan]

Hi all,


i am creating a program for simple calculation (+,-,*,/) and i need the user to give the 1st number, operator, 2nd number. This is what i did, is this right, whats my mistake?

#include <iostream>
using namespace std;
int main(int argc, char *argv[]) {
	float n1;
	float n2;
	char op;
	float answer
	cin>>n1;
	cin>>op;
	cin>>n2;
	switch (op)
	{
	case "+": answer=(n1+n2);
			  cout<<"The answer is:"<<answer<<endl
				break;
	case "-": answer=(n1-n2);
			  cout<<"The answer is:"<<answer<<endl
				break;
	case "x": answer=(n1*n2);
			  cout<<"The answer is:"<<answer<<endl
				break;
	case "/": answer=(n1/n2);
			  cout<<"The answer is:"<<answer<<endl
				break;
    default: cout<<"Make sure you are using, '+' for addition, '-' for subtraction, '*' for multiplication and '/' for division!";			
	}
	return 0;
}

THANKS A LOT IN ADVANCE, THIS IS GOING TO HELP ME A LOT!!!

[Mats]

You called your char op but in the switch statement, you have written switch (operator) instead of op.

[MikeyBoy]

In your case statements, you are using double-quotes. These are string literals, so their value is a memory address, i.e. a pointer to char.

For single char variables, you should use single quotes.