2

我是一个相当新的程序员，但我认为我的 google-fu 非常称职，我花了几个小时搜索。

我有一个简单的 SDL 应用程序，它从二进制文件中读取（2 个字节作为幻数，然后每个“tile”5 个字节）然后显示缓冲区中的每个 tile，字节决定 x、y、id、passability等等。所以这只是水平加载。

它在任何 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 回答