未执行Javascript true条件

从不执行True条件，而是始终执行false条件。我似乎找不到我到底做错了什么

<script>
$(document).ready(function(){
  var userEmail = $('input#userEmail').val();
  $("#email-submit").click(function(){
    //False condition
    if(!validateEmail(userEmail)) { 
      $('#emailErrors').text("Enter a valid Email");
    }
    //true conidition
    if(validateEmail(userEmail)) { 
      alert("true");
      $.post('includes/loginFunction.php', {userEmail: userEmail}, function (data){
        $('#emailErrors').text(data);
      });
    }
  });
});
function validateEmail($email) {
   var emailReg = /^([\w-\.]+@([\w-]+\.)+[\w-]{2,4})?$/;
   return emailReg.test( $email );
} 
</script>

$（文档）.ready（函数（）{
var userEmail=$（'input#userEmail'）.val（）；
$（“#电子邮件提交”）。单击（函数（）{
//虚假条件
如果（！validateEmail（userEmail））{
$（“#emailErrors”）.text（“输入有效电子邮件”）；
}
//真实条件
if（validateEmail（userEmail））{
警惕（“真实”）；
$.post（'includes/loginFunction.php'，{userEmail:userEmail}，函数（数据）{
$（'#emailErrors'）。文本（数据）；
});
}
});
});
函数validateEmail（$email）{
var emailReg=/^（[\w-\.]+@（[\w-]+\.）+[\w-]{2,4}）？$/；
返回emailReg.test（$email）；
} 

您需要在单击处理程序中检查

输入#用户电子邮件。否则，您只存储了执行
$（document）.ready（）
回调时输入的值

$("#email-submit").click(function(){
  var userEmail = $('input#userEmail').val();

  if (!validateEmail(userEmail)) { 
    $('#emailErrors').text("Enter a valid Email");
  } else {
    // true condition
  }
}

如上所述，您还可以使用
else
而不是调用
validateEmail
两次

此外，在
validateEmail
函数中，您正在声明参数
$email
。通常，在缓存jQuery（或其他库）对象时，将
$
作为变量前缀的惯例是。您可能应该将其切换为简单的
电子邮件
您需要在单击处理程序中检查
输入#用户电子邮件
。否则，您只存储了执行
$（document）.ready（）
回调时输入的值

$("#email-submit").click(function(){
  var userEmail = $('input#userEmail').val();

  if (!validateEmail(userEmail)) { 
    $('#emailErrors').text("Enter a valid Email");
  } else {
    // true condition
  }
}

如上所述，您还可以使用
else
而不是调用
validateEmail
两次

此外，在
validateEmail
函数中，您正在声明参数
$email
。通常，在缓存jQuery（或其他库）对象时，将
$
作为变量前缀的惯例是。您可能应该将其切换为简单的
电子邮件
您需要在单击处理程序中检查
输入#用户电子邮件
。否则，您只存储了执行
$（document）.ready（）
回调时输入的值

$("#email-submit").click(function(){
  var userEmail = $('input#userEmail').val();

  if (!validateEmail(userEmail)) { 
    $('#emailErrors').text("Enter a valid Email");
  } else {
    // true condition
  }
}

如上所述，您还可以使用
else
而不是调用
validateEmail
两次

此外，在
validateEmail
函数中，您正在声明参数
$email
。通常，在缓存jQuery（或其他库）对象时，将
$
作为变量前缀的惯例是。您可能应该将其切换为简单的
电子邮件
您需要在单击处理程序中检查
输入#用户电子邮件
。否则，您只存储了执行
$（document）.ready（）
回调时输入的值

$("#email-submit").click(function(){
  var userEmail = $('input#userEmail').val();

  if (!validateEmail(userEmail)) { 
    $('#emailErrors').text("Enter a valid Email");
  } else {
    // true condition
  }
}

如上所述，您还可以使用
else
而不是调用
validateEmail
两次

此外，在
validateEmail
函数中，您正在声明参数
$email
。通常，在缓存jQuery（或其他库）对象时，将
$
作为变量前缀的惯例是。您可能应该将其切换为简单的
电子邮件




不是问题的根源，而是不应该用2个
if
条件检查电子邮件两次，而应该使用
else
功能
validateEmail
工作，你正在测试的值是什么？不是问题的根源，而是用2个
if
条件检查电子邮件两次，您应该只使用
else
功能
validateEmail
工作，您正在测试的值是什么？不是问题的根源，而是使用
else
功能
validateEmail
工作，而不是使用2
条件检查电子邮件两次，您正在测试的值是什么？不是问题的根源，但是您不应该使用2个
if
条件检查电子邮件两次，而应该使用
else
函数
validateEmail
工作，您正在测试的值是什么？