CS50 pset4 调整大小不太舒服。标题似乎已正确更新，直到我写在文件上

Posted 2023-02-16

【中文标题】CS50 pset4 调整大小不太舒服。标题似乎已正确更新，直到我写在文件上

【英文标题】：CS50 pset4 resize less comfortable. Header seems to be updated correctly, until I write on the file

【发布时间】：2018-05-28 06:00:52

【问题描述】：

我想知道是否有人可以为我指出这个问题的正确方向（没有答案，只是提示问题所在）。该问题的想法是根据作为命令参数提供的整数来调整 BMP 图像的大小。如果数字为 2，则文件将是两倍大小（以像素为单位）。

我测试了我为更新标题而创建的代码，当我运行它时文件的大小是正确的。但是，在尝试调整像素大小后，文件的大小在某些情况下会急剧增加或减少到大约 50 字节。不知道是什么错误。

这是代码：

// resizes a BMP file

#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 integer (1-100) infile outfile\n");
        return 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;
    

    int n= atoi(argv[1]);

    // 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;
    

    // modify bitmapinfoheader for height an width

    bi.biWidth = bi.biWidth*n;
    bi.biHeight = bi.biHeight*n;

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

    // modify bisizeimage
    bi.biSizeImage = (((bi.biWidth*(sizeof(RGBTRIPLE)))+padding)*abs(bi.biHeight));

    // write outfile's BITMAPFILEHEADER
    fwrite(&bf, sizeof(BITMAPFILEHEADER), 1, outptr);

    // modify bfsize before writing it
    bf.bfSize = (sizeof(BITMAPFILEHEADER) + sizeof(BITMAPINFOHEADER) + sizeof(bi.biSizeImage));

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

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

        // temporary storage for a triple
        RGBTRIPLE triple;
        // temporary storage for scanline
        RGBTRIPLE scanline[bi.biWidth];
        //array position counter
        int arraypos = 0;

        // iterate over pixels in scanline
        for (int j = 0; j < bi.biWidth; j++)
        
            // read RGB triple from infile
            fread(&triple, sizeof(RGBTRIPLE), 1, inptr);


            //write to array n times
            for (int l = 0; l < n; arraypos++, l++)
            
                scanline[arraypos] = triple;

            
        
        for (int m = 0; m < n; m++)
        
             // write array to outfile
            fwrite(&scanline, sizeof(scanline), 1, outptr);
            // Add padding)
            for (int k = 0; k < padding; k++)
            
                fputc(0x00, outptr);
            
        

        // skip over padding, if any
        fseek(inptr, padding, SEEK_CUR);
    

    // close infile
    fclose(inptr);

    // close outfile
    fclose(outptr);

    // success
    return 0; 

【问题讨论】：

欢迎来到 Stack Overflow！寻求调试帮助的问题（“为什么这段代码不起作用？”）必须包括所需的行为、特定的问题或错误以及在问题本身中重现它所需的最短代码（参见@987654321 @)。没有明确问题陈述的问题对其他读者没有用处，很可能不会得到回答。 有关更多帮助，请查看“How to ask”

您应该在调试器中运行您的程序并检查使用的值。提示：bi.biWidth = bi.biWidth*n; 的位置是您的问题的一部分。

【参考方案1】：

bf.bfSize = (sizeof(BITMAPFILEHEADER) + sizeof(BITMAPINFOHEADER) +
    sizeof(bi.biSizeImage));

sizeof(bi.biSizeImage) 就是 sizeof(int)。这应该是文件大小，所以你需要bi.biSizeImage 本身。

另一个问题是你改变了宽度和高度，但你没有保存旧的宽度和高度。如果新的宽度和高度更大，那么您将到达文件末尾。

如下更改，同时添加fread的错误检查

int width_save = bi.biWidth;
int height_save = bi.biHeight;

bi.biWidth *= n;
bi.biHeight *= n;

//calculate width in bytes
int wb_in = ((width_save * 24 + 31) / 32) * 4;
int wb_out = ((bi.biWidth * 24 + 31) / 32) * 4;

bi.biSizeImage = wb_out * abs(bi.biHeight);
bf.bfSize = 54 + bi.biSizeImage;

fwrite(&bf, sizeof(BITMAPFILEHEADER), 1, outptr);
fwrite(&bi, sizeof(BITMAPINFOHEADER), 1, outptr);

RGBTRIPLE *line_in = malloc(wb_in); 
RGBTRIPLE *line_out = malloc(wb_out);

for(int i = 0; i < abs(height_save); i++)

    fread(line_in, 1, wb_in, inptr);

    for(int a = 0; a < width_save; a++)
        for (int b = 0; b < n; b++)
            line_out[a * n + b] = line_in[a];

    for(int a = 0; a < n; a++)
        fwrite(line_out, 1, wb_out, outptr);

【讨论】：

感谢您的回答！老实说，你回答的某些部分让我感到困惑，但它的开头有解决我的代码的线索。我正在用新值迭代 infile 的像素和填充。

sizeof(BITMAPFILEHEADER) 总是 14，sizeof(BITMAPINFOHEADER) 总是 40。总数是 54，我忘了解释。我使用了一个公式来计算 “字节宽度”，它确保该值始终是 4 的倍数。您使用了类似的方法，将字节宽度计算为“宽度 * 3 + 填充”，注意sizeof(RGBTRIPLE) 也应该是 3。