Entrar Ainda não tem acesso?
1
resposta

Não sei se está certo

Referente ao curso Lógica de programação: praticando com desafios, no capítulo Projeto Sorteador de números e atividade Mão na massa: proteção na entrada de valores

por DAVI LUCAS DOS SANTOS SILVA
| 4.2k xp | 1 posts

eu coloquei um if na função de sortear para resolver o problema, ficou assim

if (ate <= de) {
        alert("Não é recomendado, que o 'até' seja menor ou igual ao 'de'")
        } else {
    for (let i = 0; i < quantidade; i++) {
        numero = NA(de, ate);
        while (sorteados.includes(numero )) {
            numero = NA(de, ate)
        }

        sorteados.push(numero);
    }
    console.log(sorteados);
    Result.innerHTML =  `<label class="texto__paragrafo">Números sorteados:  ${sorteados}</label>`
    aSB();  }
}
1 resposta
por Lorena Garcia
| 10548.4k xp | 15690 posts
Alura Scuba Team

Oii Davi!

A sua lógica tá quase lá, mas dá pra melhorar um pouquinho para seguir boas práticas e deixar o código mais limpo.

O que você fez certo:

  • Colocou um if para verificar se o valor de até é menor ou igual ao valor de de. Isso é exatamente o que precisava ser feito.
  • Exibiu um alert para avisar o usuário. Ótimo.
  • Usou um else para só continuar com o sorteio se os valores estiverem corretos. Funciona, sim!

O que podemos melhorar:

  1. Parar a função logo no início com return: Isso evita que todo o resto do código fique dentro do else, o que deixa o código mais limpo e mais fácil de ler.

  2. Mensagem mais clara: É legal manter uma linguagem bem direta para o usuário entender rapidamente.

Aqui nessa atividade na parte da Opinião do instrutor tem a versão com o Return.

  • O return encerra a função se os valores estiverem errados, sem precisar colocar tudo no else.
  • Isso evita a necessidade de identar todo o restante do código, deixando mais limpo e legível.

Você está no caminho certo!

Continue firme nos estudos.

Alura Conte com o apoio da comunidade Alura na sua jornada. Abraços e bons estudos!