C按N因子调整BMP文件大小,我做错了什么?

时间:2017-03-31 16:01:06

标签: c image algorithm resize

C的新手一直在用我的代码调整24位未压缩位图的大小。我试图将这个图像缩小n倍,我觉得我接近它正常工作,但我输出的图像仍然不正确。

我可以发布输入24位未压缩BMP(small.bmp)的图片我用于测试,我的程序正在输出的图片(resized.bmp),以及由一个因子缩放的small.bmp的正确图像4应该看起来像,如果这将是有帮助的。请问。

resize.c

#include <stdio.h>
#include <stdlib.h>

#include "bmp.h"

int main(int argc, char *argv[])
{
    // ensure proper usage
    if (argc != 4)
    {
        fprintf(stderr, "Usage: ./resize scale infile outfile\n");
        return 1;
    }

    int n = atoi(argv[1]);
    // remember filenames
    char *infile = argv[2];
    char *outfile = argv[3];

    // open input file 
    FILE *inptr = fopen(infile, "r");
    if (inptr == NULL)
    {
        fprintf(stderr, "Could not open %s.\n", infile);
        return 2;
    }

    // open output file
    FILE *outptr = fopen(outfile, "w");
    if (outptr == NULL)
    {
        fclose(inptr);
        fprintf(stderr, "Could not create %s.\n", outfile);
        return 3;
    }

    // read infile's BITMAPFILEHEADER
    BITMAPFILEHEADER bf;
    fread(&bf, sizeof(BITMAPFILEHEADER), 1, inptr);


    // read infile's BITMAPINFOHEADER
    BITMAPINFOHEADER bi;
    fread(&bi, sizeof(BITMAPINFOHEADER), 1, inptr);

    // ensure infile is (likely) a 24-bit uncompressed BMP 4.0
    if (bf.bfType != 0x4d42 || bf.bfOffBits != 54 || bi.biSize != 40 || 
        bi.biBitCount != 24 || bi.biCompression != 0)
    {
        fclose(outptr);
        fclose(inptr);
        fprintf(stderr, "Unsupported file format.\n");
        return 4;
    }
    int oldpadding = (4 - (bi.biWidth * sizeof(RGBTRIPLE)) % 4) % 4;
    bi.biWidth = bi.biWidth * n;
    bi.biHeight = bi.biHeight * n;

    // determine padding for scanlines
    int padding = (4 - (bi.biWidth * sizeof(RGBTRIPLE)) % 4) % 4;

bi.biSizeImage = ((sizeof(RGBTRIPLE) * bi.biWidth) + padding) * abs(bi.biHeight);
bf.bfSize = bi.biSizeImage + sizeof(BITMAPFILEHEADER) + sizeof(BITMAPINFOHEADER);
// write outfile's BITMAPFILEHEADER
fwrite(&bf, sizeof(BITMAPFILEHEADER), 1, outptr);

    // write outfile's BITMAPINFOHEADER
    fwrite(&bi, sizeof(BITMAPINFOHEADER), 1, outptr);


    // iterate over infile's scanlines
    for (int i = 0, biHeight = abs(oldHeight); i < biHeight; i++)
    {

        // store scanline in an array pixel by pixel for vertical scaling.
        RGBTRIPLE scanline[bi.biWidth - 1];

        // iterate over pixels in scanline
        for (int j = 0; j < bi.biWidth; j++)
        {
            //check if we've hit padding in original bmp.
            if (j % oldpadding == 0 && j != 0) 
            {
                //skip the padding.
                fseek(inptr, oldpadding, SEEK_CUR);
            }

            // temporary storage
            RGBTRIPLE triple;

            fread(&triple, sizeof(RGBTRIPLE), 1, inptr);

            for (int h = 0; h < n; h++)
            {
                //scale horizontally, save each scanline pixel to our array.
                scanline[j] = triple;
            }

        }

        for (int x = 0; x < n; x++)
        {
            //write scanlines n - 1 times.
            for (int y = 0; y < bi.biWidth; y++)
            {
                    fwrite(&scanline[y], sizeof(RGBTRIPLE), 1, outptr);
            }

            //write padding if any for current scanline.
            for (int z = 0; z < padding; z++)
            {
                fputc(0x00, outptr);
            }
        }

    }

    // close infile
    fclose(inptr);

    // close outfile
    fclose(outptr);

    // success
    return 0;
}

