chelitsy
@chelitsy
все при мне

Насколько правильно и оптимально написан код?

Интересует реализация скрипта именно на JS не jQuery. Хочу понять, на сколько он правильно и оптимально написан?
<body>
    <div>
        <input type="password" id="password-input" placeholder="Введите пароль">
        <label><input type="checkbox" id="check"> Показать пароль</label>
    </div>
    <script>
        const cb = document.querySelector('#check');
        cb.onclick = function (){
            let input = document.getElementById('password-input');
            if (input.getAttribute('type') == 'password') {
            input.setAttribute('type', 'text');
        } else {
            input.setAttribute('type', 'password');
        }}
    </script>
</body>
  • Вопрос задан
  • 175 просмотров
Решения вопроса 1
0xD34F
@0xD34F Куратор тега JavaScript
Сравнивайте, делайте выводы:

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

Войдите, чтобы написать ответ

Войти через центр авторизации
Похожие вопросы