[java]vocali e array

[el92]

Ciao sto svolgendo un esercizio che non riesco a risolvere:
trova le vocali in una stringa;mi consiglia poi di usare il metdo boolean isVocale(char c), che verifica se c è una vocale e sta proprio qui il dilemma.
L'esercizio non dice se usare un array semplice o un ArrayList.Io l'ho risolto con un ArrayList e viene tutto.Poi per curiosità ho provato a risolverlo con un array normale ma non riesco a risolverlo:

import java.util.*;
class Vocale {

public char []vocali={'a','e','i','o','u'};
 //verifico se c è una vocale
public boolean isVocale(char c){ 
    for(int pos=0;pos<vocali.length;pos++)
    if(vocali[pos]==c)
    return true;
        
}
}

il compilatore mi dà errore perchè non trova il ritorno del metodo.Qualcuno ha ualche suggerimento?Aiutooo mi sto prdendo in un bicchiere d'acqua

[LeleFT]

Ed ha ragione.
Tu ritorni true se e solo se la variabile passata è contenuta nell'array... ma non ritorni mai false nel caso non fosse una vocale. O meglio, nel caso in cui la lettera non sia una vocale non fai nessun return, ed il compilatore ovviamente si arrabbia.
Per questo tipo di problemi bisognerebbe sempre usare l'approccio single function exit point, ovvero un solo punto in cui la funzione restituisce il risultato. Ed il risultato si ha sempre e solo alla fine:

public boolean isVocale(char c){ 
    // Dichiaro subito la variabile che conterrà il valore di ritorno
    boolean ret = false;

    // Eseguo l'elaborazione
    for(int pos=0;pos<vocali.length;pos++) {
       if(vocali[pos]==c) {
          ret = true;
       }
    }
     
    // Alla fine, e solo alla fine, ritorno il valore (e lo ritorno SEMPRE)
    return ret;  
}

E' un approccio meno efficiente (per renderlo più efficiente è sufficiente mettere un break nel ciclo for dopo aver assegnato true alla variabile ret), ma più manutenibile e sicuramente più chiaro da capire.

Ciao.