Отправка файлов по TCP с использованием C ++, получение неправильного размера - PullRequest
Новая
       50

Отправка файлов по TCP с использованием C ++, получение неправильного размера

0 голосов
/ 16 апреля 2020

Я очень новичок в программировании сокетов, и я пытаюсь отправить по TCP-соединению, но получаю несколько ошибок.

вот мой код 

        FILE* File;
        char* Buffer;
        unsigned long Size;

        File = fopen("C:\\test.zip", "rb");
        if (!File)
        {
            printf("Error while readaing the file\n");
            return;
        }
        // file size 1
        fseek(File, 0, SEEK_END);
        Size = ftell(File);
        fseek(File, 0, SEEK_SET);


        Buffer = new char[Size]; 

        fread(Buffer, Size, 1, File);
        char cSize[MAX_PATH];
        sprintf(cSize, "%i", Size);

        cout << "MAX PATH " << MAX_PATH<<endl;
        cout << "cSize: " << cSize << endl;
        fclose(File);

`Итак, чтобы найти размер моего файла. большую часть кода я пробую из других вопросов здесь, но это не решило мою проблему. 'my send and recv: 

unsigned long filechunk = 1025;

unsigned long byteSent = 0;
unsigned long bytesToSend = 0;

send(Sub, cSize, MAX_PATH, 0); // File size to client

while (byteSent < Size) {

    if ((Size - byteSent) >= filechunk) {
        bytesToSend = filechunk;
    }
    else {
        bytesToSend = Size - byteSent;
    }
    if (send(Sub, Buffer + byteSent, bytesToSend, 0)) {
        std::cout << "Sent: ";
    }
    byteSent += bytesToSend;
    std::cout << "Size : "<<Size<<"    BytesSent : "<<byteSent<<"   Bytes to send: " << bytesToSend << std::endl;
    system("pause");

на стороне клиента: 

int Size;
char* Filesize = new char[5000000]; // is there a better way? my sfiles size are unknown but at least 50mb

if (recv(Socket, Filesize, 5000000, 0)) // File size
{
    Size = atoi((const char*)Filesize);
    printf("File size: %d\n", Size);
}

char* Buffer = new char[Size];
FILE* File;
File = fopen("test.zip", "wb"); //start copying from the server, creating the file first.
std::string convert;
long conv;

std::cout << "Size: " << Size << std::endl;

int total=Size;
int byteRecv = 0;
int recvCheck;
int bytes = 1025;

//getting the file
while (byteRecv < Size ) {

    recvCheck = recv(Socket, Buffer, bytes, 0);
    if (recvCheck >0) // File
    {
        fwrite(Buffer, 1, byteRecv, File);

        std::cout << "Recieved:" << byteRecv << std::endl;

        Size -= byteRecv;
        byteRecv += byteRecv;
        std::cout << "Error: " << WSAGetLastError();
    }
    else {
        std::cout << "Error: " << WSAGetLastError();
        total += 1; // the loop often get into infinit loop so i force it in case of this error.
        if (total > 3) {
            break;
        }
    }
}
fclose(File);

Итак, я знаю, что это не очень эффективно, и я не уверен, что есть подобные вопросы, так как я копался здесь уже несколько недель.

- есть ли лучший способ, как я могу сделать чарса * []? так как я еще не знаю размер файлов, которые я хочу отправить. - ftell () и sifeof () работают одинаково? -При проверке размера, полученного с сервера, он ошибается. Пример: файл сервера: 32633513, размер recv: 3263 - большая часть кода, который я взял из других проблем и объединил его. если вы видите что-то ненужное, скажите мне об этом.

1 Ответ

0 голосов
/ 16 апреля 2020

Есть много неправильных вещей, но это может сначала исправить вашу проблему:

На стороне клиента замените (вы оба уменьшаете общее количество байтов и полученное с неправильным значением): 

Size -= byteRecv;
byteRecv += byteRecv;

с: 

byteRecv += recvCheck; // actualizes the count of received bytes

Другая проблема - размер вашего буфера. Никогда не пытайтесь получить весь файл в памяти, это вообще нонсенс; файлы обычно управляются чанками чанками. Поскольку вы читаете не более 1025 байтов в каждом l oop, тогда используйте только буфер размером 1025, вам больше не нужно. То же самое для чтения и письма ...

...