No te aconsejo que accedas a la cadena por índice (cadena.charAt
). No está bien visto en programación funcional y, en muchos casos, existe alguna alternativa mejor. Por ejemplo, se puede usa s.head
en lugar de s.charAt(0)
, o s.tail
en lugar de s.substring(1)
,
Así mismo, en scala se recomienda no usar el return
para salir de una función, eso es más típico de programación imperativa.
Básicamente, lo que necesitas es crear un "filtro", algo tan simple como:
def vocales(cadena: String): String = cadena.filterNotfilter(esVocal)
Podemos crearla como función recursiva, pasando una función de chequeo como argumento, algo así:
def filtro(s: String, f: (Char) => Boolean): String =
if (s.isEmpty)
""
else if (f(s.head))
s.head + filtro(s.tail, f)
else
filtro(s.tail, f)
Con lo que nos quedaría finalmente:
def vocales(s: String) = filtro(s, esVocal)
La función filtro se puede aplicar a más objetos. Podemos crear una función filtro genérica que sirva para cualquier tipo de secuencia:
def filtro[A](s: Seq[A], f: (A => Boolean)): Seq[A] =
s match {
case (x +: xs) if f(x) => x +: filtro(xs, f)
case (_ +: xs) => filtro(xs, f)
case Nil => Nil
}
Con esta definición, nuestra función vocales
quedaría así:
def vocales(s: String) = filtro(s, esVocal).mkString
Por cierto, la función esVocal
también se podría hacer más simple:
def esVocal(c: Char) = "aeiou" contains c