Niewielki kalkulator || Sprawdzenie zadania.

Question

Treść zadania:

Napisz niewielki kalkulator, który pobiera na wejściu jeden z operatorów arytmetycznych oraz dwa argumenty, po czym wyświetla wynik obliczeń otrzymanych na podstawie tych danych.
 

Rozwiązanie:

#include <iostream>
#include <string>

using namespace std;

int main()
{
    string moj_operator;
    int liczba_1, liczba_2, wynik;
    cout<<"Podaj operator arytmetyczny: ";
    cin>>moj_operator;
    if((moj_operator=="+")||(moj_operator=="-")||(moj_operator=="/")||(moj_operator=="*"))
    {
        cout<<"Podaj pierwsza liczbe: ";
        cin>>liczba_1;
        cout<<"Podaj druga liczbe: ";
        cin>>liczba_2;

    if (moj_operator=="/")
    {
        wynik=liczba_1/liczba_2;
        cout<<wynik;
    }

    if (moj_operator=="*")
    {
        wynik=liczba_1*liczba_2;
        cout<<wynik;
    }

    if (moj_operator=="+")
    {
        wynik=liczba_1+liczba_2;
        cout<<wynik;
    }

    if (moj_operator=="-")
    {
        wynik=liczba_1-liczba_2;
        cout<<wynik;
    }
    }
    else
    {
        cout<<"Niepoprawny operator";
    }

    cin.ignore();
    cin.get();
}

Witam napisałem taki właśnie kod. Zastanawiam się czy zadanie rozwiązałem w miarę dobrze. Obecnie zaczynam zapoznawać się z pętlami. Oczywiście znalazłem teraz rozwiązanie z użyciem przełącznika, swoją drogą o wiele lepsze. Z góry dziękuje za pomoc, cenne rady. Wytłumaczenie co robię źle. Dopiero zaczynam więc nie chce już na początku wpajać sobie złe nawyki.

Answer 1

Można się ew. przyczepić do dzielenia. Spróbuj wprowadzić 5/0. A jak na początki to całkiem dobrze. :)

Comments

Dodatkowo:

if( ... ) {
 // A
 // B
}

if( ... ) {
 // C
 // B
}

if( ... ) {
 // D
 // B
}

można zamienić na:

if( ... ) {
 // A
}

if( ... ) {
 // C
}

if( ... ) {
 // D
}

// B

A nawet...

if( ... ) // A
if( ... ) // C
if( ... ) // D

// B

Gdzie // X to zestaw instrukcji.

Answer 2

Niektórzy uważają że stado if'ów do tak prostego programu jest złym nawykiem ponieważ nie jest to zbyt wydajne rozwiązanie. Ale do rzeczy: Kombinujesz dobrze tylko nie jest potrzebne zaprzęganie klasy string do tego programu. Znacznie wydajniejsze jest użycie zwykłej zmiennej typu char i jak porównujesz to trzeba używać pojedynczych nawiasów.

char znak;
cin >> znak;
switch(znak)
{
    case '+':    //robi praktycznie to samo co if(znak=='+')
    {
        //...
        break;
    }
    //...
    default:
    {
       / /...
        break;
    }
}

 

Answer 3

Dziękuje za odpisałeś. Poprawiłem kod. Mam nadzieję że twoje porady dobrze wprowadziłem w życie.

 

#include <iostream>

using namespace std;

int main()
{
    char znak;
    int liczba_1, liczba_2;
    cout<<"Podaj operator arytmetyczny: ";
    cin>>znak;
    cout<<"Podaj pierwsza liczbe: ";
    cin>>liczba_1;
    cout<<"Podaj druga liczbe: ";
    cin>>liczba_2;

switch(znak)
{
    case '/':
    {
        if(liczba_2 != 0)
        cout<<liczba_1/liczba_2;
        else
        cout<<"Nie dziel przez 0";
        break;
    }
    case '*':
    {
        cout<<liczba_1*liczba_2;
        break;
    }
     case '+':
    {
        cout<<liczba_1+liczba_2;
        break;
    }
     case '-':
    {
        cout<<liczba_1-liczba_2;
        break;
    }

}
    cin.ignore();
    cin.get();
}

 

Comments

Wygląda dobrze tylko dla przejrzystości dodałbym odstęp w całej instrukcji switch. Czytaj linijki 16-42 jeden tab(lub 4 spacje) w prawo.

---

Mam nadzieję że dobrze zrozumiałem.

#include <iostream>

using namespace std;

int main()
{
    char znak;
    int liczba_1, liczba_2;
    cout<<"Podaj operator arytmetyczny: ";
    cin>>znak;
    cout<<"Podaj pierwsza liczbe: ";
    cin>>liczba_1;
    cout<<"Podaj druga liczbe: ";
    cin>>liczba_2;

    switch(znak)
{
    case '/':
    {
        if(liczba_2 != 0)
        cout<<liczba_1/liczba_2;
        else
        cout<<"Nie dziel przez 0";
        break;
    }
    case '*':
    {
        cout<<liczba_1*liczba_2;
        break;
    }
     case '+':
    {
        cout<<liczba_1+liczba_2;
        break;
    }
     case '-':
    {
        cout<<liczba_1-liczba_2;
        break;
    }

}
    cin.ignore();
    cin.get();
}

 

---

tak tylko jeszcze nawiasy klamrowe w linijkach 17 i 42 nie żebym się czepiał ale to naprawdę poprawia jakość i czytelność kodu.