我在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;
刷新缓冲区。
另外,您的
upperList
和lowerList
函数显然进入了一个无限循环,这意味着在LineEditor
内部实现的迭代逻辑被某种方式破坏了。同样,看起来您将字符串本身保留在
LineEditor
对象中作为L
成员。同时,按值将LineEditor
传递给upperList
和lowerList
。这意味着在upperList
和lowerList
内部所做的任何更改都不会对调用代码中的le
产生任何影响,因为upperList
和lowerList
将对LineEditor
的独立副本起作用。