fseek не читает все данные? - PullRequest
Новая
       2

fseek не читает все данные?

1 голос
/ 29 марта 2011

Я делаю установщик c ++, и я добавил к исполняемому файлу и файл для извлечения, и 8-байтовый размер файла для извлечения в программе.Моя программа выходит из-за ошибки чтения файла, что не так?К сведению, у меня нет никаких знаний об управлении файлами c, кроме того, что я узнал сегодня.Я пишу файл test_before.tar.gz, который составляет 161 байт, исполняемый файл имеет длину 12335 байт, а размер файла составляет 8 байт и содержит 0000161. Что не так?При необходимости попросите дополнительную информацию. 

#include <iostream>
#include <cstdio>
#include <cstdlib>
#include <cstring>
using namespace std;

int main(int argc, char* argv[])
{
    cout << "Opening the executable as read-only!" << endl;
    FILE *exeFile; // The executable file pointer
    FILE *outFile; // The file to write pointer

    // Check whether a file name was supplied
    if(argc < 2)
    {
        cout << "Please enter the file to write!" << endl;
        return 1;
    }

    // Open the executable as read-only
    if((exeFile = fopen(argv[0], "rb")) == 0)
    {
        cout << "Error opening the executable!" << endl;
        return 1;
    }

    cout << "Getting the executables size!" << endl;

    // Get the files size
    fseek(exeFile, 0, SEEK_END);
    int size = ftell(exeFile);

    cout << "Reading ofset!" << endl;   

    // Read the ofset bytes contained in the last 7-bytes
    char filesize_char[9];
    fseek(exeFile, -8, SEEK_END);
    fgets(filesize_char, 9, exeFile);

    // Convert
    int filesize = atoi(filesize_char);
    int ofset = (size - filesize) - 8;

    cout << "The ofset size is " << ofset << " bytes!" << endl;
    cout << "The file size is " << filesize << " bytes!" << endl;

    cout << "Reading the file to extract!" << endl;

    // Create the variable to contain the file and goto the ofset
    char* contents = new char[filesize + 1];
    fseek(exeFile, ofset, SEEK_SET);

    // Read the file to extract
    if(fread(contents, sizeof(char), filesize + 1, exeFile) != sizeof(contents))
    {
        // Error has occured
        if(feof(exeFile)) {
            cout << "Premature end of file!" << endl;

            // Delete variables so they dont "leak"
            fclose(exeFile);
            delete[] contents;

            return 1;
        } else {
            cout << "File read error!" << endl;

            // Delete variables so they dont "leak"
            fclose(exeFile);
            delete[] contents;

            return 1;
        }
    }

    cout << "Writing the file to " << argv[1] << "!" << endl;

    // Write the file to extract
    if((outFile = fopen(argv[1], "wb")) == 0)
    {
        cout << "Error opening the file to write!" << endl;

        // Delete variables so they dont "leak"
        fclose(exeFile);
        fclose(outFile);
        delete[] contents;

        return 1;
    }

    fwrite(contents, 1, sizeof(contents), outFile);

    //delete variables
    fclose(exeFile);
    fclose(outFile);
    delete[] contents;

    return 0;
}

1 Ответ

1 голос
/ 30 марта 2011

Вам вообще не нужен цикл while.В конце концов, вы уже распределили память, которая будет содержать все ваши данные.Переместите указатель файла в начало данных fseek(exeFile, ofset, SEEK_SET), затем используйте fread, чтобы прочитать его целиком, а затем используйте fwrite, чтобы записать его в outFile.

. Вам следует открыть exeFileи outFile с флагами "rb" и "wb", иначе ваш код будет надежно работать только с текстовыми данными.

...