Question

我正在编写一个程序，该程序接收两个重复的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");
}

Answer 1

小心写与阅读完全相同的格式。 PNM格式为well defined，您正在正确阅读。然而，在写作程序中有一些错误：

打开文件"w"或"wb"将其截断为0字节;
最佳做法是始终检查fopen是否成功;
读取实际ASCII数据可以使用fscanf，二进制数据fread来完成。同样，编写ASCII应该使用fprintf，而只能使用fwrite编写二进制数据。
如果您想确保编写之前读过的相同数据，则需要将其保存在某处。 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);
}

通过上述方法，您现在可以看到对角线＆＃39;是不正确的！我不打算解决这个问题（我想我们无法看到发生的事情会阻止你在你的轨道上），但这里有一些指示只是为了指导你：

如果您正在更改数据 in situ

，则无需返回Image *
无需检查每个单个像素

检查更改像素的坐标...

编写PNM P6的问题

1 个答案: