1

我在验证 javascript 中的表单时遇到了困难。我目前只检查一个文本字段,但它不起作用。我的代码如下:

索引.html:

<html xmlns = "http://www.w3.org/1999/xhtml">
<head> 
    <title>
        Validation Form
    </title>
    <script type = "text/javascript" src ="vForm.js">
    </script>
</head>
<body>
    <form id = "myForm" action ="">
        First name: <input type="text" name="fname"></br>
        Last name: <input type="text" name="lname"></br>
        Password: <input type="password" name="pass1"></br>
        Re-enter password: <input type="password" name="pass2"></br>
        Email: <input type="text" name="email"></br>
        Phone: <input type="text" name="phone"></br>
        Address: <input type="text" name="add"></br>
        Date: <input type="date" name="date"></br>
        Time: <input type="time" name="time"></br>
        <input type="reset" name="reset">
        <input type="submit" name="submit">
    </form>
    <script type = "text/javascript" src ="vFormRun.js">
    </script>
</body>
</html>

vForm.js:

function validateForm()
{
    var fname = document.getElementById("fname");
    var lname = document.getElementById("lname");
    var pass1 = document.getElementById("pass1");
    var pass2 = document.getElementById("pass2");
    var email = document.getElementById("email");

    if(fname == "")
    {
        alert("Please enter first name")
        return false;
    }
    else
    {
        return true;
    }
}

vFormRun.js:

document.getElementById("myForm").onsubmit = validateForm;
4

3 回答 3

3

你需要给.value它每一个。而且,给一个id相同的name

function validateForm()
{
 var fname = document.getElementById("fname");
 var lname = document.getElementById("lname");
 var pass1 = document.getElementById("pass1");
 var pass2 = document.getElementById("pass2");
 var email = document.getElementById("email");

 if(fname.value == "")
 {
   alert("Please enter first name")
   return false;
 }
 else
 {
   return true;
 }
}
于 2012-09-30T17:55:04.767 回答
2
document.getElementById("fname");

只有当你有一个 ID 为 的元素时,这才有效fname,而你没有。

您可以将 ID 属性设置为如下元素:

<input type="text" name="fname" id="fname">

或者,您可以像这样引用表单元素:

var fname = document.forms["myForm"]["fname"]

然后你想value在比较时得到它的属性。

fname.value

标签是自动关闭的<br>,所以它应该<br />代替</br>

于 2012-09-30T17:55:47.203 回答
-1

这是解决方案...

function validateForm(form) {
    var fname = form.fname,
        lname = form.lname,
        pass1 = form.pass1,
        pass2 = form.pass2,
        email = form.email;

    if(fname && fname.value === "") {
        alert("Please enter first name");
        document.getElementById('result').innerHTML = 'Invalid';
        return false;
    }
    document.getElementById('result').innerHTML = 'Passed';
    return true;
}
<form id="myForm" action="" onsubmit="validateForm(this)">
    First name: <input type="text" name="fname"><br/>
    Last name: <input type="text" name="lname"><br/>
    Password: <input type="password" name="pass1"><br/>
    Re-enter password: <input type="password" name="pass2"><br/>
    Email: <input type="text" name="email"><br/>
    Phone: <input type="text" name="phone"><br/>
    Address: <input type="text" name="add"><br/>
    Date: <input type="date" name="date"><br/>
    Time: <input type="time" name="time"><br/>
    <input type="reset" name="reset">
    <input type="submit" name="submit">
  <p id="result">
  </p>
</form>

我在这里纠正了一些问题。

  1. 我将所有var声明更改为使用一个var声明。这是一个最佳实践。
  2. 在 if 语句中,我添加了对变量的检查fname以确保它存在且不为空(防止空引用错误)。
  3. 在 if 语句中,您需要检查字段的value属性,而不是字段本身。在您的旧代码中,如果它是空白的,那么该字段应该在那里并且总是返回true。
  4. 我将比较更改为使用===而不是==. 使用时==,如果值为“false”或0,则返回true。请参阅“ JavaScript 中 == 和 === 之间的区别”。
  5. 您在语句末尾缺少一个分号alert
  6. 如果主体if以 a 结尾,return则不需要else块。减少代码量(下载速度更快)并使其更易于阅读。
于 2012-09-30T18:19:37.773 回答