Question

我是编程的新手，所以这可能有一个简单的解决方案。这是我想要做的事情。

我的程序加载.bmp图像，然后获取宽度和高度以找出图像有多少像素。然后它使用calloc（）为每个像素的RGB值制作三个数组，其中包括RED，BLUE，GREEN的数组。我使用了while（）循环来获取第一个像素并将RGB值放在它们各自数组的第一个元素中。然后它继续前进并为第二个像素，第三个，第四个像素做同样的事情......这就是我的问题所在，它似乎并不想将值放在数组中。我是这样做的，

while(temp=fgetc(fp) != NULL)
{
BLUE[current_pixel]=temp;

temp=fgetc(fp);
GREEN[current_pixel]=temp;

temp=fgetc(fp);
RED[current_pixel]=temp;

current_pixel++;
}

current_pixel是一个变量，可以跟踪我现在正在看的像素。

所以我猜我真正想知道的是为什么我不能这样做 BLUE [current_pixel] =温度; 编译时我没有收到任何错误，我使用printf（）语句来检查问题所在。

我试过了 BLUE [1] =温度并且它工作正常，但这对我的程序没有好处，因为我无法继续下一个像素来保存值。

提前感谢您的帮助！

编辑：我仍然无法让它工作，所以我只是在这里发布整个程序。

#include <stdio.h>
#include <stdlib.h>
int main()
{
FILE*fp;
int current_pixel=0;
int temp=0;
int paddingremoved=0;
int cycle=0;
int *RED;
int *BLUE;
int *GREEN;
int imgstrt=0;
int width=0;
int height=0;
int padding=0;

fp=fopen("C:\\Users\\Jason\\Documents\\test.bmp","rb");

if(fp==NULL)
{
printf("Error: File could not be opened");
getchar();
return(0);
}


fseek(fp,10,SEEK_SET);
fread(&imgstrt,1,1,fp);
printf("Image Starts At:%d\n",imgstrt);

fseek(fp,18,SEEK_SET);
fread(&width,4,1,fp);
printf("Image Width:%d\n",width);

fseek(fp,22,SEEK_SET);
fread(&height,4,1,fp);  
printf("Image Height:%d\n",height);

padding=(4 -(width*3)%4)%4;
printf("Padding:%d\n",padding);
getchar();

RED = (int*)calloc(height*width+1,sizeof(int));
GREEN = (int*)calloc(height*width+1,sizeof(int));
BLUE = (int*)calloc(height*width+1,sizeof(int));

if(RED == NULL)
{printf("Red Allocation Faliure\n");}
else{printf("Red Allocation Successful\n");}

if(GREEN == NULL)
{printf(" Green Allocation Faliure\n");}
else{printf("Green Allocation Successful\n");}

if(BLUE == NULL)
{printf("Blue Allocation Faliure\n");}
else{printf("Blue Allocation Successful\n");}

fseek(fp,54,SEEK_SET);
/*---------------------Main Loop--------------------------------*/
while((temp = fgetc(fp)) != EOF)
{
    BLUE[current_pixel]=temp;
    temp=fgetc(fp);
    GREEN[current_pixel]=temp;
    temp=fgetc(fp);
    RED[current_pixel]=temp;
    cycle++;
    current_pixel++;

    printf("[%d,%d,%d::%d] ",RED[current_pixel],GREEN[current_pixel],BLUE[current_pixel],current_pixel);
/*------------------------------------------------------------------*/

    /*---------------------Padding Remover-----------------------*/
    if(cycle==width)
    {printf("\n");
    while (paddingremoved!=padding)
    {fgetc(fp); 
                paddingremoved++;}
    cycle=0;
    paddingremoved=0;}
    /*-----------------------------------------------------------*/
}

getchar();
free(RED);
free(BLUE);
free(GREEN);
return(0);
}

Answer 1

由于C运算符优先级，您的while条件读作：

temp=(fgetc(fp) != NULL)

而不是

(temp=fgetc(fp)) != NULL

使用parens。此外，fgetc在错误/完成时返回EOF，而不是NULL。

Answer 2

由于运算符优先级，您的代码始终将每个BLUE像素设置为1。 while（）循环条件将fgetc(fp)的结果与NULL进行比较，然后将temp设置为该比较的结果（为0或1）。因此每个蓝色像素都将设置为1。

请改为尝试：

while((temp = fgetc(fp)) != EOF)

在C中访问数组的变量元素

2 个答案: