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

Isso pode ser considerado uma má prática? (Aula 3 - Atividade 10 - Acertando o alvo)

  function dispara(evento) {

        var x = evento.pageX - tela.offsetLeft;
        var y = evento.pageY - tela.offsetTop;
        var acertoX1 = x >= 290;
        var acertoX2 = x <= 310;
        var acertoY1 = y >= 190;
        var acertoY2 = y <= 210;


        if(acertoX1 && acertoX2 && acertoY1 && acertoY2){
            alert("acertou!")
        }
    }

Eu criei quatro variáveis para usá-las no "If". Achei mais legível, porém mas trabalhoso de fazer.

1 resposta
solução!

você poderia criar 2 variáveis apenas, acertouX e acertou Y, acho que ficaria melhor do que 4 variaveis. Mas sim ficou legível de qualquer forma. O importante é você deixar sempre o código o mais legivel possivel para as próximas pessoas, e refatorar quando achar que precisa XD