Question

我目前正在尝试学习套接字和线程如何工作，为此，我开始编写一个程序来创建一些聊天室：

#include <iostream>
#include <windows.h>
#include <process.h>
#include <winsock2.h>
#include <vector>

using namespace std;

struct par {
    SOCKET* s;
    string* buffer;
    bool keep = 1;
};

struct HANDLES {
    HANDLE hRecv;
    HANDLE hSend;
};

unsigned int WINAPI recvThread(LPVOID param) {
    par* data = (par*)param;
    char* inBuf;
    unsigned short int inSize;
    string aux = inBuf;
    string* auxptr = data->buffer;
    SOCKET* s = data->s;
    while(true) {
        inSize = recv(*s, inBuf, 2000, 0);
        inBuf[inSize] = '\0';
        *auxptr = aux; // ???
    }
}

unsigned int WINAPI sendThread(LPVOID param) {
    par* data = (par*)param;
    string auxBuf = "NULL";
    string* auxptr = data->buffer;
    SOCKET* s = data->s;
    while (true) {
        if(auxBuf != (*auxptr)) {
            string auxstring = *auxptr;
            const char* auxchar = auxstring.c_str();
            send(*s, auxchar, strlen(auxchar), 0);
            auxBuf = auxstring;
        }
        Sleep(200);
    }
}

int main()
{
    WSADATA wsa;
    WSAStartup(MAKEWORD(2,2), &wsa);
    string buf = "NULL";
    vector<HANDLES> hVec;

    SOCKET s;
    s = socket(AF_INET, SOCK_STREAM, 0);

    sockaddr_in server;

    server.sin_family = AF_INET;
    server.sin_port = htons(8888);
    server.sin_addr.s_addr = INADDR_ANY;

    bind(s, (sockaddr*)&server, sizeof(server));

    listen(s, 5);

    sockaddr_in clientdir;
    int siz = sizeof(clientdir);
    SOCKET cliente;
    while((cliente = accept(s, (sockaddr*)&clientdir, &siz))) {
        par* newParams;
        newParams->s = &cliente;
        newParams->buffer = &buf;
        if(cliente != INVALID_SOCKET) {
            HANDLES aux;
            aux.hRecv = (HANDLE)_beginthreadex(NULL, 0, recvThread, (void*)newParams, 0, 0);
            aux.hSend = (HANDLE)_beginthreadex(NULL, 0, sendThread, (void*)newParams, 0, 0);
            hVec.push_back(aux);
        }
    }
}

遇到的第一个问题出现在：newParams-＆gt; s =＆amp; cliente; 使服务器崩溃。如果有人能告诉我我做错了什么以及应该实施的任何其他修复，我将不胜感激。

Answer 1

第一个问题是你有一个未初始化的指针（newParams），它没有指向任何有效的地方。然后使用，这将导致未定义的行为。

当您修复上述问题时（通过为par分配新的new结构并将newParams指向该对象），您有另一个问题：指针到结构中的SOCKET和string成员。所有指针，无论你有多少（或几个）连接和线程都将指向相同 SOCKET或string。

如果接收和发送线程应该共享SOCKET和string，那么它们不是将这些成员作为指针，而是在需要时复制的值，但线程共享 structure ，它通过将相同的newParams指针传递给两个线程来解决。

未初始化的指针以及数据争用还存在许多问题（因为您不保护线程共享的资源）。

我的建议是退一步，并废弃所有代码。然后重新编码它，同时采取小步骤，一次添加一点，同时确保它的工作，并且你理解如何它的工作原理。 Rubber-duck debugging以及使用实际的调试器会有很大帮助。和看书。并且练习。而不是急于尝试一次完成所有事情。

C ++：无法使套接字，线程和结构起作用

1 个答案: