Программа изменения размера изображения выводит искаженное изображение

Question

Я работаю над изменением изображения с помощью c. Предполагалось увеличить изображение на целочисленную шкалу от 1 до 100 включительно. Проблема, с которой я сталкиваюсь, заключается в том, что выходное изображение оказывается искаженной версией оригинала, и мне трудно определить, что его вызывает. Я попытался использовать printf для отслеживания всех функций и переменных, и все они, кажется, работают правильно. Происходит то, что для ввода изображения квадрата с другим цветным квадратом в середине я получаю вывод квадрата увеличения (по шкале), но квадрат в середине превратился в прямоугольник и переместился рядом с верхняя часть картины. Я чувствую, что проблема заключается во вложенных циклах. Любая помощь приветствуется. Пожалуйста, дайте мне знать, если вы хотите просмотреть снимок искажения.

Спасибо!

Вот код:

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

#include "bmp.h"

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

// check for number in first argument
for (int i = 0 , n = strlen(argv[1]); i < n; i++)
{
    int k = isdigit(argv[1][i]);
    if (k == 0)
    {
        fprintf(stderr, "Cannot use %s for scale. Please enter a valid value\n", argv[1]);
        return 1;
    }
}

//remember scale
int scale = atoi(argv[1]);

// check for value of first argument
if (scale > 100)
{
    fprintf(stderr, "%i is greater than 100\n", scale);
    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;
}

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

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

 // update header
BITMAPFILEHEADER outbf = bf;
BITMAPINFOHEADER outbi = bi;

//define new dimensions
outbi.biWidth = scale * bi.biWidth;
outbi.biHeight = scale * bi.biHeight;

//define new padding
int outpadding = (4 - (outbi.biWidth * sizeof(RGBTRIPLE)) % 4) % 4;

//define new sizes
outbi.biSizeImage = ((sizeof(RGBTRIPLE) * outbi.biWidth) + outpadding) * abs(outbi.biHeight);
outbf.bfSize = outbi.biSizeImage + bf.bfOffBits;

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

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

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

        // iterate over pixels in scanline
        for (int j = 0; j < bi.biWidth; j++)
        {
            // temporary storage
            RGBTRIPLE triple;

            // read RGB triple from infile
            fread(&triple, sizeof(RGBTRIPLE), 1, inptr);

            // write pixel per scale
            for (int w = 0; w < scale; w++)
            {
                // write RGB triple to outfile
                fwrite(&triple, sizeof(RGBTRIPLE), 1, outptr);
            }
        }

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

        // then add it back (to demonstrate how)
        for (int k = 0; k < outpadding; k++)
        {
            fputc(0x00, outptr);
        }
    }
}

// close infile
fclose(inptr);

// close outfile
fclose(outptr);

// success
return 0;

}

Answer 1

Вы читаете ввод для каждой повторяющейся горизонтальной линии, поэтому квадрат сместился вверх.

Он должен быть перемещен из петли вертикальной шкалы. Я предлагаю вам выделить память для одной строки сканирования, прочитать всю строку в буфер, а затем записать ее из буфера.

// iterate over infile's scanlines

// TEMPORARY STORAGE
RGBTRIPLE *buff = malloc (bi.biWidth * sizeof(RGBTRIPLE));

for (int i = 0, biHeight = abs(bi.biHeight); i < biHeight; i++)
{
    // THIS MOVED UP OUT OF THE LOOP
    // read RGB triple from infile
    fread(buff, sizeof(RGBTRIPLE), bi.biWidth, inptr);

    for (int h = 0; h < scale; h++)
    {
        // iterate over pixels in scanline
        for (int j = 0; j < bi.biWidth; j++)
        {
            // write pixel per scale
            for (int w = 0; w < scale; w++)
            {
                // write RGB triple to outfile - FROM BUFFER
                fwrite(&buff[j], sizeof(RGBTRIPLE), 1, outptr);
            }
        }

        // then add it back (to demonstrate how)
        for (int k = 0; k < outpadding; k++)
        {
            fputc(0x00, outptr);
        }
    }
    // MOVED DOWN OUT OF LOOP
    // skip over padding, if any
    fseek(inptr, padding, SEEK_CUR);
}
free(buff);

Код должен проверять возвращаемое значение из fread, количество прочитанных элементов. Поскольку это не так, вы не заметили, что данные больше не читаются, и ваш RGBTRIPLE triple продолжал хранить ранее прочитанные значения.