Question

我将二进制格式的游戏地图保存到文件中。文件的一般结构是：

（4byte）width
（4byte）height
宽*高：
- （1byte）tile
宽*高：
- （4byte）数据

为了减小文件的大小，我决定将所有内容保存在字符中，因为我知道要读取的数字的确切大小。我做了两个函数，getByte和makeInt：

unsigned char getByte(int src, short int c) //get byte number c from src
{
    return ((char*)(&src))[c];
}

int makeInt(char src[4]) //build int from src
{
    int r=0;
    for (int i=3; i>=0; i--)
        r+=(r<<8)+((unsigned char*)(src))[i];
    return r;
}

以下是我保存和加载地图的方法：

void Map::save(const char *fname)
{
    ofstream f;
    f.open(fname,fstream::out | fstream::binary);

    for (int i=3; i>=0; i--)
        f.put(getByte(W,i));
    for (int i=3; i>=0; i--)
        f.put(getByte(H,i));

    for (int i=0; i<W; i++)
        for (int j=0; j<H; j++)
            f.put(tiles[i][j]);

    for (int i=0; i<W; i++)
        for (int j=0; j<H; j++)
            for (int k=3; k>=0; k--)
                f.put(getByte(data[i][j],k));

    f.close();
}

void Map::load(const char *fname)
{
    ifstream f;
    f.open(fname,fstream::in | fstream::binary);

    char bytes[4];

    for (int i=3; i>=0; i--)
        f.get(bytes[i]);    
    W=makeInt(bytes);

    for (int i=3; i>=0; i--)
        f.get(bytes[i]);
    H=makeInt(bytes);

    for (int i=0; i<W; i++)
        for (int j=0; j<H; j++)
            f.get(tiles[i][j]);

    for (int i=0; i<W; i++)
        for (int j=0; j<H; j++)
        {
            for (int k=3; k>=0; k--)
                f.get(bytes[k]);
            data[i][j]=makeInt(bytes);
        }

    f.close();
}

Tiles（char）保存并加载良好，一切正常。但是int不是。由于奇怪的错误，当我保存256时，我加载257.而不是预期的512，我读514.我不想让它变脏（r- = r / 256）...我也遇到了一些问题迹象，这可能就是为什么这一切都会发生的原因。在添加一些未签名的强制转换之前，127之后是-127，-126，-125等。如果这有助于这里是地图类：

class Map
{
    public:
        int W;
        int H;

        char tiles[MW][MH]; //MW and MH are macros, now they are 64.
        int data[MW][MH];

        void save(const char *fname);
        void load(const char *fname);

    /* moar code */
};

我通过使用这种模式生成地图来测试它：

for (int i=0; i<64; i++)
    for (int j=0; j<64; j++)
        map.data[i][j]=(i*64+j);

Answer 1

这是不正确的（函数makeInt（char [4]））：

r+=(r<<8)+((unsigned char*)(src))[i];

你想要r = r<<8 + ...。实际上，您不止一次地累积该值。

Answer 2

r+=(r<<8)+((unsigned char*)(src))[i];应为r=(r<<8)+((unsigned char*)(src))[i];

现在，我普遍担心的是，您可能不必过于担心在这里或那里保存一个字节。如果你真的想节省空间，可以使用一些适当的压缩技术 - 霍夫曼编码，字节对或类似的东西。

你确定使用f.write（）和f.read（）节省了空间 - 我很确定你的代码会更简单 - 除非你打算将文件存储在一台机器上，否则它们跨越到具有不同字节顺序的另一个，然后将它们加载到那里的代码中，一次存储一个字节需要做很多额外的工作。虽然我确信这是一个很好的学习练习。至少编写一个函数来一次存储一个int可能有助于清理代码。

Int到字节和向后

2 个答案: