Nie mieszaj operacji na DOm z logiką, zrób funkcję, która przyjmuje wyraz i zwraca liczbę samogłosek. Pobieranie wartości pola i wyświetlanie wyniku powinno odbywać się osobno. Btw, liczenie samogłosek można ładniej rozwiązać:
const samogłoski = ['a', 'e', 'i', 'o', 'u', 'y']
function liczSamogłoski (wyraz) {
return [...wyraz]
.filter(litera => samogłoski.includes(litera))
.length
}
CodePEn: https://codepen.io/caderek/pen/KvEaBZ?editors=0011