1

Все было бы прекрасно, но проблема с последним else if. Он с каждым объектом выдает alert независимо значение true или false. А мне нужно чтобы выдал один раз когда false.

$(document).ready(function() {
  var users = [{
    "name": "A",
    "password": "1"
  }, {
    "name": "B",
    "password": "2"
  }, {
    "name": "C",
    "password": "3"
  }, {
    "name": "D",
    "password": "4"
  }];

  $(document).on("click", ".check-data", function() {
    var CheckName = $('.check-name').val();
    var CheckPassword = $('.check-pass').val();

    for (var i = 0; i < users.length; i++) {
      var DataName = users[i].name;
      var DataPassword = users[i].password;

      if (CheckName === DataName && CheckPassword === DataPassword) {
        alert('Connected');
        break;
      } else if (CheckName === "" && CheckPassword === "") {
        alert('Required to enter a name and password');
        break;
      } else if (CheckName === DataName && CheckPassword === "") {
        alert('Required to enter a password');
        break;
      } else if (CheckName === DataName && CheckPassword !== DataPassword) {
        alert('Wrong password');
        break;
      } else if (CheckName !== DataName) {
        alert('Name not found');
      }
    }
  });
});
<script src="https://ajax.googleapis.com/ajax/libs/jquery/2.1.1/jquery.min.js"></script>

<form>
  <a>Check Name</a>
  <input class="check-name"></input>
  <br>
  <a>Check Password</a>
  <input class="check-pass" type="password"></input>
  <br>
  <button class="check-data">Check Data in object</button>
</form>

2 ответа 2

0

Вы же понимаете, что такой способ авторизации бесполезен на сайте? Любой может открыть исходный код странице и узнать пароль! Проверка пароля должна производится сервером!

Но, давайте представим, что этот пример просто для обучения JS.

Вы циклом перебираете значения и производите проверки, которые должны быть на разных этапах?

Во-первых, не нужно проверять пустое ли поле, с каждым юзером, проверьте значения перед циклом:

 if (CheckName === "" ) {
        alert('Required to enter a name');
        return;;
 } 
 if (CheckPassword === "") {
        alert('Required to enter a password');
        return;;
 }

Во-вторых, перебирая имена, не надо каждый раз выводить ошибку. Создайте переменную, которая будет фиксировать найден ли пользователь:

 var namefound = false;
 for (var i = 0; i < users.length; i++) {
       if (CheckName === DataName) {
            namefound = true;
       }
 ...

И если совпадения не произошло, выведите ошибку в конце:

 if (!namefound) {
      alert('Name not found');
      return;
 }

$(document).ready(function() {
  var users = [{
    "name": "A",
    "password": "1"
  }, {
    "name": "B",
    "password": "2"
  }, {
    "name": "C",
    "password": "3"
  }, {
    "name": "D",
    "password": "4"
  }];
 
  $(document).on("click", ".check-data", function() {
    var namefound = false;
    var CheckName = $('.check-name').val();
    var CheckPassword = $('.check-pass').val();


    if (CheckName === "") {
      alert('Required to enter a name');
      return;
    } else if (CheckPassword === "") {
      alert('Required to enter a password');
      return;
    }

    for (var i = 0; i < users.length; i++) {
      var DataName = users[i].name;
      var DataPassword = users[i].password;
      if (CheckName === DataName) {
        namefound = true;
      }
      if (CheckName === DataName && CheckPassword === DataPassword) {
        alert('Connected');
        Connected = true;
        break;
      } else if (CheckName === DataName && CheckPassword !== DataPassword) {
        alert('Wrong password');
        return;
      }

    }
    if (!namefound) {
      alert('Name not found');
      return;
    }
  });
});
<script src="https://ajax.googleapis.com/ajax/libs/jquery/2.1.1/jquery.min.js"></script>

<form>
  <a>Check Name</a>
  <input class="check-name"></input>
  <br>
  <a>Check Password</a>
  <input class="check-pass" type="password"></input>
  <br>
  <button class="check-data">Check Data in object</button>
</form>

7
  • exit? функция которая должна запускаться в конце?
    – DimenSi
    4 фев 2017 в 20:58
  • @DimenSi я не знаю, что планирует автор вопроса делать после всей проверки, потому на всякий случай exit
    – Crantisz
    4 фев 2017 в 21:03
  • Ну поставьте exit(), а то я например не мог понять существует такой оператор или это функция :) А у меня 2-3 месяца опыта в js
    – DimenSi
    4 фев 2017 в 21:05
  • Действительно, спутал с php. Странно что работает все. Заменил на return;
    – Crantisz
    4 фев 2017 в 21:11
  • Большое спасибо за советы!
    – Nerijus
    4 фев 2017 в 21:37
0

Вы break забыли написать, из-за чего цикл не прерывается.

Имхо, называйте переменные по другому. CheckName вводит заблуждение, кажется, что это функция которая проверяет имя. Можете например testedName или просто name.

Ваш ответ

Нажимая на кнопку «Отправить ответ», вы соглашаетесь с нашими пользовательским соглашением, политикой конфиденциальности и политикой о куки

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.