0

Имеется код:

Сервер:

#define _WINSOCK_DEPRECATED_NO_WARNINGS
#pragma comment(lib, "ws2_32.lib")
#include <WinSock2.h>
#include <iostream>
#include <string>
#include <conio.h>
#pragma warnings(disable:4996)

using namespace std;

struct day {
    string name;
    int temp;
};

void generateArray(day *arr, const int SIZE);

int main() {
    WSAData wsaData;
    //Запрашиваемая версия библиотеки winsock. Потом понадобится, чтобы загрузить библиотеку
    WORD dllVersion = MAKEWORD(2, 1);
    //Загружаем библиотеку. Первый параметр - запрашиваемая версия библиотеки. Второй - ссылка на структуру
    if (WSAStartup(dllVersion, &wsaData) != 0) {
        cout << "Error in library loading" << endl;
        exit(1);
    }
    SOCKADDR_IN addr;
    //Localhost
    addr.sin_addr.s_addr = inet_addr("127.0.0.1");
    //Порт
    addr.sin_port = htons(1111);
    //Семейство портов
    addr.sin_family = AF_INET;
    //Семейство протоколов, протокол

    SOCKET sListen = socket(AF_INET, SOCK_STREAM, NULL);
    bind(sListen, (SOCKADDR*)&addr, sizeof (addr));
    listen(sListen, SOMAXCONN);

    SOCKET newConnection;
    int size = sizeof(SOCKADDR);
    newConnection = accept(sListen, (SOCKADDR*)&addr, &size);

    if (newConnection == 0) {
        cout << "Error in creating newConnection" << endl;
        exit(1);
    }
    else {
        cout << "Client connected" << endl;
        char mes[256];
        const int DAYAM = 7;
        day *arr = new day[DAYAM];


        generateArray(arr, DAYAM);
        recv(newConnection, mes, sizeof(mes), NULL);
        string inDay = mes;
        for (int i = 0; i < DAYAM; i++) {
            if (inDay == arr[i].name) {

                const char* temper = arr[i].temp+"\0";
                send(newConnection, temper, sizeof(temper), NULL);
                break;
            }
        }



    }

    return 0;
}

void generateArray(day * arr, const int SIZE)
{
    arr[0].name = "Monday";
    arr[1].name = "Tuesday";
    arr[2].name = "Wednesday";
    arr[3].name = "Thursday";
    arr[4].name = "Friday";
    arr[5].name = "Saturday";
    arr[6].name = "Sunday";
    for (int i = 0; i < SIZE; i++) {
        arr[i].temp = i + 15;
    }
}

Клиент

#define _WINSOCK_DEPRECATED_NO_WARNINGS

#pragma comment(lib, "ws2_32.lib")
#include <WinSock2.h>
#include <iostream>
#include <string>
#pragma warnings(disable:4996)


using namespace std;
int main() {
    WSAData wsaData;
    //Запрашиваемая версия библиотеки winsock. Потом понадобится, чтобы загрузить библиотеку
    WORD dllVersion = MAKEWORD(2, 1);
    //Загружаем библиотеку. Первый параметр - запрашиваемая версия библиотеки. Второй - ссылка на структуру
    if (WSAStartup(dllVersion, &wsaData) != 0) {
        cout << "Error in library loading" << endl;
        exit(1);
    }
    SOCKADDR_IN addr;
    //Localhost
    addr.sin_addr.s_addr = inet_addr("127.0.0.1");
    //Порт
    addr.sin_port = htons(1111);
    //Семейство портов
    addr.sin_family = AF_INET;
    //Семейство протоколов, протокол

    SOCKET sListen = socket(AF_INET, SOCK_STREAM, NULL);
    bind(sListen, (SOCKADDR*)&addr, sizeof(addr));
    listen(sListen, SOMAXCONN);

    SOCKET connection = socket(AF_INET,SOCK_STREAM,NULL);
    if (connect(connection, (SOCKADDR*)&addr, sizeof(addr)) != 0) {
        cout << "Error in connection" << endl;
        return 1;
    }
    cout << "Connection succseed!" << endl;


    cout << "Enter the day" << endl;
    string day;
    cin >> day;
    const char *str = day.c_str();
    send(connection, str, sizeof(day), NULL);
    char temper[10];
    recv(connection, temper, sizeof(temper), NULL);
    cout << temper << endl;


    system("pause");
    return 0;
}

Задача - клиент вводит день недели, сервер ему в ответ выдает погоду в этот день. Передача данных от клиента к серверу происходит успешно, а вот обратно - нет. В чем проблема и как исправить? (За то, что очень криво передаю числа, заранее извиняюсь, но так и не разобрался как отправить не const char*)

3
  • А цикла не предусмотрено на сервере? Он так после первого ацепта вывалиться.. И newConnection == 0 есть нормальное состояние, по нему надо перезапускать ацепт, ошибка это -1.
    – NewView
    12 сен 2018 в 20:51
  • Сравнить inDay == arr[i].name так нельзя, используйте strcmp, memcmp и тд
    – NewView
    12 сен 2018 в 20:57
  • @NewView, почему нельзя? Это ж 2 string'a 12 сен 2018 в 21:25

1 ответ 1

1

Прежде всего проблема в практически полном отсутствии проверки ошибок. Например размер полученных данных в recv(newConnection, mes, sizeof(mes), NULL); остается неизвестным, более того, mes может вообще остаться неинициализированным, а функция может не дождаться запрошенных 256 байт. В const char* temper = arr[i].temp+"\0"; происходит смещение указателя явно за пределы массива. sizeof(temper) - это вовсе не длинна строки, а размер указателя, sizeof(day) - это вовсе не длина строки, а размер объекта ::std::string. Также не следует использовать #define _WINSOCK_DEPRECATED_NO_WARNINGS и #pragma warnings(disable:4996).

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.