2013-10-29 48 views
-2

有人可以检查我的JavaScript代码,如果他们是正确的?我无法看到电子邮件警报。我试图点击提交按钮,但名称警报后,电子邮件不起作用。检查我的JavaScript代码

function doValidate() 
{ 
    if (document.appointment.requiredname.value =="") 
    { 
     alert("Please put your name"); 
     document.appointment.requiredname.focus(); 
     return false; 
    } 
    var readmail = document.appointment.requiredemail.value; 
    var checkatsymbol = readmail.indexof("@"); 
    var checkdotsymbol = readmail.lastindexof("."); 
    if (checkatsymbol < 1 || checkdotsymbol+2>=readmail.length) 
    { 
     alert("Please put the correct email address"); 
     document.appointment.requiredemail.focus(); 
     return false; 
    } 
    if (document.appointment.requiredphone.value =="") 
    { 
     alert("Please put your phone"); 
     document.appointment.requiredphone.focus(); 
     return false; 
    } 
    if (document.appointment.requireddate.value =="") 
    { 
     alert("Please put your appointment date as DD/MM/YYYY"); 
     document.appointment.requireddate.focus(); 
     return false; 
    } 
    if (document.appointment.requiredtime.value =="") 
    { 
     alert("Please put your appointment time as HH:MM AM/PM"); 
     document.appointment.requiredtime.focus(); 
     return false; 
    } 
    return (true); 
} 
+2

你是不是让你的Javascript控制台错误?没有'indexof'和'lastindexof'这样的函数。他们是'indexOf'和'lastIndexOf'。 – Barmar

回答

0

你必须删除您if条件return false;为了在函数中执行如下因素代码

+2

这是正确的。 –

+0

@Hanky웃Panky:阅读问题**,但后名称警报,电子邮件不工作** – Nauphal

+0

@Hanky웃Panky::) – Nauphal

0

WIRTE '的indexOf' 不 '的indexOf' 取代:

var checkatsymbol = readmail.indexof("@"); 
var checkdotsymbol = readmail.lastindexof("."); 

有:

var checkatsymbol = readmail.indexOf("@"); 
var checkdotsymbol = readmail.lastindexOf("."); 
0

JavaScript区分大小写。

var checkatsymbol = readmail.indexof("@"); 
var checkdotsymbol = readmail.lastindexof("."); 

应该是:

var checkatsymbol = readmail.indexOf("@"); 
var checkdotsymbol = readmail.lastIndexOf("."); 
+0

我没有改变,但同样的问题, –

+0

这里是链接,其中u可以看到页面 HTTP ://www.99degreesnorth.com/test/ –

+0

我收到一个错误,因为你拼写'lastIndexOf'为'lastindexOf'。 – Barmar

0
var x=document.appointment.requiredemail.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; 
    } 

这个片段来验证电子邮件应该工作!

0

你或许应该前往CodeReview,一个StackExchange论坛