我的程序解码图像有什么问题?

时间:2013-06-02 00:57:16

标签: c image pixel decode ppm

我给了一个图像文件,真实图像隐藏在随机像素后面。我必须通过将红色值乘以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);
}

2 个答案:

答案 0 :(得分:1)

  1. 您在output之前使用fopen

  2. 你的while循环是infinte,因为你只执行check = fscanf(input, "%d\n", &value);一次。你可能意味着:

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

答案 1 :(得分:0)

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

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");
 [...]