将项目添加到 C++ 中的列表

Question

我在我的 C++ 应用程序中使用两个类。代码如下：

class MyMessageBox
{
public:
    void sendMessage(Message *msg, User *recvr);
    Message receiveMessage();
    list<Message> dataMessageList;
};

class User
{
public:
    MyMessageBox *dataMsgBox;
};

msg是一个指向Message类的派生类对象的指针。我已经实现了 sendMessage 函数，如下所示：

void MyMessageBox::sendMessage(Message *msg, User *recvr)
{
    Message &msgRef = *msg;
    recvr->dataMsgBox->dataMessageList.push_back(msgRef);
}

当我编译此代码时，出现以下错误： 对“消息的 vtable”的未定义引用。请帮我解决这个问题。

谢谢， 拉凯什。

Answer 1

我不知道你想用 msgRef 做什么，但这是错误的。您曾经是一名 Java 程序员吗？

如果 Message 是 Message 派生类的基类，则需要在列表中存储指针。将 list 更改为 list；并且 push_back(msgRef) 应变为 push_back(msg)，完全删除 msgRef 代码。

另外，从风格上来说，将大量 -> 运算符链接在一起并不是一个好主意。在这种情况下，最好在 User 上实现一个方法，将 Message 添加到它自己的列表中并调用它。

Answer 2

对于初学者来说，如果要在标准 C++ 容器中存储多态对象，则应该存储指向该对象的指针，而不是基类的对象。如果不这样做，就会遇到对象切片问题。另外，帮自己一个忙，将指针包装在智能指针中以防止资源泄漏 - 我建议使用 boost::shared_ptr<>。

鉴于您没有向我们展示 Message 的代码，我们只能猜测问题是什么。由于它引用的是 vtable，因此可能是这样的：

• 您没有将 Message 的任何类成员声明为virtual。从析构函数开始将是一个好主意
• 您忘记链接到包含 Message 编译代码的对象文件

顺便说一句，在 sendMessage() 中创建附加引用是不必要的，恕我直言也不需要完全有助于可读性。只需在调用 push_back() 时取消引用 msg 指针即可。

Answer 3

如果要处理子类消息，则需要使用消息指针列表而不是消息对象，并管理它们的生命周期。为了方便起见，我建议

list< boost::shared_ptr<Message> > datamessageList

使用 boost 库。 （无意冒犯，但您确实需要多阅读一些有关 C++ 和指针的内容：看起来您尝试了代码的各种排列，直到获得编译结果...）

Answer 4

我认为这是一个稍微人为的错误消息，表明您尚未为消息类实现构造函数。请查看此处和这里...

鉴于您尝试将指针传递给对象列表时，编译器可能会抱怨它无法将 Message* 转换为 Message。尝试按照 Kylotan 的建议将列表更改为 Message* 列表。

是编译错误还是链接错误？

Answer 5

由于有些人提出了更好的解决方案：请查看 std::queue 或 std::deque 对消息进行排队。所以现在你有：

std::queue<std::tr1::shared_ptr<Message> > dataMessageQueue;