Czy da sie usprawnić bardziej ten kalkulator

Question

Witam,

Niedawno zacząłem swoją przygodę z C++. Napisałem dosyć prosty program, klakulator. Chciałbym się dowiedzieć czy można go jakoś usprawnić (krótszy kod). Z góry mówię, że pisany na linuxie Mint.

#include <iostream>
#include <unistd.h>
#include <stdio.h>

using namespace std;

float Liczba1, Liczba2, Wybor;
int main()
{
    cout << "Podaj 1 liczbę:";
    cin >> Liczba1;
    cout << endl;

    cout << "Podaj 2 liczbę:";
    cin >> Liczba2;
    cout << endl;

    cout << "Menu główne" << endl;
    cout << "------------------" << endl;
    cout << "1. Dodawanie" << endl;
    cout << "2. Odejmowanie" << endl;
    cout << "3. Mnożenie" << endl;
    cout << "4. Dzielenie" << endl;
    cout << "5. Wyjdź z programu" << endl;
    cin >> Wybor;

    if(Wybor == 1 )
    {
    cout << "Suma = " << Liczba1 + Liczba2;
    }

    else if (Wybor == 2 )
    {
    cout << "Różnica = " << Liczba1 - Liczba2;
    }

    else if (Wybor == 3 )
    {
    cout << "Iloczyn = " << Liczba1 * Liczba2;
    }

    else if (Wybor == 4 )
    {
        if (Liczba1 == 0)
        {
            cout << "Nie dzielimy przez zero!!!";
        }

        else if (Liczba2 == 0)
        {
            cout << "Nie dzielimy przez zero!!!";
        }
        else
        {
        cout << "Iloraz = " << Liczba1 / Liczba2;
        }
    }

    else if (Wybor == 5 )
    {
    return 0;
    }

    else
    {
    cout << "Nie ma takiej opcji!!!" << endl;
    }

    getchar();getchar();

    return 0;
}

 

Answer 1

Witam

według mnie możesz poprawić ten kod wprowadzając następujące ulepszenia

1. Pozbycie się zmiennych globalnych (float Liczba1, Liczba2, Wybor;) wrzucenie je do funkcji main. Więcej: http://www.drzewniak.slupsk.pl/~ks/c/c_024.html
2. Zamiast tylu ifów możesz wykorzystać switch. Więcej http://cpp0x.pl/kursy/Kurs-C++/Poziom-1/Warunek-wielokrotnego-wyboru-switch-case/17
3. Możesz podzielić kod na funkcje i wykorzystać je w main. Na przykład możesz stworzyć funkcje void displayMenu(); która by wyświetlała menu.http://cpp0x.pl/kursy/Kurs-C++/Poziom-2/Funkcje-pierwsze-starcie/291
4. Ewentualnie na przyszłość lepiej by było jakbyś nadawał angielskie nazwy zmiennych/funkcji itp. tak aby kod był bardziej uniwersalny i inni użytkownicy mogli go zrozumieć.

Comments

Dzięki sprobóję to zrobić

---

Ok, jakby coś to wrzucaj na pewno ktoś Ci pomorze.

PS

Nie używaj polskich znaków diakrytycznych ą/ę/ź/ć itp w kodzie. W komentarzach możesz.

cout << "Menu główne" << endl;
//zamień na 
cout << "Menu glowne" << endl;

 

---

Dlaczego? w konsoli wyświetlają sie poprawnie

---

Widocznie masz ustawione na polskie kodowanie w konsoli.

---

codeblocks na linuxie ma tak domyślnie

Answer 2

Usprawnić != krótszy kod. Obecnie tworzone oprogramowanie skupia się na wysokiej czytelności kodu, a w drugiej kolejności na jego długości. Poza tym, w tak małych programach nie ma potrzeby dużego skupiania się na wydajności, ponieważ obecne kompilatory i komputery, na których uruchamiamy programy są wystarczająco wydajne i szybkie.

Answer 3

1. Lepiej formatuj kod - dodawaj wcięcia tam gdzie się je powinno dodawać. Znacznie polepsza to czytelność kodu:

       if(Wybor == 1)
       {
            cout << "Suma = " << Liczba1 + Liczba2;
       }
       else if (Wybor == 2)
       {
          cout << "Różnica = " << Liczba1 - Liczba2;
       }
       else if (Wybor == 3)
       {
           cout << "Iloczyn = " << Liczba1 * Liczba2;
       }
   

2. Wynikiem dzielenia 0 / 5 jest 0. Więc warunek liczba1 == 0 jest błędny. Natomiast gdybyś miał kilka warunków prowadzących do jednego skutku warto by użyć koniunkcji logicznej:

   if (warunek1 && warunek2)
   {
        ...
   }
   

3. CodeBlock ma opcję "zatrzymującą program po jego wykonaniu", getchary i inne takie konstrukcje w kodzie są zbędne. Jakiekolwiek inne IDE też powinno mieć takie opcje.
4. Jeżeli nie napiszesz return 0 program i tak zwróci 0.

Answer 4

[...] czy można go jakoś usprawnić (krótszy kod) [...]

Przepraszam, nie mogłem się powstrzymać ;P

#include <iostream>
using namespace std;

float Liczba1, Liczba2;
unsigned Wybor;
const char *t = (char*)u"潐慤‱楬穣쑢㪙倀摯橡㈠氠捩扺駄:敍畮朠苅돃湷੥ⴭⴭⴭⴭⴭⴭⴭⴭⴭㄊ‮潄慤慷楮੥⸲传敤浪睯湡敩㌊‮湍앯於楮੥⸴䐠楺汥湥敩㔊‮祗摪뫅稠瀠潲牧浡ੵ匀浵⁡‽刀돃볅楮慣㴠 汉捯祺‽一敩搠楺汥浩⁹牰敺⁺敺潲℡!汉牯穡㴠 楎⁥慭琠歡敩灯橣Ⅹ℡\n硜〥栲h";

void (*w[])() = {
    []{cout << &t[147] << Liczba1 + Liczba2;},
    []{cout << &t[156] << Liczba1 - Liczba2;},
    []{cout << &t[168] << Liczba1 * Liczba2;},
    []{Liczba2 == 0 ? cout << &t[175] : cout << &t[208] << Liczba1 / Liczba2;},
    []{exit(0);},
    []{cout << &t[212];}};

int main() {
    std::cout << std::endl;
    cout << &t[ 0], cin >> Liczba1, cout << endl;
    cout << &t[17], cin >> Liczba2, cout << endl;
    cout << &t[34], cin >> Wybor;
    w[min(Wybor-1, 5u)]();
    getchar();getchar();
}

 

Comments

Nie wiedziałem że tak się da. Lecz ja to muszę pierw zrozumieć

---

Ale przeciez kolega jeszcze nie zna programowania obiektowego.

---

Więc trzeba się nauczyć!!

---

To powodzenia!