Насколько правильно и оптимально написан код?
Сравнивайте, делайте выводы:

document.querySelector('#check').addEventListener('change', e => {
  document.querySelector('#password-input').type = e.target.checked
    ? 'text'
    : 'password';
});
Если вам не нужна обязательно глобальная константа, оберните свой код в анонимную функцию.