Eu escrevi o meu código separando as variáveis imagem e botao em dois ifs diferentes, pois achei mais organizado dessa maneira. Ficou dessa forma:
if (imagem.classList.contains('dashboard__item__img--rented')) {
imagem.classList.remove('dashboard__item__img--rented');
} else {
imagem.classList.add('dashboard__item__img--rented');
}
if (botao.classList.contains('dashboard__item__button--return')) {
botao.classList.remove('dashboard__item__button--return');
botao.textContent = 'Alugar';
} else {
botao.classList.add('dashboard__item__button--return');
botao.textContent = 'Devolver';
}
Porém, como na aula tudo foi feito em um if só, não sei dizer qual seria a melhor prática para o dia a dia. Dois ifs deixaram o código organizado, mas um if só pode deixar o código "mais leve", talvez. O que é o ideal para o mercado de trabalho?