无法从“void”转换到“令牌” (C++)

Question

编辑：将令牌结构/枚举添加到代码块

我是 C++ 新手，所以如果我错过了一些明显的东西，请原谅我。我正在尝试编写 C++ 版本的调车场算法，但它无法编译，因为它给出了错误：“无法从 'void' 转换为 'Token'（在我标记的行上）。”谁能告诉我为什么会出现这个错误？

typedef enum TokenType { None, Number, Operator, LeftParens, RightParens };

struct Token
{
    enum TokenType type;
    union
    {
        int num;
        char op;
    };
};

list<Token> DoShuntingYard(list<Token> tokenList)
{
    stack<Token> opStack;
    list<Token> output;
    while (!tokenList.empty())
    {
        ****(This Line) Token t = tokenList.pop_front();
        switch (t.type)
        {
        case Number:
            output.push_back(t);
            break;
        case Operator:
            if (!opStack.empty())
            {
                Token op2 = opStack.top();
                if ((IsLeftAssoc(t) && GetOpPrecedence(t) <= GetOpPrecedence(op2)) || (!IsLeftAssoc(t) && GetOpPrecedence(t) < GetOpPrecedence(op2)))
                {
                    output.push_back(opStack.pop());
                }
            }
            break;
        }
    }
}

Answer 1

问题是 pop_front 不返回值。如果要删除第一个元素并读取其值，可以分两步完成：

Token t = tokenList.front();
tokenList.pop_front();

整个 STL 都采用此约定，主要是出于效率原因。通过让 front 返回值而 pop_front 不返回任何内容，您可以根据需要捕获该值，但如果您只想删除该值，则无需创建只需调用 pop_front 即可删除已删除对象的不必要副本。

稍后您将使用此代码遇到类似的错误：

output.push_back(opStack.pop());

要解决此问题，请将其分成两行：

output.push_back(opStack.top());
opStack.pop();

希望这会有所帮助！

Answer 2

它会给你错误，因为 std::list<>::pop_front() 是一个 void 函数。它不返回任何内容。然而，你使用它就像它返回一些东西一样。所以，问题确实是针对您的：您为什么尝试使用 void 函数作为返回值函数？ Token t = tokenList.pop_front() 行是什么意思？

如果您尝试从列表中“弹出”第一个元素，则可能的步骤顺序将包括

Token t = tokenList.front();
tokenList.pop_front();