La riga if con tutte le consonanti è inutile se puoi fare la seguente assunzione sulle tue stringhe: se non è uno spazio e non è una vocale, allora è una consonante. Quindi se sei sicuro che non ci siano numeri, punteggiatura, simboli, ecc. allora ti basta un else. Altrimenti può andare bene così. Magari valuta la possibilità di creare un paio di funzioni isVocale(char c) e isConsonante(char c) per alleggerire il main, ma questa è più che altro una questione di stile.