Question

我有一个问题是从我之前的一个问题（Conditionally Breaking A Long Sequence Of Inputs?）发展而来的，但到目前为止，还没有人能给我一个满意的答案，而我所有的努力都失败了。

我试图让用户在不需要的情况下尽早进入输入。即，对于以下代码：

cout << '\n' << "Monster A's name is: ";
cin >> nameA;
cout << '\n' << "Monster A rolled: ";
cin >> rollM_A;
cout << '\n' << "Monster A's Dex is: ";
cin >> DexA;
cout << '\n' << "Monster A's Mod is: ";
cin >> ModA;
cout << '\n' << "Monster A's Level is: ";
cin >> LvlA;

//etc.

实际上最多支持12个怪物。如果用户只想使用3-4个，我希望他们能够跳过剩下的部分并为自己节省大量的击键次数。我已经确保将所有变量初始化为0，并且有一个函数可以在以后从存储向量中删除0个元素。所有缺失的都是远离这个输入链。我尝试过各种形式的while循环包装，如下所示：

while(cin.get() != '#') {
   cout << '\n' << "Monster A's name is: ";
   cin >> nameA;
   //etc...
}

但是在输入所需的字符后，代码会一遍又一遍地输出所有提示（“Monster A的名字是：”等），而不会继续或接受进一步的输入。这就像代码卡在无限循环中，即使它应该在#input上留下循环。

有什么想法吗？真的坚持了一段时间，如果有人能提供替代解决方案，或者至少让我知道自己的缺陷，我将非常感激。

谢谢！

Answer 1

你的代码片段引发了无限循环，因为std :: cin的运算符＆gt;＆gt;不采用“行尾”字符（回车键），因此它仍然在流中。

因此，当您第二次到达cin.get（）时，缓冲区中仍然有一个字符（从验证怪物名称时开始输入）。 cin.get（）接受它，看到它不是'＃'，然后转到下一个std :: cin，它也会这样做。您可以通过忽略char：

来解决此问题

while(cin.get() != '#')
{
   cout << '\n' << "Monster A's name is: ";
   cin >> nameA;
   //etc...
   cin.ignore();
}

Answer 2

我刚刚测试了这组代码，它似乎与你想要的一样。当然，您必须对其进行修改以适合您的原始应用程序。

std::string in;

while (true) {
    std::cout << "Enter a name\n";
    std::cin >> in;
    if (in == "#")
        break;
    std::cout << "\nMonster A's name is: " << in << "\n";
}

为了合并怪物数量的限制，而不是将true参数传递给while循环，只需添加一个计数器来创建多少怪物并在该条件下中断：

int num_monsters = 0;
while (num_monsters <= 12) {
    ...
    num_monsters++;
}

希望这有帮助。

Answer 3

面向对象和救援的stdlib：

#include <iostream>
#include <string>
#include <vector>

using namespace std;

class Monster {
  string name;
  int rolled, dex, mod, level;

  template<class input, class output, class container>
    friend input& read_monster(input& i, output& o, container &c) {
      string n;
      o << "\nMonster's name is: " << flush;
      i >> n;
      if( !i || (n == "#") ) {
        o << "\nEnd of monster list";
        i.setstate(ios::failbit);
        return i;
      }
      Monster m;
      m.name = n;
      o << "\nMonster's rolled is " << flush;
      i >> m.rolled;
      if( !i ) return i;
      o << "\nMonster's dex is " << flush;
      i >> m.dex;
      if( !i ) return i;
      o << "\nMonster's mod is " << flush;
      i >> m.mod;
      if( !i ) return i;
      o << "\nMonster's level is " << flush;
      i >> m.level;
      if( !i ) return i;
      o << "\n";
      c.push_back(m);
      return i;
    }

  template<class output>
    friend output& operator<<(output& o, Monster&m) {
      return o << "Monster('" << m.name <<"', R/" << m.rolled
        << ", D/" << m.dex << ", M/" << m.mod << ", L/" << m.level
        << ")\n";
    }

  Monster() : name(), rolled(), dex(), mod(), level() {}
};

int main(int, char**) {
  vector<Monster> v;
  while( read_monster(cin, cout, v) && (v.size() <= 12) )
    ;

  for( auto m: v )
    cout << m;
  return 0;
}

Answer 4

由于你正在进行面向行的输入，你真的应该这样做使用std::getline阅读。并且由于面向行的输入由线块组成，每个线都定义一个对象，你应该创建一个类，并阅读它。（另外，你需要更多错误处理，因为你需要处理错误任何中间线。）

显然，这些问题中的每一个都应该分开一个单独的功能：

std::string
getLineFromPrompt( std::string const& prompt )
{
    std::string results;
    std::cout << prompt;
    std::getline( std::cin, results );
    return results;
}

template <typename T>
bool
tryGetValueFromPrompt( std::string const& prompt, T& value )
{
    std::string line = getLineFromPrompt( prompt );
    std::istringstream parser( line );
    return parser >> value >> std::ws && parser.get() == EOF;
}

template <typename T>
void
getValueFromPrompt( std::string const& prompt, T& value )
{
    while ( ! tryGetValueFromPrompt( prompt, value ) ) {
        std::cout << "Illegal input, try again" << std::endl;
    }
}

class Monster
{
    std::string name;
    int roll;
    int dexterity;
    int mod;
    int level;
public:
    Monster( std::string const& name )
        : name( name )
    {
        getValueFromPromt( name + " rolled:", roll );
        getValueFromPromt( name + " dexterity:", dexterity );
        getValueFromPromt( name + " mod:", mod );
        getValueFromPromt( name + " level:", level );
    }
};

bool
isEndFlag( std::string const& line )
{
    return line.empty() || line[0] == '#';
}

最后，循环：

std::vector <Monster> monsters;
std::string nextName = getLineFromPrompt();
while ( ! isEndFlag( nextName ) ) {
    monsters.push_back( Monster( nextName ) );
    nextName = getLineFromPrompt();
}

正如您所看到的，将每个个人关注因素分解为一个单独的函数可以生成更简单的代码。

Answer 5

你可以宣传这些信息，例如'按＃跳过这个问题'？而不是让用户猜出哪个键可以跳过这个问题？

基于角色，不能早期从用户输入中断？

5 个答案: