Refaktoryzacja - zabawa w zmienianie zdan

Question

Witam ! 

Czy moglibyście ocenić mój kod oraz stwierdzić jakie elementy muszą podlegać ponownej refaktoryzacji ?

Z góry dziękuję za pomoc ! :)

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

using namespace std;

string zdanie;
char wybor;

void kasowanie(string z);
void wstawianie(string w);
void zastepowanie(string zas);

int main()
{
    for(;;)
    {


        cout<<"Napisz Twoje zdanie : ";
        getline(cin,zdanie);

        system("cls");

        cout<<"Wybierz operacje :"<<endl<<endl;
        cout<<"1. Kasowanie czesci zdania."<<endl;
        cout<<"2. Wstawianie slowa do zdania."<<endl;
        cout<<"3. Zastepowanie slowa."<<endl;
        cout<<"4. Wyjdz z programu."<<endl;
        wybor=getch();

        switch(wybor)
        {
            case'1':
            {

                kasowanie(zdanie);

            }break;

            case'2':
            {

                wstawianie(zdanie);

            }break;

            case'3':
            {

                zastepowanie(zdanie);

            }break;

            case'4':
            {

                system("cls");
                cout<<"Wybrales zamkniecie programu";
                Sleep(2000);
                exit(0);

            }break;

            default:
            {

                cout<<"Nie ma takiej opcji w menu"<<endl;
                Sleep(2000);

            }
        }

        getchar();getchar();
        system("cls");

    }
    return 0;
}

void kasowanie(string z)
{
    int x,y;

    system("cls");

    cout<<"Twoje zdanie : "<<z<<endl<<endl;
    cout<<endl<<"Podaj liczbe literki od ktorej zaczac kasowanie(wlacznie ze spacjami) : ";
    cin>>x;
    cout<<endl<<"Ile literek skasowac : ";
    cin>>y;

    x-=1;

    z.erase(x,y);

    system("cls");
    cout<<z;
}

void wstawianie(string w)
{
    int x;
    string y;

    system("cls");

    cout<<"Twoje zdanie : "<<w<<endl<<endl;
    cout<<endl<<"Podaj liczbe literki od ktorej zaczac wstawianie(wlacznie ze spacjami) : ";
    cin>>x;
    cout<<endl<<"Jakie slowo wstawic : ";
    cin>>y;

    x-=1;
    y=" "+y;

    w.insert(x,y);

    system("cls");
    cout<<w;
}

void zastepowanie(string zas)
{
    int x,y;
    string b;

    system("cls");

    cout<<"Twoje zdanie : "<<zas<<endl<<endl;
    cout<<"Podaj liczbe literki od ktorej zaczac zastepowanie(wlacznie ze spacja) : "<<endl;
    cin>>x;
    cout<<"Podaj liczbe liter, ktore zastapic : "<<endl;
    cin>>y;
    cout<<"Jakie slowo wstawic : "<<endl;
    cin>>b;

    x-=1;

    zas.replace(x,y,b);

    system("cls");
    cout<<zas;
}

Pozdrawiam ! 

Answer 1

Na pewno możesz rozdzielić pliki main.cpp funkcje.cpp funkcje.h, czy jak tam sobie je nazwiesz.