Não vi muito sentido lógico em utilizar a variável chute
na função de limpar o campo. São duas finalidades diferentes, portanto não acho sensato utilizar uma mesma variável para duas coisas distintas. Dependendo do avançar do sistema, isso poderia dar conflito.
A variável chute
é utilizada para armazenar o valor que o usuário digitou no campo. Nessa função de limpar o campo, estamos utilizando a variável para armazenar o próprio campo. Portanto, o sensato é utilizar uma nova variável, e não reciclar uma que está sendo utilizada para outra coisa. No caso, eu utilizaria o nome campoChute
, que penso ser mais adequado.
Mas ainda assim, não vejo a necessidade de utilizar variável alguma nessa função, uma vez que podemos utilizar da seguinte forma:
function limparCampo() {
document.querySelector('input').value = '';
}
Fora isso, como achei interessante utilizar o valor do chute no texto de "maior" ou "menor", melhor não bagunçar essa variável. Ao limpar o campo, pode ser que a pessoa esqueça qual número ela acabou de digitar, então achei bom deixar registrado na frase:
if (chute > numeroSecreto) {
exibirTextoNaTela('p', 'O número secreto é menor do que ' + chute + '.');
} else {
exibirTextoNaTela('p', 'O número secreto é maior do que ' + chute + '.');
}