C++不能用键盘输入

Question

如果没有输入“密码”，代码会询问用户的名字。如果找到该名称，则将其名称指定为用户。

代码打击是一个更大的简化版本，但问题是完全一样的。在else语句中，std::cout << "Hello " << FirstName << std::endl;无限重复。如果我删除std::cout << "Hello " << FirstName << std::endl;我无法输入任何内容。就好像我的键盘被拔掉了一样。尝试使用goto，个案，甚至制作一个功能，但问题仍然存在。有任何想法吗？

#include <fstream> 
#include <iostream> 
#include <string> 

int main() 
{ 
std::string User; 
std::string text; 
std::string PassWord; 
std::cin >> PassWord; 
    if(PassWord == "password") 
    { 
     User = "Ryan"; 
    } 
    else if (PassWord != "password") 
    { 
     std::string line; 
     std::string LastName; 
     std::string FirstName; 
     std::string AttemptUser; 
     std::ifstream NameFile; 
     int offset; 

     std::cout << "Name?" << std::endl; 
     NameFile.open("C:\\Filepath\\Names.txt"); 

     if(NameFile.is_open()) 
     { 
      while(!NameFile.eof()) 
      { 
       while(std::cin >> FirstName >> LastName)//<---Needing to press Ctrl-z still a problem! 
       AttemptUser = FirstName + ' ' + LastName; 
       if((offset = line.find(AttemptUser, 0)) != std::string::npos) 
       { 
        NameFile.close(); 
        std::cout << "Hello " << FirstName << std::endl; 
        User = FirstName; 
       } 
      } 
     } 
    } 
    do 
    { 
     std::getline(std::cin, text); 
     if(text.find("the code") != std::string::npos) 
     { 
      std::cout << "Yes " << User << ", this is the code" << std::endl; 
     } 
    } 
while (text != "close"); 
system("pause"); 
return 0; 
} 

Answer 1

好了，这里有几个问题，但SO是所有关于学习和固定，所以：

问题零 将一个

using namespace std; 

之前main，并停止把std::前在std::命名空间中的所有内容。不是功能上的差异，但你应该知道这意味着什么。

问题，第一：

  while(std::cin >> FirstName >> LastName)//<---Needing to press Ctrl-z still a problem! 
      AttemptUser = FirstName + ' ' + LastName; 

这将永远重复，除非你强制退出程序（CTRL + C，CTRL + Z等），您可能意味着包括之后的大括号while(...)。不管接下来，这两条线形成一个完整的块，这意味着该行

AttemptUser = FirstName + ' ' + LastName; 

会被一遍又一遍地执行，直到条件

std:cin >> FirstName >> LastName; 

计算为false，这将不会发生。在C++规范中查看cin。

你可能有什么意思向做的是：

  while(!(std::cin >> FirstName >> LastName))//<---Needing to press Ctrl-z still a problem! 
      AttemptUser = FirstName + ' ' + LastName; 

的区别是微妙的，但很重要。而且，使用字符串作为布尔操作数是愚蠢的。

问题，第二

你永远不会读取文件。它被打开，检查eof，并且最终（理论上它虽然在这个实现中永远不会发生）被关闭，但是你永远不会从文件中读取任何东西。这意味着::eof将总是是假的，假设文件存在并且不是空的。

问题，第三

if(PassWord == "password") 
{ 
    ... 
} 
else if (PassWord != "password") 
{ 
    .... 
} 

好吧，这好像是说

if(I am alive) 
{ 
    call me! 
} 
else if (I am not alive) 
{ 
    send flowers or raid the closets; 
} 

跳过()在else。在（很多）其他事情中是多余的。

问题，第四

块

 while(!NameFile.eof()) 
     { 
      while(std::cin >> FirstName >> LastName)//<---Needing to press Ctrl-z still a problem! 
      AttemptUser = FirstName + ' ' + LastName; 
      if((offset = line.find(AttemptUser, 0)) != std::string::npos) 
      { 
       NameFile.close(); 
       std::cout << "Hello " << FirstName << std::endl; 
       User = FirstName; 
      } 
     } 

将在史诗的方式完全失败，如果offset永远等于line.find(AttemptUser,0)，因为你关闭该文件，然后检查它的eof在下循环，这将始终发生，除非你已经知道如何通过键盘输入什么也没有。这可能很困难。

其他

如果使用Visual Studio，进入Ctrl+A，然后Ctrl+KD。它应该重新格式化您的代码（取决于您的键绑定），并向您展示有关您的块/缩进结构的有趣事情，这可能会让您感到惊讶。

猜测

我要去无路可退，这不能不说：

 while(!NameFile.eof()) 
     { 
      while(std::cin >> FirstName >> LastName)//<---Needing to press Ctrl-z still a problem! 
      AttemptUser = FirstName + ' ' + LastName; 
      if((offset = line.find(AttemptUser, 0)) != std::string::npos) 
      { 
       NameFile.close(); 
       std::cout << "Hello " << FirstName << std::endl; 
       User = FirstName; 
      } 
     } 

真的应该是这样的：

std::cin >> FirstName >> LastName; 

if (FirstName != "" || LastName != "") 
{ 
    AttemptUser = FirstName + " " + LastName; 
    while(!NameFile.eof()) 
    { 
     getline(NameFile,line); //<< !!!!!!!!!!!!!!! READ THE FILE 
     if (line.find(AttemptUser,0) != string::npos) 
     { 
      /// found 
      cout << "Hello " << FirstName << endl; 
      User = FirstName; /// I hope you don't have multiple users with the same first name. 
      break; // <------- REALLY FREAKING IMPORTANT 
     } 
    } 
    cout << "Who are you?\n"; 
} 

我可以继续下去。修正这个问题，意识到VS2012 Express实际上有一个调试器，并回来提出更多问题。

Answer 2

FirstName + LastName = AttemptUser;是不是做你期待什么。 它应该很可能是AttemptUser = FirstName + ' ' + LastName;。

它卡在一个infite循环中，因为AttemptUser永远不会被赋予一个值，这导致我说你应该总是初始化你的变量。声明不带值的变量然后像在这里所做的那样引用它们可能会导致非常意外的结果。只要学习一个习惯，即在声明一个变量时始终给出一个初始值，并且可以为您节省很多麻烦。

当你做了FirstName + LastName = AttemptUser;你实际上将AttemptUser的值设为LastName。

编辑 不得不做一些调试...变量line从未分配。您没有正确地阅读文件或在任何地方使用其数据。使用std::getline(fstream, string)逐行读取文件。 while(std::cin >> FirstName >> LastName)也应该反转，如“如果用户输入为，则有效时再次读取”。