Entrar Ainda não tem acesso?
Solucionado (ver solução)
Solucionado
(ver solução)
1
resposta

Estrutura Código

Referente ao curso Lógica de programação: mergulhe em programação com JavaScript, no capítulo Condicionais e concatenação e atividade Trabalhando com condicionais

por Anderson da Silva Santos
| 2.4k xp | 2 posts

Bom dia voltei a estudar programação através da Alura e no exercício em questão eu montei uma estrutura onde ele usa um 'alert' para mostrar qual a porcentagem de desconto que o cliente está tendo e gostaria apenas de saber se ela esta correta.

let quantidadeMilhas = prompt('Digite aqui a sua quantidade de milhas: ');

let porcentagemDesconto = 0;

if(quantidadeMilhas >= 30000){
    porcentagemDesconto = porcentagemDesconto + 20;
    alert('Seu desconto é de 20%')
}
else{
    if(quantidadeMilhas > 5000){
        quantidadeMilhas = quantidadeMilhas + 10;
        alert('Seu desconto é de 10%')
    }
    else{
        alert('Voce nao possue desconto')
    } 
}

Principalmente na questão de boas praticas onde existe um 'else' dentro de outro 'else' e se nesse caso essa seria a pratica correta do mercado de fazer esse codigo ou existe outro modo mais usado para esse segundo 'else'

1 resposta
solução!
por Victor Costa Santos
| 1922k xp | 3688 posts
Alura Scuba Team Monitor de ForumOps

Olá Anderson!

Que bom que você voltou, fico feliz e saiba que estamos aqui contigo!

Seu código está funcional, o que já representa uma boa evolução. Contudo, conforme avançamos no aprendizado, perceberemos que não basta apenas saber fazer, é crucial entender como funciona. A otimização é importante, e um código que seja tanto funcional quanto otimizado é sempre desejável.

A estrutura de else dentro de else é válida, mas pode ser simplificada para melhorar a clareza.

Com isso em mente, podemos buscar por algo assim:

let quantidadeMilhas = prompt('Digite aqui a sua quantidade de milhas: ');

let porcentagemDesconto = 0;

if (quantidadeMilhas >= 30000) {
    porcentagemDesconto = 20;
    alert('Seu desconto é de 20%');
} else if (quantidadeMilhas > 5000) {
    porcentagemDesconto = 10;
    alert('Seu desconto é de 10%');
} else {
    alert('Você não possui desconto');
}

Nesse código eu usei else if para evitar o aninhamento de else dentro de else, o que pode ser confuso.

E mais um pequeno detalhe é que eu ajustei a atribuição de porcentagemDesconto para ser direta, sem a necessidade de incrementar.

Dessa forma, o código fica mais limpo e fácil de entender. A lógica continua a mesma, mas a legibilidade melhora um pouco, mas pouca coisa mesmo, seu código já estava muito bacana, para esse momento, continue assim!

Reforço que em casos de dúvidas, conte sempre com o fórum da comunidade Alura! Bons estudos!

Sucesso ✨

Um grande abraço e até mais!

Caso este post tenha lhe ajudado, por favor, marcar como solucionado ✓. Bons Estudos!