Mój kalkulator ... proszę o pomoc

Question

Witam , proszę o pomoc, otóż próbuję zrobić mój pierwszy kalkulator , jednak mam pewien problem...

Proszę  spojrzeć .Kiedy próbuje zastsować instrukcje if dla jednego string to wyskakują mi dwie na raz , niezależnie jaka cyfrę wpiszę. Proszę mnie uświadomić gdzie zrobiłem błąd . Zaznaczę że jestem początkujący w tym całym programowaniu.

Z góry dziękuje

}        
system("cls");
cout << "Somonta 3000  vs 1.000" <<endl;  
cout<<"\n"<<endl;
cout<< "Witaj w swoim osobistym kalkulatorze 'Somoneta'."<<endl;
cout<<"Wpisz swoje imie:";
getline(cin, imie, '\n');
cout<<"Witaj "<<imie<<endl;
cout <<"Zobaczmy co my tu mamy do wyboru:"<<endl;
Sleep(500);
cout <<"1.Dodawanie"<<endl;
Sleep(500);
cout <<"2.Odejmowanie"<<endl;
Sleep(500);
cout <<"3.Mnozenie"<<endl;
Sleep(500);
cout <<"4.Dzielenie"<<endl;
Sleep(500);
cout <<"5.Procenty"<<endl;
cout<<"\n"<<endl;
cout<<"Wybierz 1 , 2 , 3 ,4 , lub 5 : ";
cin>>wybor;
//dodawanie
if(wybor==1);
{
cout<<"Czesc "<<imie<<" Wybrales dodawanie.."<<endl;
}               
//odejmowanie
if(wybor==2);
{
cout<<"Czesc "<<imie<<" Wybrales odejmowanie.."<<endl;
}               
system("pause");
}

 

Comments

By dostać pomoc musisz zamieścić swój kod w formie tekstowej. Masz do tego na tym forum specjalną funkcję do wklejania kodu w przystępnej formie.

Answer 1

Dla następnych ifów dopisz else, czyli tak: if(warunekA) {...}  else if (warunekB) {...} else if ....

Comments

Cześć , próbuje tak jak piszesz , ale wyskakuje mi błąd : 43 D:\kurss x++\BezNazwy1.cpp expected `;' before "else"

---

spróbuj if(wybor!=1)

---

Już działa , dzięlki za pomoc

Answer 2

Usuń średniki przy obu instrukcjach warunkowych if.

Comments

Ok działa , dzięki :)

---

Jak już  C☺ndzi zasugerował można by dodać else przy kolejny if'ach. Tzn komputer nie będzie sprawdzał wszystkich warunków gdy jeden się spełni. Taka mała oszczędność obliczeń. PS. Zostaw łapkę tym którzy pomogli :)

    if(wybor==1)
    {
        cout<<"Czesc "<<imie<<" Wybrales dodawanie.."<<endl;
    }
    //odejmowanie
    else if(wybor==2)
    {
        cout<<"Czesc "<<imie<<" Wybrales odejmowanie.."<<endl;
    }

 

---

Oszczędności obliczeń nie będzie, bo kompilator (przynajmniej z włączonym optymalizowaniem) wie, że jeśli liczba równa jest 1, to nie jest równa 2.

Answer 3

Pokaż kod bo na zdjęciu słabo widzę

Comments

Juz wpisałem .

Answer 4

Dlaczego nie zastosujesz funkcji warunkowej case?

Comments

Tutaj nie ma takiej pilnej potrzeby ;)

---

No wiem, że nie ma pilnej potrzeby, ale wydaję mi się, że byłoby o wiele wygodniej, a także byłoby bardziej czytelniej.

---

Od kiedy to:

switch(wybor)
{
case 1:
    {/**/}
    break;
case 2:
    {/**/}
    break;    
case 3:
    {/**/}
    break;
case 4:
    {/**/}
    break;  
}

Jest bardziej wygodne i czytelne niż to:

if(wybor==1) {/**/}
if(wybor==2) {/**/}
if(wybor==3) {/**/}
if(wybor==4) {/**/}

Moim zdaniem switch przy takich prostych instrukcjach to utrudnianie sobie życia. Co innego gdyby było dużo warunków z alternatywą:

if(wybor==1 || wybor==2 || wybor==3 || wybor==4) {/**/}
if(wybor==5 || wybor==6 || wybor==7 || wybor==8) {/**/}

switch(wybor)
{
case 1: case 2: case 3: case 4:
    {/**/}
    break;
case 5: case 6: case 7: case 8:
    {/**/}
    break;    

}