[Hetelek]

Hey, I am trying to make a XDBF class, and I am reading a file and this is getting ridiculously annoying.

First: I open my file...

openedFile = fopen(path, "r+");

It works, and my whole constructor works correctly:

XDBF::XDBF(const char* path)
{
    openedFile = fopen(path, "r+");

    if(openedFile == NULL)
    {
        //File could not be opened.
        throw 0;
    }

    fseek(openedFile, 0, SEEK_SET);
    fread(h, sizeof(Header), 1, openedFile);


    if(h->magic == 0x46424458)
    {
        //Little endian detected.

        SwapEndian(h->magic);
        SwapEndian(h->entry_count);
        SwapEndian(h->entry_table_length);
        SwapEndian(h->free_space_table_entry_count);
        SwapEndian(h->free_space_table_length);
        SwapEndian(h->version);
    }
    else if(h->magic != 0x58444246)
    {
        //Invalid file. (MAGIC)
        throw 1;
    }

    Entry *temp_entries = new Entry[h->entry_count];
    fread(temp_entries, sizeof(Entry), h->entry_count, openedFile);

    for(unsigned int i = 0; i < h->entry_count; i++)
    {
        SwapEndian(temp_entries[i].type);
        SwapEndian(temp_entries[i].identifier);
        SwapEndian(temp_entries[i].length);
        SwapEndian(temp_entries[i].offset_specifier);
    }

    private_entries = temp_entries;
}

I know it works correctly, because I looked at all of the entries that it loaded and stuff.

Now, the constructor ends, and I do this to extract an entry:

    XDBF a("C:\\Users\\Stevie\\Desktop\\file.gpd");
    FILE *f;
    f = fopen("C:\\Users\\Stevie\\Desktop\\a.png", "wb");
    fseek(f, 0, SEEK_SET);

    XDBF::Entry *entry = a.get_entry_by_id(0x1C);
    char *buff = a.extract_entry(entry);
    a.close();

    int size = entry->length;
    QMessageBox::information(this, "", QString::number(size), QMessageBox::Ok);
    fwrite(buff, sizeof(char), size, f);

    fclose(f);

Now, here are the 2 functions I call(get_entry_by_id, and extract_entry):

XDBF::Entry* XDBF::get_entry_by_id(long long identifier)
{
    Entry* entries = get_entries();
    for(unsigned int i = 0; i < get_header()->entry_count; i++)
        if(entries[i].identifier == identifier)
            return &entries[i];

    return NULL;
}

char *XDBF::extract_entry(Entry *entry)
{
    if(openedFile == NULL)
        return NULL;

    unsigned int offset = get_offset(entry->offset_specifier, h);
    fseek(openedFile, offset, SEEK_SET);
    char *data = new char[entry->length];
    unsigned int poophead = (unsigned int)data;

    fread(data, entry->length, sizeof(char), openedFile);

    return data;
}

Now once I get the char* from 'extract_entry', I write it to a file... It's really dumb because it reads the data incorrectly... I don't know how it's possible, but it gets the bytes:

89 50 4E 47 0A...

out of the file, while the real bytes are:

89 50 4E 47 0D...

The bytes that I posted first don't even exist in the GPD... I also look at the location that it seeks to before reading the entry in the 'extract_entry' function, and it's correct...

I am new the C++, so there is probably some stupid stuff I did in the code posted. Thanks for any help.

[austin12456]

"rb+" instead of "r+"

r is for text files, and rb is read binary

[Hetelek]

Originally Posted by austin12456

"rb+" instead of "r+"

r is for text files, and rb is read binary

-___________________________________-

That worked. Please close.

[Chris]

Closed.