C ++中的中缀到后缀转换器没有输出

Question

我制作了一个函数infixToPostfix() ，它将中缀表达式转换为后缀表达式。 但是我的程序在运行时不会产生任何输出。 请指出我程序中的错误。

代码：

#include <iostream>
#include <cstring>
#include <stack>
#include <cstdlib>
using namespace std;

int isOperator(char x)
{
    return (x == '-' || x == '+' || x == '/' || x == '*');
}

int precedenceOfOperators(char x)
{
    if (x == '+' || x == '-')
    {
        return 1;
    }
    else if (x == '/' || x == '*')
    {
        return 2;
    }

    return 0;
}

char *infixToPostfix(char infix[])
{
    int i = 0, j = 0;
    stack<char> lobby;
    char *postfix = (char *)malloc((strlen(infix + 1)) * sizeof(char));
    while (infix[i] != '\0')
    {
        if (!isOperator(infix[i]))
        {
            postfix[j] = infix[i];
            i++;
            j++;
        }
        else
        {
            if (!lobby.empty())
            {
                if (precedenceOfOperators(infix[i]) > precedenceOfOperators(lobby.top()))
                {
                    lobby.push(infix[i]);
                }
                else
                {
                    postfix[j] = lobby.top();
                    lobby.pop();
                    j++;
                }
            }
            else if (lobby.empty())
            {
                lobby.push(infix[i]);
            }
        }
    }
    while (!lobby.empty())
    {
        postfix[j] = lobby.top();
        lobby.pop();
        j++;
    }
    return postfix;
}

执行：

int main()
{
    char infix[] = {'a', '-', 'b', '+', 'c', '*', 'd'};
    char *postfix = infixToPostfix(infix);
    for (int j = 0; postfix[j] != '\0'; j++)
    {
        cout << postfix[j];
    }
    return 0;
}

逻辑：

1. 我创建了一个字符指针变量来保存我们的后缀表达式。 现在，我们遍历我们的中缀表达式。 如果我们收到一个操作数，我们将它连接到后缀变量。 否则，如果遇到运算符，我们将继续执行以下步骤：

• 考虑运算符及其相对优先级（'/' 和 '*' 比 '+' 和 '-' 具有更高的优先级）。
• 如果堆栈为空或其最上面的运算符具有较低的相对优先级，则将此运算符-优先级对推入堆栈“大厅”。
• 否则，继续从堆栈“大厅”弹出运算符并将它们连接到后缀表达式，直到最顶层的运算符相对于当前运算符的优先级变得更弱。

2. 如果到达 EOE，则从堆栈“大厅”中弹出每个元素（如果有），并将它们也连接到后缀表达式。

Answer 1

您的代码超过了时间限制，因为它陷入了无限循环。 您尚未更新i variable - 它是循环的 else 部分中的中缀数组的索引，即当infix[i]是运算符时。 即在这部分代码中

else
        {
            if (!lobby.empty())
            {
                if (precedenceOfOperators(infix[i]) > precedenceOfOperators(lobby.top()))
                {
                    lobby.push(infix[i]);
                }
                else
                {
                    postfix[j] = lobby.top();
                    lobby.pop();
                    j++;
                }
            }
            else if (lobby.empty())
            {
                lobby.push(infix[i]);
            }
        }

这是提供完美输出的更新代码。 （我根据自己的方便做了一些小改动，你可以保持代码不变并在 else 部分添加i++ ）

#include <iostream>
#include <cstring>
#include <stack>
#include <cstdlib>
using namespace std;

int isOperator(char x)
{
    return (x == '-' || x == '+' || x == '/' || x == '*');
}

int precedenceOfOperators(char x)
{
    if (x == '+' || x == '-')
    {
        return 1;
    }
    else if (x == '/' || x == '*')
    {
        return 2;
    }

    return 0;
}

string infixToPostfix(char infix[])
{
    int i = 0, j = 0;
    if(sizeof(infix)==0)
    {
        return "";
    }
    int n=sizeof(infix)/sizeof(infix[0]);
    stack<char> lobby;
    string postfix = "";
    while (i < n)
    {
        if (!isOperator(infix[i]))
        {   postfix=postfix+infix[i];
            i++;
            j++;
        }
        else
        {
            if (!lobby.empty())
            {
                if (precedenceOfOperators(infix[i]) > precedenceOfOperators(lobby.top()))
                {
                    lobby.push(infix[i]);
                }
                else
                {
                    postfix = postfix+lobby.top();
                    lobby.pop();
                    j++;
                }
            }
            else if (lobby.empty())
            {
                lobby.push(infix[i]);
            }
            i++;
        }
    }
    while (lobby.empty()==false)
    {
        postfix = postfix+lobby.top();
        lobby.pop();
        j++;
    }
    return postfix;
}
int main()
{
    char infix[] = {'a', '-', 'b', '+', 'c', '*', 'd'};
    string postfix = infixToPostfix(infix);
    for (int j = 0; j < postfix.size() ; j++)
    {
        cout << postfix[j];
    }
    return 0;
}

Answer 2

• 再次浏览我的代码后，我也发现了一些其他逻辑错误。 我找到了一种更好的方法，并重组了函数infixToPostfix()的 while 循环的 else 部分。
• 而不是将中缀表达式视为字符数组，而是将其视为字符串，类似地，后缀表达式也被视为字符串。

更新infixToPostfix() ：

string infixToPostfix(string infix)
{
    int i = 0;
    stack<char> lobby;
    string postfix;
    if (infix.size() == 0)
    {
        return "";
    }
    while (infix[i] != '\0')
    {
        if (!isOperator(infix[i]))
        {
            postfix = postfix + infix[i];
            i++;
        }
        else
        {
            while (!lobby.empty() && precedenceOperator(infix[i]) <= precedenceOperator(lobby.top()))
            {
                postfix += lobby.top();
                lobby.pop();
            }
            lobby.push(infix[i]);
            i++;
        }
    }
    while (!lobby.empty())
    {
        postfix = postfix + lobby.top();
        lobby.pop();
    }
    return postfix;
}

• 正如其他答案中所提到的， i variable也应该在 while 循环的 else 部分中更新。