Kalkulator w C++

0

Witam!
Dopiero co zaczynam programować w C++ i prosiłbym o ocenienie mojego kalkulatora.

#include <iostream>
#include <conio.h>
#include <cstdio>
#include <windows.h>
#include <cstdlib>

using namespace std;

int dzialanie, pierwsza1, druga1, a;
int main(void)
{
    while (1) {

        cout << "<:::>KALKULATOR<:::>" << endl;
        cout << "" << endl;

        cout << "Jakie dzialanie chcesz wykonac ?:" << endl;

        cout << "[1]Mnozenie" << endl, cout << "[2]Odejmowanie" << endl, cout << "[3]Dodawanie" << endl, cout << "[4]Dzielenie" << endl;
        cin >> dzialanie;

        switch (dzialanie) {
        case 1:
            system("cls");
            cout << "Podaj pierwsza liczbe:" << endl;
            cin >> pierwsza1;

            cout << "Podaj druga liczbe:" << endl;
            cin >> druga1;

            a = pierwsza1 * druga1;

            cout << "Wynik:" << a;
            break;

        case 2:
            system("cls");
            cout << "Podaj pierwsza liczbe:" << endl;
            cin >> pierwsza1;

            cout << "Podaj druga liczbe:" << endl;
            cin >> druga1;

            a = pierwsza1 - druga1;

            cout << "Wynik:" << a;
            break;

        case 3:
            system("cls");
            cout << "Podaj pierwsza liczbe:" << endl;
            cin >> pierwsza1;

            cout << "Podaj druga liczbe:" << endl;
            cin >> druga1;

            a = pierwsza1 + druga1;

            cout << "Wynik:" << a;
            break;

        case 4:
            system("cls");
            cout << "Podaj pierwsza liczbe:" << endl;
            cin >> pierwsza1;

            cout << "Podaj druga liczbe:" << endl;
            cin >> druga1;

            a = pierwsza1 / druga1;

            cout << "Wynik:" << a;
            break;
        }

        getch();

        system("cls");
    }

    return 0;
}

Wersja MAC

#include <iostream>
#include <conio.h>
#include <cstdio>
#include <cstdlib>

using namespace std;

int dzialanie, pierwsza1, druga1, a;
int main(void)
{
    while (1) {

        cout << "<:::>KALKULATOR<:::>" << endl;
        cout << "" << endl;

        cout << "Jakie dzialanie chcesz wykonac ?:" << endl;

        cout << "[1]Mnozenie" << endl, cout << "[2]Odejmowanie" << endl, cout << "[3]Dodawanie" << endl, cout << "[4]Dzielenie" << endl;
        cin >> dzialanie;

        switch (dzialanie) {
        case 1:
            system("clear");
            cout << "Podaj pierwsza liczbe:" << endl;
            cin >> pierwsza1;

            cout << "Podaj druga liczbe:" << endl;
            cin >> druga1;

            a = pierwsza1 * druga1;

            cout << "Wynik:" << a;
            break;

        case 2:
            system("clear");
            cout << "Podaj pierwsza liczbe:" << endl;
            cin >> pierwsza1;

            cout << "Podaj druga liczbe:" << endl;
            cin >> druga1;

            a = pierwsza1 - druga1;

            cout << "Wynik:" << a;
            break;

        case 3:
            system("clear");
            cout << "Podaj pierwsza liczbe:" << endl;
            cin >> pierwsza1;

            cout << "Podaj druga liczbe:" << endl;
            cin >> druga1;

            a = pierwsza1 + druga1;

            cout << "Wynik:" << a;
            break;

        case 4:
            system("clear");
            cout << "Podaj pierwsza liczbe:" << endl;
            cin >> pierwsza1;

            cout << "Podaj druga liczbe:" << endl;
            cin >> druga1;

            a = pierwsza1 / druga1;

            cout << "Wynik:" << a;
            break;
        }

        getch();

        system("clear");
    }

    return 0;
}

Link do pliku exe z kalkulatorem:
https://www.dropbox.com/s/nzjuihcx1ngqc7w/Kalkulator.exe?dl=0

Plik jest także w załączniku

1
  1. Fatalne formatowanie kodu. http://format.krzaq.cc
  2. Zmienne globalne (zło)
  3. Dziwna konwencja cout << foo << endl, cout << bar << endl. Użyj średnika jak człowiek.
  4. system("cls") działa tylko na windowsie

Tyle naprędce

1
  1. Sformatuj kod (np. tutaj http://format.krzaq.cc/ albo w IDE)
  2. Powtarzasz jeden fragment kodu kilka razy
cout << "Podaj pierwsza liczbe:" << endl;
cin >> pierwsza1;

cout << "Podaj druga liczbe:" << endl;
cin >> druga1; 
0

Dodałem wersje mac i sformatowałem kod.

0

Wersja mac includuje windows.h? :)

0

Wywal conio i cls-y, użyj przenośnej biblioteki np. ncurses i będziesz miał jedną wersję na wszystkie systemy.

1 użytkowników online, w tym zalogowanych: 0, gości: 1