1

我在 C++ 中使用 switch 语句来创建一个简单的菜单。除了 2 种情况外，一切都有效，字母'U'和'l', 在运行时使用时就像一个cin永远不会被填充的语句。无法输入其他选项，也无法通过输入字符来停止它，实际上必须按 Ctrl-Z 退出。据我所知，语法是正确的，所有其他选项都有效。

编辑：即使完全删除 while 也会导致相同的结果

cin >> command;

while(command!='Q')
{
  switch(command)
  {
     case 'L':
        le.left();
        break;
     case 'R':
        le.right();
        break;
     case 'I':
        cin >> str;
        le.insert(str);
        break;
     case 'D':
        le.erase();
        break;
     case 'C':
        cin >> c;
        le.change(c);
        break;
     case 'U':
        cout << "Applying uppercase";      //doesn't even print to screen
        upperList(le);
        cout << "Uppercase applied" << endl;
        break;
     case 'l':
        lowerList(le);
        break;
     case 'P':
        cout<<le;
        break;
     case 'Q':
        break;
     default:
        cout << "Invalid command.";
  }
  cin >> command;
}

le.插入：

void LineEditor::insert(const string& s)
{
for(int i=0;i<s.size();i++)
  L.insert(it, s.at(i));

itBegin=L.begin();  //iterator to beginning of list, used in <iterator>::distance later
}

upperList 及其 ToUpper 函数：

char ToUpper(char c)
{
c=toupper(c);
return c;
}

void upperList(LineEditor le)
{
char c;

le.setBegin();

while(!le.end())
{
  c=le.at();
  le.apply(*ToUpper, c);
  le.right();
}
}

和 le.end()

bool LineEditor::end() const
{
if(it!=L.end())
  return false;
else
  return true;
}

将其缩小到 LineEditor::setBegin()

void LineEditor::setBegin()
{
while(it!=L.begin())           //supposed to move iterator to beginning of list
  --it;
}
4

1 回答 1

0

您的

cout << "Applying uppercase";

不打印，因为标准输出是行缓冲的。你需要

cout << "Applying uppercase" << flush;

刷新缓冲区。

此外，您的upperListandlowerList函数显然进入了一个无限循环，这意味着内部实现的迭代逻辑LineEditor以某种方式被破坏了。

此外，看起来您将字符串本身LineEditor作为成员保留在对象内L。同时你传递LineEditor到upperList和lowerList 值。这意味着在内部完成的任何更改都不会对调用代码产生任何影响，因为upperList并且将在.lowerListleupperListlowerListLineEditor

于 2013-11-04T03:40:18.377 回答