在linux中读取不同的文件？ C++答案

【问题标题】：Files reading differently in linux? C++在linux中读取不同的文件？ C++
【发布时间】：2011-04-18 21:29:32
【问题描述】：

我是一个相当新的程序员，但我认为我的 google-fu 很能干，我花了几个小时搜索。

我有一个简单的 SDL 应用程序，它从二进制文件中读取数据（2 个字节作为幻数，然后每个“tile”5 个字节）然后它显示缓冲区中的每个图块，字节决定 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;
}

提前致谢！

【问题讨论】：

请贴出代码。您似乎对编译器将如何做事做出了一些假设；不同的平台对诸如结构包装之类的事情有（稍微）不同的约定。贴出代码，很快会有人知道的。
您可能希望包括您在每个平台上使用的编译器。
我已经用更多相关信息更新了代码
你在 windows 和 linux 上使用的是同一个二进制文件吗？在那种情况下，你是如何创建这个二进制文件的？
是的，它是同一个文件（我使用保管箱，所以它完全同步）我在十六进制编辑器中查看了该文件，两者都是相同的。（创建二进制文件是一个单独的程序）

标签： c++ linux mingw32

【解决方案1】：

您的数组处理不正确。
C/C++ 中的数组索引从 0 开始。

您已将“tile_buffer”定义为大小为“BYTES_PER_TILE”的数组。
如果 BYTES_PER_TILE 为 5，则您的数组将包含元素 tile_buffer[0] 到 tile_buffer[4]。

在您的内部 for 循环中，您从 1 循环到 5，因此会发生缓冲区溢出。

我不知道这是否是您的问题的原因，但这肯定无济于事。

【讨论】：

谢谢，修复代码解决了问题！让我想知道为什么它在 Windows 上正常工作。

【解决方案2】：

这可能不是答案，但是基于 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]]);
    }
}

【讨论】：

我是一个新程序员，但我会看看那个代码，看看我是否可以用它来优化我的程序，谢谢！