为什么这个 while 循环在询问名称之前执行两次

Question

我有一个函数可以验证用户的名字和姓氏。但是， while 循环执行了两次，我找不到原因......任何帮助

string getName()
{
     string name = "";
     bool stop = false;
     string::iterator it;

     while(!stop)//name validation loop
     {
           cout << "enter name: ";
           getline(cin, name);

           for(it = name.begin(); it < name.end(); it++)
           {
                if(tolower(*it) > 'z' || tolower(*it) < 'a' && tolower(*it) != ' ') 
                { 
                      cout << "Error, only letters! \n"; 
                      break;
                }
                if(it == name.end()-1) { stop = true; }
           }
           cout << "here\n";
     }
     return name;
}

Answer 1

如果 cin 缓冲区中仍有一些 newspace 字符，则可能会发生这种情况。 std::ws 从流中正确文本之前提取空格，例如换行符、空格、制表符。尝试一下：

string getName()
{
     string name = "";
     bool stop = false;
     string::iterator it;

     while(!stop)//name validation loop
     {
           cout << "enter name: ";
           std::cin >> std::ws;   // <--- drop whitespaces
           getline(cin, name);

           for(it = name.begin(); it < name.end(); it++)
           {
                if(tolower(*it) > 'z' || tolower(*it) < 'a' && tolower(*it) != ' ') 
                { 
                      cout << "Error, only letters! \n"; 
                      break;
                }
                if(it == name.end()-1) { stop = true; }
           }
           cout << "here\n";
     }
     return name;
}

Answer 2

您的循环存在一些问题，这就是我重写它的方式，使其与您的实现尽可能相似：

string getName()
{
   bool stop;
   string name;
   string::iterator it;

   do {
      stop = true;
      cout << "enter name: ";
      getline(cin, name);

      for(it = name.begin(); it < name.end(); it++)
      {
          if( (tolower(*it) > 'z' || tolower(*it) < 'a') && tolower(*it) != ' ' )
          { 
                cout << "Error, only letters! \n"; 
                stop = false;
                break;
          }
      }
      cout << "here\n";

     } while (!stop);

   return name;
}

使用您在这里的内容：

if(tolower(*it) > 'z' | | tolower(*it) < 'a' && tolower(*it) != ' ')

由于 & 具有更高的优先级，因此它相等到：

if(tolower(*it) > 'z' ||(tolower(*it) <'a ' && tolower(*it) != ' '))

这不是你想要的。

编辑：因为' '（空格）是< 'a' 你的代码实际上不会引起问题，但我不确定你是故意这样做的。

我也不喜欢这样： if(it == name.end()-1) { stop = true; }，并且 do...while 循环更适合。

编辑：您可能还想添加对空输入（所有空格）的检查，因为这看起来是一个无效的名称。

编辑（再次）： Loki 是对的，在调用 getName 之前，输入流上必须有垃圾。您可以使用 Mścisław 的解决方案或 cin.ignore()。