我在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.insert:

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;
}

最佳答案

你的

cout << "Applying uppercase";


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

cout << "Applying uppercase" << flush;


刷新缓冲区。

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

同样,看起来您将字符串本身保留在LineEditor对象中作为L成员。同时,按值将LineEditor传递给upperListlowerList。这意味着在upperListlowerList内部所做的任何更改都不会对调用代码中的le产生任何影响,因为upperListlowerList将对LineEditor的独立副本起作用。

10-02 07:12