Вход через проверку логина и пароля в javascript?

Question

[Rufix]

Можете пожалуйста помочь найти ошибку в этом скрипте? Понимаю, что скорее всего код неграмотен и возможно с грубыми ошибками (2ой день обучения), но сам я к сожалению найти ошибку не смог.

document.getElementById(btn).onclick = function(){
	var login = document.getElementById('login').value;
	var password = document.getElementById('password').value;
	if (login.value == 'admin') && (password.value == '12345') alert ('Добро пожаловать!');
	else alert ('Вход запрещен!');	
}

(На всякий случай поясню суть кода: при вводе в input верного логина и пароля (логин admin и пароль 12345, должно вывести "Добро пожаловать", иначе "Вход запрещен!".)

Answer 1

[Владимир Проскурин]

1) Переменные логин и пароль всегда равны admin и 12345, не зависимо от того, какие поля ввели, т.к. вы используете = а не | Вообще не понятно, зачем вы так пишите.
2) Условный оператор if требует свои скобки. Нужно вот так
if( (login === 'admin') && (pass === '12345'))
вообще, дополнительные скобки не нужны.

Comments

[Rufix]

Если честно не понял:(
Предположил, что ты имел ввиду в этом участке

var login = document.getElementById('login').value = admin;
	var password = document.getElementById('password').value = 12345;

где я в конце написал их значения.
Я попробовал убрать и не помогло. Если проблема не в этом, то можно как-то подробнее?

[Владимир Проскурин]

Rufix, почему вы пишите так?

var login = document.getElementById('login').value = admin;

[Владимир Проскурин]

Rufix, вот такой код будет

document.getElementById(btn).onclick = function() {
   var login = document.getElementById('login').value;
   var password = document.getElementById('password').value;
   if(login === 'admin' && password === '12345') {
      alert('Добро пожаловать');
   } else {
      alert('Вход запрещен');
   }
}

Вы должны понять, почему.

[Rufix]

Владимир Проскурин, Не помогло.
К кнопке и инпутам ничего не надо добавлять, все правильно?

<input id="login" type="text">
<input id="password" type="text">
<button id="btn" type="button">Enter</button>

[Владимир Проскурин]

Rufix, в html все ок. А в js замените это
document.getElementById(btn)
на это
document.getElementById('btn')

[Владимир Проскурин]

Rufix, https://jsfiddle.net/acnu3kj1/1/

[Rufix]

Владимир Проскурин, Заработало, спасибо :) Очень много нюансов в этом языке

Answer 2

[Сергей Соколов]

//document.getElementById(btn).onclick = function(){
document.getElementById("btn").onclick = function(){
  var login = document.getElementById('login').value;
  var password = document.getElementById('password').value;
//  if (login.value == 'admin') && (password.value == '12345') alert ('Добро пожаловать!');
  if (login == 'admin' && password == '12345') alert ('Добро пожаловать!');
  else alert ('Вход запрещен!');	
}

Попробовать

Comments

[Rufix]

Не сработало. Консоль ругается на первую строку.

[Сергей Соколов]

Rufix, "btn" в кавычках. Обновил

[Владимир Проскурин]

Rufix, да, т.к. у вас как я понял нет переменной btn, либо объявите ее
var btn = 'btn';
либо оберните в кавычки
document.getElementById("btn")