2016-08-14 53 views
1

我试图制作一个失败消息,它根据提交时的错误将不同的字符串加在一起。但是,即使条件满足,失败消息也不会出现。 我想知道我的代码中的错误在哪里。谢谢。失败消息未出现在表单验证事件中

这是在链接到HTML文档的外部JavaScript文件中。

function validationEvent() { 
    var flag = true; 
    var alertmsg = "There were errors found in your registration:"; 
    var givenname = document.getElementById("gname").value; 
    var surname = document.getElementById("sname").value; 
    var address = document.getElementById("address1").value; 
    var city = document.getElementById("city").value; 
    var pcode = document.getElementById("pcode").value; 
    var email = document.getElementById("email").value; 
    var phone = document.getElementById("phone").value; 
    var cnum = document.getElementById("cnum").value; 
    if (givenname == null || givenname == "") { 
     alertmsg += " Given Name is a mandatory field."; 
     flag = false; 

    } 
    if (surname == null || surname == "") { 
     alertmsg += " Surname is a mandatory field."; 
     flag = false; 

    } 
    if (address == null || address == "") { 
     alertmsg += " Address is a mandatory field."; 
     flag = false; 
     } 
    if (city == null || city == "") { 
     alertmsg += " City is a mandatory field."; 
     flag = false; 
     } 
    if (pcode == null || pcode == "") { 
     alertmsg += " Postal Code is a mandatory field."; 
     flag = false; 
     } 
    if (email == null || email == "") { 
     alertmsg += " Email is a mandatory field."; 
     flag = false; 
     } 
    if (phone == null || phone == "") { 
     alertmsg += " Phone Number is a mandatory field."; 
     flag = false; 
     } 
    if (cnum == null || cnum == "") { 
     alertmsg += " Credit Card Number is a mandatory field."; 
     flag = false; 
      } 
    if (!vormCredit()) { 
     alertmsg += " Visa Cards must start with 4 and Mastercard Cards must start with 5."; 
     flag = false; 

    } 
    if (!stateCheck()) { 
     alertmsg += " Invalid Postal Code."; 
     flag = false; 

    } 
    if (!checkCredit()) { 
     alertmsg += " Invaid Credit Card Number."; 
     flag = false; 

    } 
    if (!flag) { 
     alert(alertmsg); 
     return false;} 

    else { 
     alert("Thank you for your subcription."); 
     return true;} 


    }; 

编辑:从'if'循环中删除'return false',并将其仅添加到if(!标志)循环中。

+0

不要在你的if语句中调用'return false'。它将退出该功能并停止它的执行。如果有任何错误,你的代码将永远不会因为'return'语句而达到'!flag'条件。 –

回答

0

这里最好使用switch声明。它处理的情况下,当你有更多的案件。您也可以使用break