Почему моя переменная не инициализируется?

Question

Я просто перехожу из c в C++ и пытаюсь построить калькулятор. Int 'result' не инициализируется с помощью операций maths. Логика заключается в том, что в зависимости от операции 'будет назначено другое значение, присвоенное «результату». Кажется, это не работает.

#include<iostream> 
#include<cstring> 
#include<cmath> 
using namespace std; 

int main() 
{ 
    int n1, n2; 
    char s,r; 
    int result = 0; 
    cout<< "Enter a calculation? (y/n)"<<endl; 
    cin>>r; 
    while(r=='y') 
    { 
     cout <<"Enter the first number"<<endl; 
     cin>>n1; 
     cout<<"Enter the operator"<<endl; 
     cin>>s; 
     cout<<"Enter the second number"<<endl; 
     cin>>n2; 

     if ('s' == '*') 
     { 
      result = n1*n2; 
     } 
     if ('s' =='+') 
     { 
      result = n1+n2; 
     } 

     if ('s' =='-') 
     { 
      result = n1-n2; 
     } 

     if ('s' =='/') 
     { 
      result = n1/n2; 
     } 
     cout << result<<endl; 
     cout<< "Enter a calculation? (y/n)"<<endl; 
     cin>>r; 
    } 
    return 0; 
} 

Answer 1

Вы должны сравнить с s не с 's'. Так что ваш код должен выглядеть

if (s == '*') 
{ 
    result = n1*n2; 
} 

Кодекса

if ('s' == '*') 

сравнивает характер s с характером *, который всегда ложно.

Answer 2

@OlafDietsche имеет это право.

Я хотел бы также рекомендовать переход на switch-case заявление:

switch(s) 
{ 
    case '*': result = n1*n2; break; 
    case '+': result = n1+n2; break; 
    case '-': result = n1-n2; break; 
    case '/': result = n1/n2; break; 
}