如果字段为空，jquery会阻止表单提交？

Question

我使用以下代码尝试更改输入框的边框颜色，并阻止表单提交如果字段为空。

虽然这改变了边框颜色，表格仍然提交，有人可以告诉我我在哪里出错了，在此先感谢

<div class="home_column">
    <div class="login_form">
        <form name="login" id="login" action="include/validate_login.php" method="post" onsubmit="return validateForm()"> 
            <h21>Username</h21><br/>
            <input type="text" class="login_form_front" id="myusername" name="myusername"><br/>

            <h21>Password</h21><br/>
            <input type="password" class="login_form_front" autocomplete="off"  id="mypassword" name="mypassword"><br/>

            <input type="submit" class="buttons_login" value="Login" id="login" name="login">
        </form> 
    </div>
</div>

function validateForm() {
    var a = document.forms["login"]["myusername"].value;
    var b = document.forms["login"]["mypassword"].value;
    if (a == null || a == "" || b == null || b == "") {
        if (a == null || a == "") { 
            document.forms["login"]["myusername"].style.borderColor = "#963634"; 
        }
        if (b == null || b == "") { 
            document.forms["login"]["mypassword"].style.borderColor = "#963634"; 
        }

        $(".form_error").show();
        $('html,body').animate({
            scrollTop: $(".form_error").offset().top - 180 
        });

        return false;
    }
}

Answer 1

这一行有错误：

scrollTop: $(".form_error").offset().top - 180 

未捕获的TypeError：无法读取未定义的属性“top”
那是因为你提供的html代码中不存在form_error。

这会使代码在return false行之前中断并通过错误。
其他一切都还可以

Answer 2

将事件对象传递给事件处理程序并使用event.preventDefault而不是return false;

onsubmit="validateForm(event);"> 

function validateForm(event) {
  // your code
   if(conditionToStopSubmit) 
       event.preventDefault();
}

Answer 3

你有一个奇怪的混合jQuery和普通的JS代码。 试试这个，将事件附加到jQuery中的表单提交，并使用preventDefault在输入为空时停止发出请求：

<form name="login" id="login" action="include/validate_login.php" method="post">
    <!-- your HTML -->
</form> 

$('#login').submit(function(e) {
    var formValid = true;
    var $username = $('#myusername');
    var $password = $('#mypassword');

    if ($username.val() == '') {
       formValid = false;
       $username.css('border-color', '#963634');
    }
    if ($password.val() == '') {
       formValid = false;
       $password.css('border-color', '#963634');
    }

    if (!formValid) {
        e.preventDefault();
        $(".form_error").show();
        $(document).animate({
            scrollTop: $(".form_error").offset().top - 180 
        });
    }
});

Answer 4

尝试在输入文本中添加所需

例

<input type="text" class="login_form_front" id="myusername" name="myusername" required>

Answer 5

既然你正在使用jQuery，为什么不在jQuery中做所有事情呢？ 看看我下面的脏版本：

(function($) {

  $("#login").on("submit", validate);

  function validate(e) {
    var $form = $("#login"),
        $uname = $form.find("input[name='myusername']"), // or $form.find("input.login_form_front") or $form.find("#myusername")
        $pwd   = $form.find("input[name='mypassword']"),
        unameVal = $.trim($uname.val()) || "", // default ""
        pwdVal = $pwd.val() || "", // no trimming as pwd could be spaces
        isValid = true;
    if (!unameVal) {
        $uname.css({ borderColor: 'red' });
        isValid = false;
    }
    if (!pwdVal.length) {
        $pwd.css({ borderColor: 'red' });
         isValid = false;
    }
    if (!isValid) {
        e.preventDefault();
    }
  }
}(jQuery));

现场演示： 验证表单提交

Answer 6

 <div class="home_column"> <div class="login_form"> <form name="login" id="login" action="include/validate_login.php" method="post" onsubmit="return validateForm()"> <h21>Username</h21><br/> <input type="text" class="login_form_front" id="myusername" name="myusername" required><br/> <h21>Password</h21><br/> <input type="password" class="login_form_front" autocomplete="off" id="mypassword" name="mypassword" required> <br/> <input type="submit" class="buttons_login" value="Login" id="login" name="login"> </form> </div> </div> 

尝试此代码，并确保您的浏览器是最新的。

在这里，我刚刚添加了一个足够的关键字“required”。