Solucionado (ver solução)
Solucionado
(ver solução)
1
resposta

[Dúvida] Melhoria do código do primeiro desafio

Na função de alterar status, eu coloquei algo que na minha visão leiga pode ser melhor para legibilidade do código e também usando algumas coisas que aprendamos nas aulas passadas, como "operador ternário"

Invés de usarmos um IF e ELSE para identificar se o elemento possui uma classe para depois remove-la ou adiciona-la, podemos usar uma função do ClassList chamado "toggle", que adiciona a classe caso ela não existe ou remove ela se ela já existir no elemento.

O código final ficou assim

function alterarStatus(id) {
    let gameClicado = document.getElementById(`game-${id}`);
    let imagem = gameClicado.querySelector('.dashboard__item__img');
    let botao = gameClicado.querySelector('.dashboard__item__button');

    botao.classList.toggle('dashboard__item__button--return');
    imagem.classList.toggle('dashboard__item__img--rented')

    botao.innerHTML = botao.classList.contains('dashboard__item__button--return') ? 'Devolver' : 'Alugar';
}

Gostária de opinião de pessoas que entendem melhor do assunto para saber se a minha abordagem foi boa para a situação.

1 resposta
solução!

Oi Guilherme!

Ficou muito boa a solução! Parabéns!

Muito maneiro a utilização da função toggle, que deixou o código bem mais simples. O ternário eu também gosto, mas tem pessoas que acham um pouco mais confuso de entender.

Bons estudos!