0

我在将其发送到数据库之前用于检查电子邮件有效性的代码存在问题。我有一个 javascript 代码，它通过以下方式检查电子邮件：

  1. 查看第一封电子邮件是否是有效的电子邮件地址
  2. 查看两个电子邮件字段是否匹配

以下是表格的相关部分：

<form action="URLGOESHERE" method="post" name="someName" onSubmit="return validation();" id="formId"> 
    <section class="sectionHeader">
        <h2>Contact Information</h2>
        <span class="important">*Required Fields</span>
    </section>
    <section id="contactInfo">
    <fieldset>
        <legend>Contact Information</legend>
        <div id="contact">
            <div class="inputGroupSameLine">
                 <label for="name" class="left"><span class="important">*</span>First Name:</label>
                 <input type="text" name="firstName" class="imp" size="20" maxlength="25" placeholder="John">
            </div>
            <div class="inputGroupSameLine">
                  <label for="name" class="left"><span class="important">*</span>Last Name:</label>         
                  <input type="text" name="lastName" class="imp" size="20" maxlength="25" placeholder="Smith">
            </div>
            <div class="inputGroupSL">
                <span class="left"><label for="org">Company/Group Name:</label></span>
                <input type="text" name="org" size="30" maxlength="50" placeholder="Auburn University">
            </div>
            <div class="inputGroupSameLine">
                <span class="left"><label for="email"><span class="important">*</span>Email:</label></span>
                <input name='email' id='email' class="imp" type='text' size='45' maxlength='45' placeholder="youremail@example.com" />
            </div>
            <div class="inputGroupSameLine">
                <span class="left"><label for="email2"><span class="important">*</span>Re-type Email:</label></span>
                <input name='email2' id='email2' class="imp" type='text' size='45' maxlength='45' placeholder="youremail@example.com"/>
            </div>
            <div id="phoneGroup">
                <span class="left"><span class="important">*</span>Phone #:</span>
                <input name='phone' type='text' class="imp" id="phone" size='12' maxlength='20' placeholder="xxx-xxx-xxxx"  />
            </div>
            <div id="extGroup">
                <span class="left">Ext:</span>
                <input name='ext' type='text' id="ext" size='12' maxlength='6' placeholder="xxxx"  />
            </div>
        </div>
    </fieldset>
</section>

这是我在提交时运行的代码：

var reEMail=/^\w[\w\-\.]+\@\w[\w\-]+(\.[\w\-]+)+$/;

function validation() {
    if (!checkImportant()) {
        alert("Please enter information in all fields marked important.");
        return false;
    }
    if (!checkAttendees())
        return false;
    if (!checkEmail($('#email'),$('#email2')))
        return false;
    return true;
}

function checkImportant()   {
    important = document.getElementsByClassName('imp');
    for (i=0; i < important.length; i++) {
        if($(important[i]).val() == "") {
            $(important[i]).focus();
            return false;
        }
    }
    return true;
}

function checkAttendees() {
    total = 0 + Number($('#numAttend').val());
    parts = 0 + Number($('#8').val()) + 0 + Number($('#12').val()) + 0 + Number($('#16').val()) + 0 + Number($('#19').val()) + 0 + Number($('#26').val()) + 0 + Number($('#26').val()) + 0 + Number($('#55').val());
    count = 0;
    if (total != (parts)) {
        alert('Please check to ensure you have entered the correct number of participants.');
        count++;
        if (count%2 == 0) {
            $('#numAttend').focus();
            return false;   
        }
        else {
            $('#8').focus();
            return false;   
        }
    }
    return true;
}

function checkEmail(email,email2)   {
    if (goodEMail2(email)) {      
        if (email.val() != email2.val()) {
          alert("Please check to make sure your email address is correct in both fields!");
          email2.focus();
          return false;
        } 
        else return true;
    }
    else {
      alert("Please input a valid email address");
      setTimeout(function(){
    $('#email').focus();
    }, 100);
      email.select();
      return false;
    } 
    return false;
}

function goodEMail2(field) {
   return _checkIt2(reEMail, field);
}

function _checkIt2(re, field){
  if (!re.test(field.val())) {
      field.select();
      field.focus();
      return false;
  }
  else return true;
}

正如您在我的 checkEmail 函数的 if else 主语句中看到的那样，我不得不使用 setTimeout 来延迟对电子邮件字段的关注。如果我取出 setTimeout，页面将不会关注元素。但是，如果我取出 setTimeout 并将指定元素更改为第二个电子邮件字段，它就可以工作。

唯一的例外是在 IE10 中（我在 FF、Chrome、Safari 和 IE10 中的测试）。

我真的不想使用这项工作，我不明白我为什么需要这样做。谁能给我一些想法或答案？

谢谢！

编辑：现在似乎无法让我的黑客工作。我不确定它之前在做什么......所以现在 focus() 在那个特定元素上根本不起作用。

4

1 回答 1

0

改变：

function checkImportant()   {
    important = document.getElementsByClassName('imp');
    for (i=0; i < important.length; i++) {
        if($(important[i]).val() == "") {
            $(important[i]).focus();
            return false;
        }
    }
    return true;
}

至：

function checkImportant()   {
    important = document.getElementsByClassName('imp');
    for (i=0; i < important.length; i++) {
        if(important[i].value == "") { // drop the jQuery reference, not needed
            important[i].focus();      // drop the jQuery reference, not needed
            return false;
        }
    }
    return true;
}

在 Chrome 中为我工作

于 2013-09-25T17:29:41.853 回答