Refaktoryzacja kodu c++

pytanie zadane 16 sierpnia 2016 w C i C++ przez Informatyk# Obywatel (1,060 p.)
edycja 16 sierpnia 2016 przez Patrycjerz

Witajcie. Cały ten program służy do tego, aby wyświetlał liczbę słów zaczynających się od samogłosek, spółgłosek lub od innych. Jak mógłbym ten kod skrócić?

#include <iostream>

using namespace std;
int main()
{
    cout<<"Podaj slowa, pojedyncze q konczy:\n";
    string slowa;
    getline(cin,slowa);
    int i = 0;
    int inne = 0;
    int samogloski = 0;
    int spolgloski = 0;
    if (isalpha(slowa[0]))
    {
        switch (slowa [i])
                {
                    case 'a':
                    case 'e':
                    case 'i':
                    case 'o':
                    case 'u':
                    case 'y':
                        samogloski++;
                        i++;
                    break;
                    default:
                        spolgloski++;
                        i++;
                        break;


                }
    }
    else
        inne++;

   while (slowa[i]!='\0')
    {
        if(slowa[i+1]=='q' && slowa[i]==' ')
            break;
        else if (slowa[i]==' ')
        {
         if (isalpha(slowa[i+1]))
            {
                switch (slowa [i+1])
                {
                    case 'a':
                    case 'e':
                    case 'i':
                    case 'o':
                    case 'u':
                    case 'y':
                        samogloski++;
                        i++;
                    break;
                    default:
                        spolgloski++;
                        i++;
                        break;


                }
            }
        else
            {
                cout<<slowa[i];
                inne++;
                i++;
            }

        }


        i++;


        }
    cout<<spolgloski<<" spolglosek\n";
    cout<<samogloski<<" samoglosek\n";
    cout<<inne<<" innych\n";
    }

1 odpowiedź

odpowiedź 16 sierpnia 2016 przez Porcupine Nałogowiec (31,560 p.)
wybrane 16 sierpnia 2016 przez Informatyk#

Najlepsza

Na pewno dobrym pomysłem będzie zrobienie funkcji:

bool isVowel(char c) {
     return c == 'a' || c == 'e' || c == 'i' || c == 'o' || c == 'u' || c == 'y';
}

I wtedy te wszystkie case'y zastępujesz if'ami w stylu if (isVowel(slowa[i+1]) ...

komentarz 16 sierpnia 2016 przez Informatyk# Obywatel (1,060 p.)

|error: a function-definition is not allowed here before '{' token|
||=== Build failed: 1 error(s), 0 warning(s) (0 minute(s), 0 second(s)) ===|

mam błąd kompilacji :/