c++ simple calculator

Question

#include <iostream>
#include <math.h>
#include <cstdlib>
#include <locale.h>
using namespace std;

void menu()
{
   
    cout << "1-Dodawanie (+)" << endl;
    cout << "2-Odejmnowanie(-) " << endl;
    cout << "3-Mnożenie(*) " << endl;
    cout << "4-Dzielenie(/)" << endl;
    cout << "5-Wyciaganie pierwiastka z liczby x ze stopnia n" << endl;
    cout << "6-Potęgowanie do n-tej potegi" << endl;
    cout << "7 wyjscie z programu bye!" << endl;
}
void dodawanie()
{
    int a, b;
    cout << "\nPodaj a:";
    cin >> a;
    cout <<"Podaj b:";
    cin >> b;
    cout << a << "+" << b << "=" << a + b << endl;
}
void odejmowanie()
{
    int a, b;
    cout << "\nPodaj a:";
    cin >> a;
    cout << "Podaj b:";
    cin >> b;
    cout << a << "-" << b << "=" << a - b << endl;
}
void mnozenie()
{
    int a, b;
    cout << "\nPodaj a:";
    cin >> a;
    cout << "Podaj b:";
    cin >> b;
    cout << a << "*" << b << "=" << a * b << endl;

}
void dzielenie()
{
    float a, b;
    cout << "\nPodaj a:";
    cin >> a;
    cout << "Podaj b:";
    cin >> b;
    if (b != 0)
    {
        cout << a << "/" << b << "=" << a / b << endl;
    }
    else
    {
        cout << "I po co dzielisz przez 0 baranie?" << endl;
    }

}
void pierwiastek()
{
    float a, stopien;
    cout << "\nPodaj a:";
    cin >> a;
    cout << "podaj stopien pierwiastka ";
    cin >> stopien;
    if (a != 0)
    {
        cout << pow(a, 1 / stopien);

    }
    else
    {
        cout << "Po co ci to było?";
    }
 
}
long int potega(long int podstawa, int wykladnik)
{
    int wynik = 1;

    for (int i = 0; i < wykladnik; i++)
        wynik *= podstawa;

    return wynik;
}


int main()
{
    string wyb;
    setlocale(LC_CTYPE, "Polish");
    cout << "--------------" << endl;
    cout << "Kalkulator " << endl;
    cout << "--------------" << endl;

    menu();
    int wybor;
    cout << "Wybierz co chcesz zrobic: ";
    cin >> wybor;
    switch (wybor)
    {
        case 1:
        {
            cout << "wybrales dodawnie";
            dodawanie();
            cout << "\nChcesz powrocic do menu wyboru?(tak/nie)";
            cin >> wyb;
            if (wyb == "tak")
            {
                system("cls");
                main();
            }
            else
            {
                cout << "Żegnaj!" << endl;
                return 0;
            }
            break;
        }
        case 2:
        {
            cout << "wybrales Odejmnowanie";
            odejmowanie();
            cout << "\nChcesz powrocic do menu wyboru?(tak/nie)";
            cin >> wyb;
            if (wyb == "tak")
            {
                system("cls");
                main();
            }
            else
            {
                cout << "Żegnaj!" << endl;
                return 0;
            }
            break;
        }
        case 3:
        {
            cout << "wybrales Mnożenie";
            mnozenie();
            cout << "\nChcesz powrocic do menu wyboru?(tak/nie)";
            cin >> wyb;
            if (wyb == "tak")
            {
                system("cls");
                main();
            }
            else
            {
                cout << "Żegnaj!" << endl;
                return 0;
            }
            break;
        }
        case 4:
        {
            cout << "wybrales Dzielenie";
            dzielenie();
            cout << "\nChcesz powrocic do menu wyboru?(tak/nie)";
            cin >> wyb;
            if (wyb == "tak")
            {
                system("cls");
                main();
            }
            else
            {
                cout << "Żegnaj!" << endl;
                return 0;
            }
            break;
        }
        case 5:
        {
            cout << "Wyciaganie pierwiastka z liczby x ze stopnia n";
            pierwiastek();
            cout << "\nChcesz powrocic do menu wyboru?(tak/nie)";
            cin >> wyb;
            if (wyb == "tak")
            {
                system("cls");
                main();
            }
            else
            {
                cout << "Żegnaj!" << endl;
                return 0;
            }
            break;
        }
        case 6:
        {
            cout << "wybrales potegowanie";
            int a, b;

            cout << "\nWpisz podstawe: ";
            cin >> a;
            cout << "Wpisz wykladnik: ";
            cin >> b;

            cout << potega(a, b) << endl;
            cout << "Chcesz powrocic do menu wyboru?(tak/nie)";
            cin >> wyb;
            if (wyb == "tak")
            {
                system("cls");
                main();
            }
            else
            {
                cout << "Żegnaj!" << endl;
                return 0;
            }
        }
        case 7:
        {
            cout << "bye" << endl;
            return 0;
            break;
        }
    }

}

Co sądzicie? Dość dluga przerwe miałem i duzo zapomnialem :/ Grunt to pisac, pisac i jescze raz pisac

Answer 1

Ten kod:

        cout << "\nChcesz powrocic do menu wyboru?(tak/nie)";
            cin >> wyb;
            if (wyb == "tak")
            {
                system("cls");
                main();
            }
            else
            {
                cout << "Żegnaj!" << endl;
                return 0;
            }
            break;

powtarzasz w każdym case'ie. Nie można dać go raz za switchem?

Do tego wywołujesz main - tego się nie robi. O tego są pętle.

Dlaczego w każdym case'ie masz cout << "wybrales blablabla";? Przenieś to do odpowiedniej funkcji.

Dlaczego w potega nie pytasz się o liczby, a w pozostałych funkcjach już tak?

 

Comments

yy.. BO nie wiedzialem jak to zrobic zeby do wszystkiego odwołwalo sie, pokaz jak to się robi bo serio nie wiem :/ W funkcji potega zrobiłem inaczej po prostu ;)