Jetbrains PHPstorm 2019.3.3 предупреждает меня, что у меня есть дублированный селектор jQuery - PullRequest
0 голосов
/ 10 марта 2020

Как я могу исправить это предупреждение о дублировании? Есть ли лучший способ сделать этот процесс для взаимодействия с базой данных и формами переднего плана? Также есть ли какие-то дополнительные улучшения, которые я мог бы добавить к этой работе, чтобы она работала лучше?

$(document).ready(function() {
  function authenticate(mode) {
    if (mode === "Login") {
      let User_email = $("input#log_User_email").val();
      let Password = $("input#log_Password").val();

      let error_collector = "";
      let $log_response = $("div.box_response p#log_response");
      $log_response.html("");

      if (User_email.length <= 0)
        error_collector += "Please enter a user email.";
      if (Password.length <= 0)
        error_collector += "Please enter a user password.";

      if (error_collector !== "") {
        $log_response.html(error_collector);
      } else {
        $log_response.html("Logging in");

        $.ajax({
          method: "POST",
          url: "Resources/PHP/Processes/UserLogin.php",
          data: {
            User_email: User_email,
            Password: Password
          },
          success: function(response) {
            if (response === "Success") {
              window.location = "details.php";
            } else {
              $log_response.html(response);
              $("input#log_Password").val("");
            }
          }
        });
      }
    }

1 Ответ

1 голос
/ 10 марта 2020

Как можно исправить это предупреждение об дублировании

Надеюсь, PHPstorm сказал вам, какой! В этом случае дубликат селектора является $("input#log_Password")

let Password = $("input#log_Password").val();
...
$("input#log_Password").val("");

. Вы можете удалить дублирование, сохранив значение селектора в первом случае в том месте, где вы прочитали значение пароля, а затем повторно его применив. позже, когда вы хотите установить значение, например

const $Password = $("input#log_Password");
let Password = $Password.val();
...
$Password.val("");
Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...