Prevenir a entrada de tarefas vazias + 1 dúvida

const criarTarefa = (evento) => {
    evento.preventDefault()

    const lista = document.querySelector('[data-list]')
    const input = document.querySelector('[data-form-input]')

    const tarefa = document.createElement('li')
    tarefa.classList.add('task')

    //Prevenir a entrada de tarefas vazias
    if(input.value != "")
    {
        const conteudo = `<p class="content">${input.value}</p>`
        tarefa.innerHTML = conteudo
        tarefa.appendChild(BotaoConclui())

        lista.appendChild(tarefa)
    }

    input.value = ""
}

Existe alguma desvantagem ou má prática ao fazer 'input.value' ao invés de usar uma variável auxiliar apenas pra atribuir isso? (Como foi feito com valor = input.value)