PROBLEMA
Tu código tiene problemas porque estás usando mal la información obtenida. Además, no te piden la posición de la cadena, sino cuantas veces aparece la misma en el Array.
El método indexOf()
de Array devuelve el índice del elemento en el Array. Pero si el Array no contiene dicho elemento, devuelve -1
.
Por otro lado, la comparación de elementos en javascript se realiza con los siguientes operadores:
>
: devuelve verdadero si el operando de la izquierda es mayor que el de la derecha, de lo contrario devuelve falso
<
: devuelve verdadero si el operando de la izquierda es menor que el de la derecha, de lo contrario devuelve falso.
===
: devuelve verdadero si los operandos son iguales y del mismo tipo.
Tu estás usando:
} else if (buenaIdea = 0){
lo cual es incorrecto, ya que estás asignando el valor 0 a la variable buenaIdea
. El resultado de esta asignación es luego evaluado en el contexto booleano, y dado que el valor de la variable es cero, se considera false
.
let variable;
if(variable = 0) {
console.log('Variable establecida a cero');
}
if(variable = 1) {
console.log('Variable establecida a uno');
}
SOLUCIÓN
Para resolver el problema, debes tener una forma de contar la cantidad de veces que aparece la cadena "buena" en el Array.
Por ejemplo, usando un simple bucle for
y un contador:
let cont = 0;
for (let i = 0; i < array.length; ++i) {
if (array[i] === "buena") {
cont++;
}
}
if (cont === 0) {
return "Fail!";
}
if (cont < 3) {
return "Active!";
}
if (cont >= 3) {
return "Jackpot!";
}
Otra forma podría ser usando una función reductora, que devolverá un valor numérico. Este valor se calcula comparando el valor del elemento actual y dependiendo de si es la cadena "buena" se suma al acumulado una unidad, de lo contrario se suma 0.
Por ejemplo:
const array = ["buena", "mala", "no", "buena"];
const cont = array.reduce((acc, cur) => {
if(cur === "buena") {
return acc + 1;
} else {
return acc;
}
}, 0);
console.log(cont); // 2
.as-console-wrapper {
min-height: 100%;
top: 0;
}
Otra forma de lograr el objetivo sería usando el método filter()
, para filtrar del array inicial sólo aquellos elementos que se corresponden con la cadena "buena", luego basta ver el tamaño del array resultante y devolver el mensaje apropiado. Por ejemplo:
const array = ["buena", "mala", "no", "buena"];
const filtrado = array.filter(element => element === "buena");
console.log(filtrado.length); // 2
.as-console-wrapper {
min-height: 100%;
top: 0;
}
Por lo tanto, una función que realice la tarea que se pide podría verse de la siguiente forma:
function ideas(array) {
const cont = array.reduce((acc, cur) => {
if(cur === "buena") {
return acc + 1;
} else {
return acc;
}
}, 0);
return cont === 0 ? "Fail!" : cont < 3 ? "Active!" : "Jackpot!";
}
console.log(ideas([])); // Fail!
console.log(ideas(['hola', 'palabra'])); // Fail!
console.log(ideas(['hola', 'buena', 'hi'])); // Active!
console.log(ideas(['buena', 'hola', 'buena', 'buena', 'tres'])); // Jackpot!
.as-console-wrapper {
min-height: 100%;
top: 0;
}
Espero que esto te ayude a resolver el problema.
==
o===
. Por otro lado,indexOf
devuelve-1
si no se encuentra el elemento en el Array, no veo que estés capturando ese posible caso.