Prosty kalkulator bardzo wczesnego początkującego programisty - problem z if

Question

Witam,

Niedawno rozpocząłem moją przygodę z programowaniem i postanowiłem napisać prosty kalkulator oparty na wybieraniu działań. Na razie jest wybór między dodawaniem a odejmowaniem. Użyłem instrukcji if do wyboru, lecz po włączeniu programu wyświetla mi się od razu linijka do podawania cyfr bez możliwości wyboru działania. Sam jakoś nie mogę się domyślić co jest nie tak więc, proszę o pomoc 

#include <iostream>

using namespace std;

string Dodawanie, Odejmowanie;
int pierwsza, druga, x;

int main()
{
    cout << "Wybierz 1, aby dodawac, wybierz 2, aby odejmowac";

    if(Dodawanie=="1")
    {
        cout << "Wybrano dodawanie";
    }
    if(Odejmowanie=="2")
    {
        cout << "Wybrano odejmowanie";
    }

    cout << "Pierwsza liczba: ";
    cin >> pierwsza;

    cout << "Druga liczba: ";
    cin >> druga;

    x = pierwsza-druga;
    x = pierwsza+druga;

    cout << "Wynik: "<<x;

    return 0;
}

 

Comments

Oczywiscie ze tak bedzie. Brakuje Ci zmiennej...

#include <iostream>
 
using namespace std;
 
int pierwsza, druga, x;
string normalna_nazwa_pls;
 
int main()
{
    cout << "Pierwsza liczba: ";
    cin >> pierwsza;
 
    cout << "Druga liczba: ";
    cin >> druga;

    cout << "Wybierz 1, aby dodawac, wybierz 2, aby odejmowac";

    cin>>normalne_nazwa_pls;
 
    ifnormalne_nazwa_pls=="1")
    {
        cout << "Wybrano dodawanie";
 x = pierwsza+druga;
cout << "Wynik: "<<x;
    }
    if(normalne_nazwa_pls=="2")
    {
        cout << "Wybrano odejmowanie";
 x = pierwsza-druga;
cout << "Wynik: "<<x;
    }
    
 
    return 0;
}

 

Answer 1

Powinno to wyglądać tak, jeżeli chcesz tylko na ifach operować
 

#include <iostream>
using namespace std;
int pierwsza, druga, x, wybor;
 
int main()
{
    cout << "Wybierz 1, aby dodawac, wybierz 2, aby odejmowac"<<endl;
    cin>>wybor;
  
    cout << "Pierwsza liczba: ";
    cin >> pierwsza;
    cout<<endl; // przenosi do następnej linii
 
    cout << "Druga liczba: ";
    cin >> druga;
    cout<<endl;
    
  	if(wybor==1)
    {
        cout << "Wybrano dodawanie";
        cout<<endl;
		x = pierwsza+druga;
    }
    if(wybor==2)
    {
        cout << "Wybrano odejmowanie";
        cout<<endl;
		x = pierwsza-druga;
    }
    
    
 	cout<<endl;
    cout << "Wynik: "<<x;
 
    return 0;
}

 

 

Dodałam od siebie kilka "enterów", aby to wszystko było bardziej przejrzyste. 
Możesz spróbować go zrobić na case'each lub używając parametrów. Są to o wiele lepsze sposoby na utworzenie prostego kalkulatora.