0

我正在编写一个程序，它接收两个重复的 PNM P6 文件，将第一个文件的内存放入缓冲区，在其上创建一条黄色对角线，并将结果写入第二个文件。当我运行它时，输出文件已损坏并且无法显示。我在查看输出时注意到它缺少应该在顶部的三行：

P6
1786 1344
255

我不知道如何以编程方式确保这些行保留在代码中——我无法弄清楚为什么它们一开始就被覆盖了。

当我手动将这些行添加到输出时，文件不再损坏。但是，没有出现黄色对角线。这是同一问题的一部分，还是我应该考虑解决其他问题？

我的代码：

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

typedef struct
{
    unsigned char red, green, blue;
} iPixel;

typedef struct
{
    int x, y;
    iPixel *data;

} Image;

Image *
ReadImage(char *filename)
{
    FILE *f = fopen(filename, "rb");
    Image *img = NULL;
    char magicNum[128];
    int  width, height, maxval;

    if (f == NULL)
    {
        fprintf(stderr, "Unable to open file %s\n", filename);
        return NULL;
    }

    fscanf(f, "%s\n%d %d\n%d\n", magicNum, &width, &height, &maxval);
/*
    printf("Magic num = %s width = %d, height = %d, maxval = %d\n",
            magicNum, width, height, maxval)
; 
*/
    if (strcmp(magicNum, "P6") != 0)
    {
        fprintf(stderr, "Unable to read from file %s, because it is not a PNM file of type P6\n", filename);
        return NULL;
    }

    img = (Image *) malloc(sizeof(Image));
    img -> x = width;
    img -> y = height;
    img -> data = (iPixel*) malloc(img -> x * img -> y * sizeof(iPixel));

    fread(img -> data, sizeof(iPixel), width*height, f);

    fclose(f);
    return img;

}

void WriteImage(Image *img, char *filename)
{
    FILE *f = fopen(filename, "wb");

    fwrite(img->data, sizeof(iPixel), img-> x * img-> y, f);
    fclose(f);
}

Image *
YellowDiagonal(Image *input)
{

    int i, j;

    for (i = 0; i < input->x; i++)
    {
        for (j=0; j < input->y; j++)
        {
            if (i==j)
            {
                input->data[i].red=255;
                input->data[i].green=255;
                input->data[i].blue=0;
            }
        }
    }

    return input;   
}

int main(int argc, char *argv[])
{
    if (argc != 3)
    {
        fprintf(stderr, "Usage: ./3A_c.c <input image file> <output image file>\n");
        exit(EXIT_FAILURE);
    }

        Image *img;

        img = ReadImage(argv[1]);
        fprintf(stderr, "Read.\n");
        YellowDiagonal(img);
        fprintf(stderr, "Diagonal line.\n");
        WriteImage(img, argv[2]);
        fprintf(stderr, "Write.\n");
}
4

1 回答 1

1

小心写出与您正在阅读的完全相同的格式。PNM 格式定义明确，您阅读正确。然而，在写作过程中，有几个错误：

  1. 打开文件时将"w"其"wb"截断为 0 字节；
  2. 最佳实践始终是检查是否fopen成功；
  3. 读取实际的 ASCII 数据可以用fscanf，二进制数据用fread. 类似地，写入 ASCII 应该使用fprintf并且仅使用二进制数据再次使用fwrite.
  4. 如果要确保写入的数据与之前读入的数据相同，则需要将其保存在某个地方。该maxval变量已读取，但未保存，因此我无法将其写回。但是，这不是一个大问题，因为您的其余代码无论如何都假定图像是 R8G8B8，因此maxval应该始终是255.

这是一个WriteImage有效的调整。

void WriteImage(Image *img, char *filename)
{
    FILE *f = fopen(filename, "wb");

    if (f == NULL)
    {
        printf ("Unable to open '%s' for writing!\n", filename);
     /* better would be: "return -1" to indicate an error, 0 otherwise */
        return;
    }

    fprintf (f, "P6\n");
    fprintf (f, "%d %d\n", img->x, img->y);
/*  actually you need to write 'maxval' here */
    fprintf (f, "%d\n", 255);

    fwrite(img->data, sizeof(iPixel), img->x * img->y, f);
    fclose(f);
}

排除上述问题后，您现在可以看到您的“对角线”不正确！我不打算解决这个问题（我想无法看到发生的事情阻止了你的脚步），但这里有一些指导你的建议：

  • Image *如果您在原地更改数据，则无需返回
  • 无需检查每个像素
  • 检查更改了哪些像素的坐标...
于 2014-05-12T08:53:30.360 回答