Помогите решить проблему

nsw, объявить переменную -_-

kekch127, а окей

nsw, объявить переменную или спросить у chatgpt как будет корректно

Narg, идентификатор не был объявлен, идентификатор написан с ошибками, заголовок, в котором объявлен идентификатор, не включен в файл, или в идентификаторе отсутствует квалификатор области, например, cout вместо std::cout

nsw, а ну вроде да это же из внешней библиотеки sockAddr. тогда хз гугли

nsw, не моя это тема, чем смог тем помог=)

lethe1337, да нечего страшного ты мне помог решить проблему

nsw, на первый взгляд, проблем в коде нет, но есть несколько моментов, которые можно улучшить


необходимо добавить обработку ошибок для функций bind() и listen(), можно использовать макросы WSAGetLastError() и perror():

if (bind(sock, (SOCKADDR*)&sockAddr, sizeof(SOCKADDR)) == SOCKET_ERROR) {
perror("bind failed");
closesocket(sock);
WSACleanup();
return 1;
}

if (listen(sock, 1) == SOCKET_ERROR) {
perror("listen failed");
closesocket(sock);
WSACleanup();
return 1;
}

Код

if (bind(sock, (SOCKADDR*)&sockAddr, sizeof(SOCKADDR)) == SOCKET_ERROR) {

    perror("bind failed");

    closesocket(sock);

    WSACleanup();

    return 1;

}



if (listen(sock, 1) == SOCKET_ERROR) {

    perror("listen failed");

    closesocket(sock);

    WSACleanup();

    return 1;

}

в функции receive_data() нет обработки ошибок при вызове recv(). Если recv() завершается с ошибкой, программа может продолжить работу и неожиданно завершиться позже. думаю нужно проверить, что значение recv_len не равно SOCKET_ERROR и recv_len не равно 0. если recv_len равно SOCKET_ERROR, то необходимо обработать ошибку, например, закрыть сокеты и завершить программу

int recv_len = recv(sock, buffer, sizeof(buffer), 0);
if (recv_len == SOCKET_ERROR) {
perror("recv failed");
closesocket(sock);
WSACleanup();
return 1;
} else if (recv_len == 0) {
std::cout << "Connection closed" << std::endl;
closesocket(sock);
WSACleanup();
return 0;
} else {
buffer[recv_len] = '\0';
std::cout << "Key pressed: " << buffer << std::endl;
}

Код

int recv_len = recv(sock, buffer, sizeof(buffer), 0);

if (recv_len == SOCKET_ERROR) {

    perror("recv failed");

    closesocket(sock);

    WSACleanup();

    return 1;

} else if (recv_len == 0) {

    std::cout << "Connection closed" << std::endl;

    closesocket(sock);

    WSACleanup();

    return 0;

} else {

    buffer[recv_len] = '\0';

    std::cout << "Key pressed: " << buffer << std::endl;

}

в текущей реализации поток receiver не останавливается, когда клиент закрывает соединение. Необходимо добавить проверку на завершение соединения и остановить поток, когда это происходит. это можно сделать, например, путем добавления проверки на значение recv_len в функцию receive_data():

void receive_data(SOCKET sock)
{
while (true) {
char buffer[256];
int recv_len = recv(sock, buffer, sizeof(buffer), 0);
if (recv_len == SOCKET_ERROR) {
perror("recv failed");
closesocket(sock);
return;
} else if (recv_len == 0) {
std::cout << "Connection closed" << std::endl;
closesocket(sock);
return;
} else {
buffer[recv_len] = '\0';
std::cout << "Key pressed: " << buffer << std::endl;
}
}
}

Код

void receive_data(SOCKET sock)

{

    while (true) {

        char buffer[256];

        int recv_len = recv(sock, buffer, sizeof(buffer), 0);

        if (recv_len == SOCKET_ERROR) {

            perror("recv failed");

            closesocket(sock);

            return;

        } else if (recv_len == 0) {

            std::cout << "Connection closed" << std::endl;

            closesocket(sock);

            return;

        } else {

            buffer[recv_len] = '\0';

            std::cout << "Key pressed: " << buffer << std::endl;

        }

    }

}

серек, это твой второй акк?

nsw, нет

nsw, это его ферма