c - 我的图像解码程序有什么问题？

Question

我得到了一个图像文件，真实图像隐藏在随机像素后面。我必须通过将红色值乘以 10 并将绿色/蓝色值设置为等于新的红色值来解码图像。（而且我不能超过颜色的最大值 255）当我运行这个程序时，它应该创建一个名为“hidden.ppm”的输出文件。我运行了我的程序，但我得到的只是“分段错误”，我不知道为什么。

void print_pixel(int a)
{
   int r, g, b;

   r = a * 10;
   g = r;
   b = r;

   if (r > 255)
   {
      r = 255;
   }

   if (g > 255)
   {
      g = 255;
   }

   if (b > 255);
   {
      b = 255;
   }

   printf("%d\n", r);
   printf("%d\n", g);
   printf("%d\n", b);
}

void decode(int arg_list, char *in[])
{
   FILE *input, *output;
   int check, value;

   fprintf(output, "P3\n");
   fprintf(output, "%d %d\n", 1024, 768);
   fprintf(output, "255\n");

   input = fopen(in[1], "r");
   output = fopen("hidden.ppm", "w");

   check = fscanf(input, "%d\n", &value);

   while (check != EOF)
   {
      print_pixel(value);
   }
}

int main(int argc, char *argv[])
{
   if (argc == 0)
   {
      printf("usage: a.out <input file>\n");
      return 1;
   }

   decode(argc, argv);
}

Answer 1

1. 你在使用output之前使用fopen它。

2. 您的 while 循环是无限的，因为您只执行check = fscanf(input, "%d\n", &value);一次。你可能的意思是：

   do{
       check = fscanf(input, "%d\n", &value);
       print_pixel(value);
   while(check != EOF);
   

Answer 2

在您的decode实现中，您在初始化之前访问输出流。将输出文件的开头移到fprintf's 上方。

void decode(int arg_list, char *in[])
{
   FILE *input, *output;
   int check, value;

   input = fopen(in[1], "r");
   output = fopen("hidden.ppm", "w");

   fprintf(output, "P3\n");
 [...]