2

私はかなり新しいプログラマーですが、私の google-fu は非常に有能であると考えており、検索に数時間を費やしました。

バイナリ ファイル (マジック ナンバーとして 2 バイト、次に「タイル」ごとに 5 バイト) から読み取る単純な SDL アプリケーションがあります。次に、各タイルをバッファーに表示し、バイトが x、y、id、通過可能性を決定します。など。つまり、実際にはレベルの読み込みです。

任意の Windows コンピューター (Windows Server 2008、7/64、および 7/32 でテスト済み) で正常に動作しますが、Linux でコンパイルすると、ランダムなタイルがランダムな位置に表示されます。RAM の間違った部分から読み取っていると言いたくなるかもしれませんが、最初の 2 バイトが外れた場合にエラーが返されるようにマジック ナンバーを実装しました。

私はこれを自分で理解したいと思っていますが、今は地獄に悩まされており、移動中にプログラミングできない限り、これ以上先に進むことはできません（私のラップトップはLinuxを実行しています）。LinuxではG ++、Windowsではmingw32g ++を使用しています。

bool loadlevel(int level_number)
{
    int length;
    std::string filename;
    filename = "Levels/level";
    filename += level_number+48;
    filename += ".lvl";
    std::ifstream level;
    level.open(filename.c_str(),std::ios::in|std::ios::binary);
    level.seekg(0,std::ios::end);
    length = level.tellg();
    level.seekg(0,std::ios::beg);
    char buffer[length];
    level.read(buffer,length);
    if (buffer[0] == 0x49 && buffer[1] == 0x14)
    {
        char tile_buffer[BYTES_PER_TILE];
        int buffer_place = 1;
        while(buffer_place < length)
        {
            for (int i = 1;i <= BYTES_PER_TILE;i++)
            {
                tile_buffer[i] = buffer[buffer_place+1];
                buffer_place++;
            }
            apply_surface(tile_buffer[1],tile_buffer[2],tiles,screen,&clip[tile_buffer[3]]);
        }
    }
    else
    {
        // File is invalid
        return false;
    }
    level.close();
    return true;
}

前もって感謝します！

4

2 に答える 2

5

配列の処理が正しくありません。
C/C++ の配列インデックスは 0 から始まります。

「tile_buffer」を「BYTES_PER_TILE」サイズの配列に定義しました。
BYTES_PER_TILE が 5 の場合、配列には tile_buffer[0] から tile_buffer[4] までの要素が含まれます。

内側の for ループでは、1 から 5 までループするため、バッファ オーバーフローが発生します。

これがあなたの問題の原因かどうかはわかりませんが、問題は解決しません。

于 2010-09-29T16:07:21.177 に答える
0

これはおそらく答えではありませんが、1 から始まる配列の処理と不要なコピーに頭が痛くなります。

これらの線に沿って何かをしてみませんか？

if ((length >= 2+BYTES_PER_TILE) && (buf[0] == CONST1) && (buf[1] == CONST2)) {
    for (char *tile = &buf[2]; tile < &buf[length-BYTES_PER_TILE]; tile+=BYTES_PER_TILE) {
        apply_surface(tile[0],tile[1],tiles,screen,&clip[tile[2]]);
    }
}
于 2010-09-29T17:18:02.210 に答える