O primeiro problema é que você não está retornando nada da função vezesLetraAparece
. Note que há ausência de uma declaração return
no corpo da função e, por causa disso, ela sempre retornará undefined
.
Portanto, colocando um return
para retornar o valor de resultado
após o laço for
terminar, teremos algo assim:
function vezesLetraAparece(frase, letra) {
var resultado = [];
for (var letra = 0; letra < frase.length; letra++) {
resultado.push (frase[letra]);
}
return resultado;
}
console.log(vezesLetraAparece('abcaba', 'a'));
Mas, ao executar o código, percebemos que há um outro erro. O código não está retornando, de fato, o número de vezes que a letra passada no 2º argumento ocorre na string. O for
simplesmente está fazendo a divisão da string em um array, sendo cada elemento um caractere.
Portanto, percebe-se que utilizar um array como resultado (ou acumulador), nesse caso, não faz muito sentido. Você também há de verificar se a letra correspondente à iteração atual corresponde (é igual) àquela que nos interessa. Você não está fazendo essa verificação também.
Enfim, precisamos utilizar um número (o qual iniciaremos como 0
) como contador e uma declaração if
para contar somente as letras que são iguais àquela que buscamos.
Teremos, então:
function vezesLetraAparece(frase, letra) {
var resultado = 0;
for (var indiceLetra = 0; indiceLetra < frase.length; indiceLetra++) {
if (frase[indiceLetra] === letra) {
resultado++; // Somamos 1 ao contador.
}
}
return resultado;
}
console.log(vezesLetraAparece('abcaba', 'a')); // 3
Note também que tive que mudar o nome da variável letra
(no for
) para indiceLetra
. Se o nome continuasse como letra
, estaríamos sobre-escrevendo o parâmetro letra
, declarado na função. Isso se chama variable shadowing e, nesse caso, é indesejável.
Também tinha um ponto e vírgula a mais depois no for
. Como ele impede o o bloco do for
de ser executado, hemos de o remover.
Para simplificar o for
um pouco (e, de quebra, lidar com um eventual problema ligado à "decomposição" de certos caracteres Unicode), você pode utilizar um laço for..of
:
function vezesLetraAparece(frase, letra) {
var resultado = 0;
for (const letraAtual of frase) {
if (letraAtual === letra) {
resultado++;
}
}
return resultado;
}
console.log(vezesLetraAparece('abcaba', 'a')); // 3
Óbvio que existem diversas outras formas de se resolver esse problema. Essas são apenas duas das incontáveis possibilidades. À medida que você for aprendendo mais sobre JavaScript, será capaz de pensar em outras formas de resolvê-lo. São, não obstante e provavelmente, as mais simples. :)