Arrays começam do índice zero (o primeiro índice é o zero, o segundo é 1, etc).
Além disso, para verificar se duas condições são verdadeiras usa-se o operador &&
(and) e não o ||
(or).
Dito isso, sua função seria apenas isso:
function maiorAlto(usuario) {
return usuario[0] >= 18 && usuario[1] >= 170;
}
console.log(maiorAlto([18, 170])); // true
console.log(maiorAlto([25, 210])); // true
console.log(maiorAlto([17, 180])); // false
console.log(maiorAlto([19, 150])); // false
Ou seja, se a idade for maior ou igual a 18 e a altura for maior ou igual a 170, a expressão toda resulta em true
. Se qualquer uma das condições não for verdade (ou a idade é menor que 18, ou a altura for menor que 170, ou ambos), o retorno é false
.
Não faz sentido ter o if (maiorAlto.length)
(e tê-lo duas vezes seguidas faz menos sentido ainda). O atributo length
de uma função indica a quantidade de parâmetros que ela espera receber, não há motivo para usá-lo aqui.
E dentro da função você usa usuario5
, sendo que o parâmetro se chama usuario
, então nunca funcionaria corretamente.
O que talvez devesse ser feito é verificar se o array usuario
tem mesmo 2 elementos e eles são números, mas não parece ser requisito do exercício.
Eu só discordo da outra resposta quanto ao fato da arrow function ser "mais simples". Isso é relativo: para mim, dependendo do caso, fica até mais confuso (e em caso de uma função simples como essa, eu acho até uma complicação desnecessária, sem nenhum ganho de fato). Vale lembrar também que uma arrow function nem sempre será 100% equivalente a uma função declarada com
function
, veja mais sobre isso aqui, aqui e aqui (não é o caso da função acima, mas é importante saber as diferenças, em vez de usar só porque parece "mais simples" ou "mais legal").