检查我的JavaScript代码

Check my javascript codes

本文关键字:代码 JavaScript 我的 检查      更新时间:2023-09-26

有人可以检查我的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 );
}

您必须从if条件中删除return false;才能在函数中执行以下代码

wirte 'indexOf' 而不是 'indexof' 替换:

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

跟:

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

Javascript 区分大小写。

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

应该是:

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

你可能应该去CodeReview,一个StackExchange论坛

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;
  }

这个验证电子邮件的代码片段应该可以工作!!