电子邮件验证程序中的布尔变量问题[重复]

Question

我正在尝试在不使用正则表达式库的情况下创建电子邮件验证程序。在我的一个函数中，我想返回一个布尔值来检查电子邮件地址中是否有 @ 符号以及它是否处于有效位置（@ 符号不能是字符串的前三个字符之一）。但是我遇到了问题，因为每次我通过输入带有@符号无效位置的电子邮件地址来运行程序时，它一直告诉我该电子邮件是有效的。请帮忙！

valid = checkEmail(email); //function call 

if(valid == true)
  cout << "Your email is valid!" << endl;

else
  cout << "Your email is invalid!" << endl;


bool checkEmail(string email)
{
  int counter;
  int length;
  bool firstThree; //checks to make sure @ is not in the first three chars

  counter = 0;
  length = email.length();
  firstThree = false;

  for(int i = 0; i < length; i++)
  {
    if(email[i] == '@')
      counter++;
  }

 for(int y = 0; y < length; y++)
 {
   if(email[0] == '@' || email[1] == '@' || email[2] == '@')
      firstThree = true;

   else
       firstThree = false;

 }

 cout << "\n" << counter << endl; //check to see if counter works

 if(counter != 1 && firstThree == true)
   return false;

 else
   return true;
}

Answer 1

我猜你需要在程序开始时定义函数bool checkEmail(string email)。基本上翻转 if else 和函数定义。

bool checkEmail(string email)
{
  int counter;
  int length;
  bool firstThree; //checks to make sure @ is not in the first three chars

  counter = 0;
  length = email.length();
  firstThree = false;

  for(int i = 0; i < length; i++)
  {
    if(email[i] == '@')
      counter++;
  }

 for(int y = 0; y < length; y++)
 {
   if(email[0] == '@' || email[1] == '@' || email[2] == '@')
      firstThree = true;

   else
       firstThree = false;

 }

valid = checkEmail(email); //function call 

if(valid == true)
  cout << "Your email is valid!" << endl;

else
  cout << "Your email is invalid!" << endl;

Answer 2

这里似乎有很多初学者的错误，所以我建议先学习编程和 c++ 的基础知识，就像其他人建议的那样，使用调试器。以下是一些错误：

函数定义

c++ 与其他通用语言不同，因为函数需要先定义才能使用。这意味着您要么需要将 checkEmail 函数移到函数调用之上，要么在函数调用之上创建一个单独的定义，例如：

bool checkemail(string email);

作为一个例子。

if 语句逻辑不正确；不必要的 for 循环：

我假设根据电子邮件的格式，如果 checkEmail 函数与正确的格式不匹配，则您希望它返回 false，并且根据您的函数当前所做的，这意味着它将返回如果前三个字符是 @ 或 如果电子邮件中不完全有一个 @ 符号，则返回 false。但是，您使用了&& 运算符，它表示and，这意味着即使您不希望它返回true（就像@Yastanub 在他的第一条评论中所说的那样）。更好的是，使用std::string::find 和while 循环和向量（类似于this method）可以大大简化整个逻辑和if 语句：

vector<size_t> posVec;
size_t pos = email.find('@', 0); //note that this can be an int instead of size_t, but it can cause stack overflow with bigger numbers
while(pos != string::npos){
    posVec.push_back(pos);
    pos = email.find('@', pos+1);
}
switch(posVec.size()){
    //if there is only one @ symbol found
    case 1:
        bool firstThree = false;
        for(int i = 0; i <= 2; i++)
            //if there is only one @ but it's in the first 3 positions, the email isn't valid
            if(posVec[0] == i)
                firstThree = true;
        return !firstThree;
    //otherwise the email doesn't work
    default:
        return false;
}

请记住包含所需的库以使这部分工作：

#include <string>
#include <vector>

这也消除了函数中第二个无用的 for 循环，因为变量 y 没有被使用，而 counter 正被用于测试

另一个说明

也没有必要使用if (valid == true)。您可以只使用 if (valid)，因为布尔值如何与 if 语句一起使用。