El problema es que estas declarando dos veces la variable "Vocales", que encima iteran por el string sin ningún sentido. Creo que planteaste mal el algoritmo, así creo que te va a servir y va a quedar muchísimo mas claro. s = "En un lugar de la Mancha" def contador_vocal (s): s = s.lower() cant_vocales = 0 vocales = "aeiou" for vocal in vocales: cant_vocales += s.count(vocal) return (cant_vocales) print (contador_vocal (s))