O código me parece razoável (para alguém que penso estar começando), embora existam alguns problemas:
Você não está pedindo 10 números, mas sim 12
Pode-se observar que, na condição num <= lim
(na qual num
é iniciado como 0
e lim
é definido como 11
), serão feitas, no total, 12 iterações, já que no intervalo inclusivo entre 0 e 11 existem 12 elementos.
Você pode corrigir isso mudando a condição para num < 10
, por exemplo, na qual num
ainda começa como 0
. Note que agora o intervalo é de 0 a 10, exclusivo – que contabilizam 10 elementos no total. Note que, para isso, além de mudar lim
, alterei o operador de menor que ou iguala (<=
) para menor que (<
).
Sobre a mensagem de que não existem números maiores que 50
Analisando esta parte do código:
for (var i = 1; i <= elementos.length; i++) {
if (elementos[i] > 50) {
maior.push(elements[i]);
} else if(elementos[i] < 50) {
console.log("Não existem números maiores que 50!");
}
}
A mensagem "Não existem números maiores que 50!" me traz a ideia de que, do total de números fornecidos pelo usuário, não existe nenhum maior que 50. No entanto, essa mensagem é impressa para cada número fornecido pelo usuário que é menor que 50 – o que me parece um erro.
Cadê o do while
?
O enunciado é claro e pede que o laço do while
seja usado, mas... Cadê?! :P Na realidade, esse laço é (raramente) necessário e pode sempre ser substituído por um while
normal (como você fez), mas se está no enunciado, a instrução deveria, em tese, ser seguida.
Enfim, corrigindo esses pontos, teríamos algo como:
const elementos = [];
do {
elementos.push(parseInt(prompt('Digite um número inteiro:')));
} while (elementos.length < 10); // Enquanto o comprimento da lista for menor que 10, execute o bloco `do`.
const maiores = [];
for (let i = 0; i < elementos.length; i++) {
const atual = elementos[i];
if (atual > 50) {
maiores.push(atual);
}
}
if (maiores.length > 0) {
console.log('Existem ' + maiores.length + ' números que são maiores que 50!');
} else {
// Note que só exibiremos esta mensagem caso nenhum número fornecido ser maior que 50.
console.log('Não existem números maiores que 50!');
}
Não é um erro, mas como arrays em JavaScript têm a propriedade length
(que retorna o comprimento do array), não é necessário manter um contador "manual" para saber quantos elementos foram inseridos. Só vá inserindo até que a quantidade de elementos satisfaça alguma condição de parada.
Note que nenhum tipo de validação foi feita para confirmar se o valor fornecido pelo usuário foi, de fato, um número. Se isso for necessário, você pode utilizar funções como Number.isNaN
aplicadas ao resultado do parseInt
.
O código acima é bem bruto, embora ideal para quem ainda está aprendendo. Futuramente talvez você aprenda sobre Array.prototype.filter
, que pode simplificar o código ainda mais (principalmente tornando o segundo laço, o for
, implícito).
let maior = elementos.filter(e=> e > 50)
?