Refaktoryzacja kodu c++

Question

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";
    }

Answer 1

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]) ... 

Comments

|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 :/

---

A gdzie wkleiłeś tą funkcję? W C++ nie możesz robić funkcji wewnątrz funkcji - funkcja isVowel() musi być po za funkcją main()

---

Dzięki za odpowiedź. Już wszystko działa :)