Prosty kalkulator - co mogę poprawić, aby był lepszy?

 #include <iostream>
#include <windows.h>

using namespace std;

float dodawanie(float a, float b)
{
    return a + b;
}

float odejmowanie(float a, float b)
{
    return a - b;
}

float mnozenie(float a, float b)
{
    return a * b;
}

float dzielenie(float a, float b)
{
    return (a/b);
}
int main()
{
    int wybor;
    float pierwsza, druga;

    while(wybor != 9)
    {
        cout << "Podaj pierwsza liczbe:";
        cin >> pierwsza;

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

        cout << "Wybierz jedna z opcji:\n"
                "1.Dodawanie\n"
                "2.Odejmowanie\n"
                "3.Mnozenie\n"
                "4.Dzielenie\n";
                "9.Wyjscie\n";

        cout << "Podaj swoj wybor: ";
        cin >> wybor;

        switch(wybor)
        {
        case 1:
            cout << "Wynik dodawania tych dwoch liczb to: " << dodawanie(pierwsza,druga) << endl;
            break;

        case 2:
            cout << "Wynik odejmowania tych dwoch liczb to: " << odejmowanie(pierwsza,druga) << endl;
            break;

        case 3:
            cout << "Wynik mnozenie tych dwoch liczb to: " << mnozenie(pierwsza,druga) << endl;
            break;

        case 4:
            if(druga == 0) cout << "Nie mozna dzielic prez zero !!!\n";
            else
            cout << "Wynik dzielenia tych dwoch liczb to: " << dzielenie(pierwsza,druga) << endl;
            break;
        }
        Sleep(2000);
        system("cls");
    }

    return 0;
}

#include <iostream>
#include <windows.h>

using namespace std;

float dodawanie(float a, float b)
{
    return a + b;
}

float odejmowanie(float a, float b)
{
    return a - b;
}

float mnozenie(float a, float b)
{
    return a * b;
}

float dzielenie(float a, float b)
{
    return (a/b);
}
int main()
{
    int wybor;
    float pierwsza, druga;

    while(true)
    {
        cout << "Wybierz jedna z opcji:\n"
                "1.Dodawanie\n"
                "2.Odejmowanie\n"
                "3.Mnozenie\n"
                "4.Dzielenie\n";
                "9.Wyjscie\n";

        cout << "Podaj swoj wybor: ";
        cin >> wybor;

        if(wybor == 9) break;
        
        cout << "Podaj pierwsza liczbe:";
        cin >> pierwsza;

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

        switch(wybor)
        {
        case 1:
            cout << "Wynik dodawania tych dwoch liczb to: " << dodawanie(pierwsza,druga) << endl;
            break;

        case 2:
            cout << "Wynik odejmowania tych dwoch liczb to: " << odejmowanie(pierwsza,druga) << endl;
            break;

        case 3:
            cout << "Wynik mnozenie tych dwoch liczb to: " << mnozenie(pierwsza,druga) << endl;
            break;

        case 4:
            if(druga == 0) cout << "Nie mozna dzielic prez zero !!!\n";
            else
            cout << "Wynik dzielenia tych dwoch liczb to: " << dzielenie(pierwsza,druga) << endl;
            break;

        default:
            cout << "Nie ma takiej opcji\n";
            break;
        }
        Sleep(2000);
        system("cls");
    }

    return 0;
}
 

cout << "Wybierz jedna z opcji:\n"
                "1.Dodawanie\n"
                "2.Odejmowanie\n"
                "3.Mnozenie\n"
                "4.Dzielenie\n";
                "9.Wyjscie\n";

#include <string>
#include <sstream>

bool isFloat( string myString ) {
    std::istringstream iss(myString);
    float f;
    iss >> noskipws >> f; // noskipws considers leading whitespace invalid
    // Check the entire string was consumed and if either failbit or badbit is set
    return iss.eof() && !iss.fail(); 
}