Question

我正在尝试使用validateForm（）方法来确保电子邮件有效（我已经完成）并确保'name'和'comments'字段不为空。出于某种原因，我不能让第二部分失效，需要一些帮助。这是我目前的代码。

<script type="text/javascript">
<!--
function validateForm() {
    var x = document.forms["myForm"]["email"].value;
    var x = document.forms["myForm"]["name"].value;
    var atpos = x.indexOf("@");
    var dotpos = x.lastIndexOf(".");
    if (atpos < 1 || dotpos < atpos + 2 || dotpos + 2 >= x.length) {
        alert("Not a valid e-mail address");
        return false;
    }
}
}
if (x == null || x == "") {
    alert("Empty Fields");
    return false;
}
}
// -->
</script>

形式：

<form name="myForm" action="http://webdevfoundations.net/scripts/formdemo.asp" onsubmit="return      validateForm()"  method="POST">
Your name: <br>
<input type="text" name="name" ><br>
<br>
Your email: <br>
<input type="text" name="email"><br>
<br>
Your comments: <br>
<textarea name="comments" rows="15" cols="50"></textarea><br><br>
<input type="submit" value="Submit">
</form>

Answer 1

您要将两个表单字段分配给变量x

Answer 2

为什么您对字段name和email使用相同的变量。这没有意义。

您的代码中还有一个语法错误，即您在检查第二个条件之前添加了额外的}。所以它使该功能超出了该功能

function validateForm() {
    var x = document.forms["myForm"]["email"].value;
    var y = document.forms["myForm"]["name"].value;
    var atpos = x.indexOf("@");
    var dotpos = x.lastIndexOf(".");
    if (atpos < 1 || dotpos < atpos + 2 || dotpos + 2 >= x.length) {
        alert("Not a valid e-mail address");
        return false;
    }
    if (x == "" || y == "") {
          alert("Empty Fields");
          return false;
       }
}

所以这里是JS Fiddle Example

Answer 3

你应该这样试试：

    function validateForm(){
        var x=document.forms["myForm"].getElementsByName('email').value;
        var y=document.forms["myForm"].getElementsByName('name').value;
        var atpos=x.indexOf("@");
        var dotpos=x.lastIndexOf(".");
        if (atpos<1 || dotpos<atpos+2 || dotpos+2>=x.length){
            alert("Not a valid e-mail address");
            return false;
        }
        if (y==null || y==""){
           alert("Empty Fields");
           return false;
        }
   }

Answer 4

您应该选择以下几点来记住以上代码

（1）首先应对空字段进行验证，然后再进行电子邮件检查。（2）根据代码，您的空字段逻辑不是validateForm（）的一部分（可能是拼写错误），因此当调用validationForm（）方法时，此逻辑将不会执行（3）你应该在方法validateForm（）的末尾写返回true （4）使用不同的字段检查变量名称

通过以上几点，解决您的问题将会有所帮助。：）

Answer 5

首先，代码中存在语法错误。似乎有两个redunant花括号。然后将两个字段分配给x，只有name字段获取其值。此外，您最好先检查电子邮件字段是否为空，然后对其进行操作。最后，我认为你的电子邮件检查是对的。例如：xxx@a.info。所以你可以试试：

function checkEmail(aEmail){
    var pattern=//w+([-+.']/w+)*@/w+([-.]/w+)*/./w+([-.]/w+)*/; 
    var objExp=new RegExp(pattern);
    if(objExp.test(str)==true){
        return true;
    }else{
        return false;
    }
}

function validateForm() {
    var x = document.forms["myForm"]["email"].value;
    var y = document.forms["myForm"]["name"].value;
    if (x == "" || y == "") {
        alert("Empty Fields");
        return false;
    }
    if (!checkEmail(y)) {
        alert("Not a valid email");
        return false;
    }
}

如何检查以确保表单字段不是空字符串？

5 个答案: