В вашем коде было две основные проблемы.
- Вы изменили ширину и высоту.
- Вы не учитывали
требуемый отступ в конце каждой строки, чтобы сделать его кратным
4 байта.
Вы также выделяете большой буфер, который вам не нужен, в зависимости от того, как был написан остальной код. Обычно я предпочитаю читать / обрабатывать одну полную строку за раз или даже полное изображение одновременно, но для этого вы хотите использовать malloc или calloc, потому что данные могут быть больше доступного стека. В этом случае, для простоты, я просто обрабатываю один пиксель за раз.
Я также избавился от getc / putc, потому что я предпочитаю fread / fwrite, и вы никогда не имеете дело с 1 байтом за раз.
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
int main()
{
FILE *fIn = fopen("tiger.bmp", "rb");
FILE *fOut = fopen("tiger_gray.bmp", "wb");
if (!fIn || !fOut)
{
printf("File error.\n");
return 0;
}
unsigned char header[54];
fread(header, sizeof(unsigned char), 54, fIn);
fwrite(header, sizeof(unsigned char), 54, fOut);
int width = *(int*)&header[18];
int height = abs(*(int*)&header[22]);
int stride = (width * 3 + 3) & ~3;
int padding = stride - width * 3;
printf("width: %d (%d)\n", width, width * 3);
printf("height: %d\n", height);
printf("stride: %d\n", stride);
printf("padding: %d\n", padding);
unsigned char pixel[3];
for (int y = 0; y < height; ++y)
{
for (int x = 0; x < width; ++x)
{
fread(pixel, 3, 1, fIn);
unsigned char gray = pixel[0] * 0.3 + pixel[1] * 0.58 + pixel[2] * 0.11;
memset(pixel, gray, sizeof(pixel));
fwrite(&pixel, 3, 1, fOut);
}
fread(pixel, padding, 1, fIn);
fwrite(pixel, padding, 1, fOut);
}
fclose(fOut);
fclose(fIn);
return 0;
}