C++ 从 cin 读取输入，直到整行都被读取

Question

我在一个字符'c'中读取了一个while循环，我检查了这个字符是否是一个运算符； “+”、“-”、“/”或“*”。 while 循环逐个字符地读取，但它不会停止……例如，如果输入的最后一个字符是“+”。它将永远留在循环中，将 char 'c' 设置为 '+'。

stack<int> num;
char c;
int n,count=0,a,b;
while (cin>>c)
{
    if (c != '+' && c != '-' && c != '/' && c != '*')
     {
         cout << c << endl;
         n = (c - 48);
         num.push(n);
         cin >> c;
         count++;
     }
     else if (c == '+' || c == '-' || c == '/' || c == '*')
     {
         cout << "count is " << count << endl;
         if (count>1)
         {
            b =  num.top();
            a = num.top();
            num.pop();
            num.pop();

            if (c == '+')
            {
                num.push(a+b);
                count--;
            }
            else if (c == '-')
            {
                num.push(a+b);
                count--;
            }
            else if (c == '/')
            {
                if (b != 0)
                {
                    num.push(a/b);
                    count--;
                }
                else
                {
                    cout << "division by zero" << endl;
                    return(0);
                }
            }
            else if (c == '*')
            {
                num.push(a*b);
                count--;
            }
            else
            {
                cout << "invalid input" << endl;
                return(0);          
            }
         }
         else
         {
             cout << "stack underflow" << c << endl;
             return(0);
         }
     }
     cout << c << endl;
 }
}

Answer 1

既然您已经澄清了您的问题，那就更清楚了。 @TeoZec 的答案应该是正确的。我只想注意您上面的代码中看起来有问题的两件事：

else if (c == '-')
{
    num.push(a+b);
    count--;
}

这里你可能想要a-b。

if (count>1)
{
    b =  num.top();
    a = num.top();
    num.pop();
    num.pop();

b 和 a 在这里将是相同的号码，您应该在获得第二个号码之前拨打pop()，例如：

if (count>1)
{
    b =  num.top();
    num.pop();
    a = num.top();
    num.pop();

Answer 2

因为您没有条件终止您的 while 循环，您需要向您的 while 添加更多条件，如下所示：

while(cin>>c && c != '+' && c != '-' && c != '*' && c != '/')
{
    //do stuff here
}

或者在执行检查以查看输入的字符之后，将 break 添加到 while 循环的内部。

Answer 3

您的代码存在几个问题。对于初学者， 我不明白if 中cin >> c 的用途 分支;您输入了一个字符，但您似乎从未使用过它； 它只是丢失了。当然，不需要if 加入else if，因为它是 if 的条件。当然，switch 远不止这些 自然你if/else if链比较单的时候 字符对许多不同的常数。和的 当然，'0' 并不总是 48（即使是，'0' 也是 比48更具可读性）。

至于将字符 c 设置为 + 时保持循环，我不知道 看见。我看到的唯一循环是while( std::cin >> c )，并且 这将读取一个新字符，或者终止。

如果您的目标是只阅读一行，那可能会更好 使用std::getline，然后遍历您输入的字符串：

std::string line;
while ( std::getline( std::cin, line ) ) {
    //  Set up your stack here...
    for ( std::string::const_iterator it = line.begin(); it != line.end(); ++ it ) {
        if ( isdigit( static_cast<unsigned char>( *it ) ) ) {
            //  process digit...
        } else if ( ispunct( static_cast<unsigned char>( *it ) ) ) {
            //  process punctuation, with eventually
            switch ( *it ) {
            case '+':
                //  addition...
                break;
            case '-':
                //  substraction...
                break;
            //  ...
            default:
                //  illegal operator...
                break;
            }
        } else {
            //  process anything else...
        }

    }
}

或者，您可以定义一个isOperator 函数，以及 行：

bool
isOperator( char ch )
{
    static std::string const legalOps( "+-*/" );
    return std::find( legalOps.begin(), legalOps.end(), ch) != legalOps.end();
}

Answer 4

这应该可行：

#include <cstdio>
...

int c; // As pointed out by James Kanze in his comment, c should be an int to detect EOF
do {
    c = getchar();
    // Do stuff
} while ((c != '\n') && c != EOF))

您不仅需要检查 EOF，还需要检查行终止符。