bmp.h

/**
 * BMP-related data types based on Microsoft's own.
 */

#include <stdint.h>

/**
 * Common Data Types 
 *
 * The data types in this section are essentially aliases for C/C++ 
 * primitive data types.
 *
 * Adapted from https://msdn.microsoft.com/en-us/library/cc230309.aspx.
 * See http://en.wikipedia.org/wiki/Stdint.h for more on stdint.h.
 */
typedef uint8_t  BYTE;
typedef uint32_t DWORD;
typedef int32_t  LONG;
typedef uint16_t WORD;

/**
 * BITMAPFILEHEADER
 *
 * The BITMAPFILEHEADER structure contains information about the type, size,
 * and layout of a file that contains a DIB [device-independent bitmap].
 *
 * Adapted from https://msdn.microsoft.com/en-us/library/dd183374(v=vs.85).aspx.
 */
typedef struct 
{ 
    WORD bfType; 
    DWORD bfSize; 
    WORD bfReserved1; 
    WORD bfReserved2; 
    DWORD bfOffBits; 
} __attribute__((__packed__)) 
BITMAPFILEHEADER; 

/**
 * BITMAPINFOHEADER
 *
 * The BITMAPINFOHEADER structure contains information about the 
 * dimensions and color format of a DIB [device-independent bitmap].
 *
 * Adapted from https://msdn.microsoft.com/en-us/library/dd183376(v=vs.85).aspx.
 */
typedef struct
{
    DWORD biSize; 
    LONG biWidth; 
    LONG biHeight; 
    WORD biPlanes; 
    WORD biBitCount; 
    DWORD biCompression; 
    DWORD biSizeImage; 
    LONG biXPelsPerMeter; 
    LONG biYPelsPerMeter; 
    DWORD biClrUsed; 
    DWORD biClrImportant; 
} __attribute__((__packed__))
BITMAPINFOHEADER; 

/**
 * RGBTRIPLE
 *
 * This structure describes a color consisting of relative intensities of
 * red, green, and blue.
 *
 * Adapted from https://msdn.microsoft.com/en-us/library/dd162939(v=vs.85).aspx.
 */
typedef struct
{
    BYTE rgbtBlue;
    BYTE rgbtGreen;
    BYTE rgbtRed;
} __attribute__((__packed__))
RGBTRIPLE;

1 个答案:

答案 0 :(得分:1)

您的代码混合了两种缩放技术:1)在数组中缩放然后将其写出来; 2)在写出数据时,读取数组中的原始大小和缩放。但是,您对第一种技术的实现并没有实际进行任何缩放:

for (int h = 0; h < n; h++)
{
    //scale horizontally, save each scanline pixel to our array.
    scanline[j] = triple;
}

因为你没有增加j,所以你只需要将相同的三倍重叠到相同的位置n次。它根本不做任何缩放。我建议放弃第一种技术,只关注第二种技术。对于初学者,修复你的数组声明:

RGBTRIPLE scanline[bi.biWidth];

然后当你读入时,只需将三元组写入正确的位置而不进行任何循环:

scanline[j] = triple;

当你把它写出来时,你的循环就会出错了。你想循环像素并将每个像素写出n次,而不是在图像上循环n次并写出所有像素(这将重复图像n次而不是按n缩放)。然后将这个全部包装在循环中以输出每一行n次。

for (int y = 0; y < n; y++) // repeat each row n times
{
    for (int x = 0; x < bi.biWidth; x++) // iterate over pixels
    {
         for (int r = 0; r < n; r++) // repeat each pixel n times
             fwrite(&scanline[x], sizeof(RGBTRIPLE), 1, outptr);
    }
    // write padding if any for current scanline.
    for (int z = 0; z < padding; z++)
    {
        fputc(0x00, outptr);
    }
}

这是缩放图像的基本算法。您可以通过执行块I / O读取和写入来提高速度,而不是在循环中调用fwrite,但正如您在注释中指出的那样,正确性&gt;效率。你可能也有一些填充问题,我还没有解决这个问题。