Question to sololearn

Question

Is this code right

#include <iostream>
using namespace std;
int main ()
{
int num1,num2;
string operator;
cout <<"Enter either addition or subtraction or multiplication or division or modulus=";
cin>>operator;
cout <<"Enter two numbers=";
cin>>num1>>num2;
switch (operator){
case "+":
cout <<num1+num2;
break;
case "-":
cout <<num1-num2;
break;
case "*":
cout <<num1*num2;
break;
case "/":
cout <<num1*num2;
break;
case "%":
cout <<num1*num2;
break;
}
return 0;
}

Answer 1

Too late ^^" but thanks c':

Answer 2

Why did you set the operator as string , a character type will work .

Answer 3

Figured out operator is a keyword and that you should use char instead of string

Answer 4

seems like it is, but you can check it by yourself using the SoloLearn compiler

Answer 5

welp lemme check it

Answer 6

Oooops, sorry o.O

Answer 7

Didn't take the time to read lol, my apologies

Answer 8

np, at least you got it figured out. 😉

Answer 9

'operator' is a keyword you can't use it as a variable name!!! ... and yes char type will work just fine.

Answer 10

yup, we literally just said that. lol

Answer 11

1. use char
2. if you don't want to, then at least add a string header
3. and change the name of it

Answer 12

doesn't work