公共成员未初始化

Question

我正在制作一个客户端服务器应用程序，服务器有一个 GUI。我正在使用 Qt。 对于通信，我使用管道。

我已将服务器应用程序分为后端和 GUI。后端有一个 PipeServer 类，在 GUI 中，我重写了 onReceiveMessage 等函数。

在我决定添加 std::queue 作为基类成员之前，一切正常。

在应用程序开始时，我遇到了一个异常，经过检查，我的队列似乎不是以 0 个元素开始的。事实上，队列似乎根本没有初始化。有两种可能性：可能是因为我的 GUI 类继承了 2 个类，并且不知何故第二个基类，即我的 PipeServer 没有正确初始化其成员，或者可能是因为 pipeServerGUI 对象被移动到不同的线程QT。

关于如何解决这个问题的任何想法？

相关代码：

class HookServer
{
    PIPEINST Pipe[INSTANCES]; 
    HANDLE hEvents[INSTANCES]; 

    VOID DisconnectAndReconnect(DWORD); 
    BOOL ConnectToNewClient(HANDLE, LPOVERLAPPED); 
    VOID GetAnswerToRequest(LPPIPEINST); 

public:
    std::queue<std::string> messages;

    int init(std::string pipename);
    int run();
    virtual void onNewConnection() {};
    virtual void onReceiveMessage(std::string message) {};
};

class HookServerGUI : public QObject, public HookServer
{
    Q_OBJECT
    void onReceiveMessage(std::string message);
    void onNewConnection();

    public slots:

    void doWork() {
        init("\\\\.\\pipe\\hookpipe");
        run();
    }

    signals:
    void signalGUI(QString message);
};

    //GUIServerCreation

    QThread *thread = new QThread;
    HookServerGUI* worker = new HookServerGUI;
    QObject::connect(worker,SIGNAL(signalGUI(const QString&)),this,SLOT(processMessage(const QString&)));

    worker->moveToThread(thread);
    thread->start();



    QMetaObject::invokeMethod(worker, "doWork", Qt::QueuedConnection);

编辑：

异常是访问冲突异常。它发生在这部分代码中：

VOID HookServer::GetAnswerToRequest(LPPIPEINST pipe)
{
    onReceiveMessage(pipe->chRequest);
    if(!messages.empty())
    {
        std::string s = messages.front();
        messages.pop();
        strcpy(pipe->chReply,s.c_str());
        pipe->cbToWrite = strlen(s.c_str()+1);
    }
}

由于 messages.empty() 返回了一个巨大的数字，它尝试读取第一个对象并以某种方式失败。

也没有 PipeServerGUI 构造函数。

编辑2：

我通过在new HookServerGUI(); 后面加上括号解决了部分问题

问题是该函数仍然不起作用，并引发访问冲突异常。它发生在 front() 行。在调试器中检查时，该函数确实有 1 个元素，所以不是因为它是空的。有什么想法吗？

EDIT3：

在第二次运行时，不幸的是 queue.size() 仍然不正确。对我来说，这就像一场数据竞赛。

Answer 1

问题出在您没有显示的代码中，看起来像是内存错误的经典案例。某处的某些代码正在写入它不拥有的内存。可能您在使用 winapi 的方式上存在错误。您需要创建一个最小的、独立的测试用例。

我认为你可能会因为不使用QLocalSocket而自取其辱：在 Windows 上，它是一个命名管道 - 正是你想要的。

此外，这是 C++ 代码。根本没有理由将 PIPEINST 或 HANDLE 放入原始 C 数组。使用QVector 或std::vector。可能其余的代码都充满了类似的 C 主义，并且某些地方出了问题。

我不会忽视缓冲区溢出，因为显然您忽略了 PIPEINST 中缓冲区的大小 - strcpy 可能会溢出缓冲区。我也不确定 PIPEINST from the example code 使用的字符类型与 std::string::c_str() 返回的字符类型相同。

即使您想使用不带QLocalSocket 的显式管道来实现您的代码，您仍然应该使用C++、QString 等并了解您的数据发生了什么。