Operator if i else

Question

Witam!

Mam problem z else i if. Nie wiem jak zrobić, aby odczytywało, że jeśli nie wpisze użytkownik znaku +, -, * lub / to wyskakiwało " Podałeś zły znak". Na chwilę obecną program nie odczytuje tego, co zapisałam w nawiasie po if. Dziękuję za każdą pomoc.

#include <iostream>

using namespace std;

int main()
{
    unsigned int a, b, c, d = 0;
    char zn;
    bool w = true;

    cout << "Witaj uzytkowniku! Milo cie widziec!" << endl << endl;
    cout << "Co ty na to abysmy wykonali dzialanie?" << endl;
    cout << "Najpierw podaj znak (+, -, *, /):" << endl;
     cin >> zn;
    if(zn == '+' || '-' || '*' || '/')
        {
        cout << "Brawo! Podalas/es dobry znak!" << endl;
    }else if(zn != '+' || '-' || '*' || '/'){
        cout << "Podałaś/eś zły znak!" << endl;
        cout << "Podaj znak jeszcze raz (+, -, *, /):" << endl;
    cin >> zn;
        }
    cout << "Teraz podaj a: ";
    cin >> a;
    cout << "A teraz b: ";
    cin >> b;


    switch(zn)
{
        case '+':
            c = a + b;
            break;
        case '-':
            c = a - b;
            break;
        case '*':
            c = a * b;
            break;
        case '/':
            c = a / b;
            break;
        default:
            cout << "Nieprawidlowe dzialanie, podales/as nieznany nam znak, wybierz znak z ramki" << endl << endl << endl << endl << endl ;
            w = false;
            break;
}
    {
    if (w) cout << a << "  " << zn << "  " << b << " = " << c << endl << endl;

    cout << "Dziekuje za wykonanie dzialania! :D";

    return 0;
    }
}

 

Answer 1

Musisz każdy symbol porównać ze znakiem wpisanym, a nie wymieniać je po kolei pomiędzy znakami ||.

Comments

Co oznacza " każdy symbol porównać ze znakiem wpisanym"? Jak to wygląda?

---

Chodzi mi o coś takiego 

if(zn=='+' || zn=='-')
{
...
}

Bo inaczej wszystko co nie jest konwertowane do 0 daje prawdopodobnie i te warunki zawsze są prawdą. Podobnie w else if. Pomyśl czy ten warunek ma sens. Czy jeśli wpisze '6' to wejdzie do tego else if'a. Czy tam w ogóle musi być warunek?

Answer 2

Posługujesz się switch-em, wykorzystaj go tak.  

ten kawałek kodu

. . .
    
bool w = true;
 
    cout << "Witaj uzytkowniku! Milo cie widziec!" << endl << endl;
    cout << "Co ty na to abysmy wykonali dzialanie?" << endl;
    cout << "Najpierw podaj znak (+, -, *, /):" << endl;
    cin >> zn;
    if(zn == '+' || '-' || '*' || '/')
    {
        cout << "Brawo! Podalas/es dobry znak!" << endl;
    } else if(zn != '+' || '-' || '*' || '/') {
        cout << "Podałaś/eś zły znak!" << endl;
        cout << "Podaj znak jeszcze raz (+, -, *, /):" << endl;
        cin >> zn;
    }

. . .

zamień na

. . .

    bool w = false;
 
    cout << "Witaj uzytkowniku! Milo cie widziec!\n\n"
         << "Co ty na to abysmy wykonali dzialanie?" << endl;
         
    while (!w)
    {
        cout << "Podaj znak (+, -, *, /):" << endl;
        cin >> zn;
     
        switch(zn)
        {
            case '+':
            case '-':
            case '*':
            case '/':
                cout << "Brawo! Podalas/es dobry znak!" << endl;
                w = true;
                break;
            default:
                cout << "Podałaś/eś zły znak!" << endl;
        }
    }

. . .