Question

我这里有一个最奇怪的问题......我在Windows中使用相同的代码（复制粘贴）到 READ 和 WRITE 和 BMP 图片。从 Linux 中的某些原因来看，每件事都可以完全正常，但是当我从某些人那里来到 Windows 10 时，我无法打开它图片，我收到一条错误信息，如何说出这样的话：

“看起来我们不支持这种文件格式。”

你知道我该怎么办？我将把代码放在下面。

修改

我已经解决了填充问题，现在它正在编写图像，但它们完全是白色，任何想法为什么？我也更新了代码。

struct BMP {
    int width;
    int height;
    unsigned char header[54];
    unsigned char *pixels;
    int size;
    int row_padded;
};

void writeBMP(string filename, BMP image) {
    string fileName = "Output Files\\" + filename;
    FILE *out = fopen(fileName.c_str(), "wb");
    fwrite(image.header, sizeof(unsigned char), 54, out);

    unsigned char tmp;
    for (int i = 0; i < image.height; i++) {
        for (int j = 0; j < image.width * 3; j += 3) {
            // Convert (B, G, R) to (R, G, B)
            tmp = image.pixels[j];
            image.pixels[j] = image.pixels[j + 2];
            image.pixels[j + 2] = tmp;
        }
        fwrite(image.pixels, sizeof(unsigned char), image.row_padded, out);
    }
    fclose(out);
}

BMP readBMP(string filename) {
    BMP image;
    string fileName = "Input Files\\" + filename;
    FILE *f = fopen(fileName.c_str(), "rb");

    if (f == NULL)
        throw "Argument Exception";

    fread(image.header, sizeof(unsigned char), 54, f); // read the 54-byte header

    // extract image height and width from header
    image.width = *(int *) &image.header[18];
    image.height = *(int *) &image.header[22];

    image.row_padded = (image.width * 3 + 3) & (~3);
    image.pixels = new unsigned char[image.row_padded];
    unsigned char tmp;

    for (int i = 0; i < image.height; i++) {
        fread(image.pixels, sizeof(unsigned char), image.row_padded, f);
        for (int j = 0; j < image.width * 3; j += 3) {
            // Convert (B, G, R) to (R, G, B)
            tmp = image.pixels[j];
            image.pixels[j] = image.pixels[j + 2];
            image.pixels[j + 2] = tmp;
        }
    }
    fclose(f);
    return image;

}

在我看来，这段代码应该是跨平台的...但它不是......为什么？

感谢您的帮助

Answer 1

检查标题

header必须以以下两个signature字节开头：0x42 0x4D。如果它有所不同，第三方应用程序会认为该文件不包含bmp图片，尽管.bmp文件扩展名。

像素的大小和存储方式也有点more complex than what you expect：您假设每个像素的位数为24，并且不使用压缩。这不保证。如果不是这种情况，您可能会读取比可用数据更多的数据，并在写回文件时损坏文件。

此外，标头的大小还取决于您使用的BMP version，you can detect using偏移量为14的4字节整数。

改进您的代码

加载文件时，请检查签名，bmp版本，每像素位数和压缩。出于调试目的，请考虑转储标头以手动检查它：

for (int i=0; i<54; i++) 
    cout << hex << image.header[i] << " ";` 
cout <<endl;

此外，当您fread()检查读取的字节数是否与您想要读取的大小相对应时，请确保您没有使用未初始化的缓冲区数据。

编辑：

检查转储后，看起来格式符合预期。但是，使用您计算出的填充大小验证标题中的填充大小似乎是错误：

image.row_padded = (image.width * 3 + 3) & (~3);     // ok size of a single row rounded up to multiple of 4
image.pixels = new unsigned char[image.row_padded];  // oops !  A little short ?

实际上你是逐行读的，但你只保留最后一个在内存中！这与您的第一个版本不同，您可以在其中读取图片的完整像素。

同样，你写下最后一行重复的高度时间。

重新考虑您的填充，使用总填充尺寸。

image.row_padded = (image.width * 3 + 3) & (~3);     // ok size of a single row rounded up to multiple of 4
image.size_padded = image.row_padded * image.height;  // padded full size
image.pixels = new unsigned char[image.size_padded];  // yeah ! 
if (fread(image.pixels, sizeof(unsigned char), image.size_padded, f) !=  image.size_padded) {
    cout << "Error: all bytes couldn't be read"<<endl; 
}
else { 
    ... // process the pixels as expected
}
...

C ++：在WINDOWS上写入BMP图像格式错误

1 个答案: