我试图在CProgramming论坛上回答我的问题,但是他们的数据库存在一些问题。
我有这些结构:
typedef struct QuadImageInfo *QuadImage;
typedef struct QuadImageInfo
{
float whiteness;
QuadImage nw, ne, sw, se;
} QuadImageInfo;
我有以下功能:
void LoadQuadImageAux(FILE *fp, QuadImage img)
{
int pos;
QuadImage ptr[4], loaded = NewQuadImage(0.0);
SetPointersToQuadrants(&img, ptr);
for(pos = 0; pos < 4; pos++)
{
switch(fgetc(fp))
{
case '0' : ptr[pos] = NewBlackQuadImage(); break;
case '1' : ptr[pos] = NewWhiteQuadImage(); break;
case 'Q' : LoadQuadImageAux(fp, ptr[pos] = loaded);
}
}
QuadImage LoadQuadImage(FILE *fp)
{
QuadImage loaded = NewQuadImage(0.0);
switch(fgetc(fp))
{
case '0' : loaded = NewBlackQuadImage(); break;
case '1' : loaded = NewWhiteQuadImage(); break;
case 'Q' : LoadQuadImageAux(fp, loaded);
}
return loaded;
}
如您所见,我在
QuadImage loaded
中创建一个LoadQuadImage
并将其传递给LoadQuadImageAux
。我的目标是访问/修改loaded
的4个内部QuadImage,然后在LoadQuadImage
中返回它。有一种简单的方法,其中包括4次“重复”代码(每个QuadImage一个)。
我想创建一个指向这4个QuadImage的指针数组,并通过
for loop
中的LoadQuadImageAux
访问/修改它们。生成
QuadImage pointers array
的代码是:void SetPointersToQuadrants(QuadImage *img, QuadImage *ptr)
{
ptr[0] = (*img)->nw;
ptr[1] = (*img)->ne;
ptr[2] = (*img)->sw;
ptr[3] = (*img)->se;
}
但是,由于某些未知原因(对我而言),
loaded
总是以NewQuadImage(0.0)
结尾(这意味着它未被修改)。编译代码here。
要运行此
compiling code
,请创建一个名为1.qad
的文件,其内容为Q00Q11111
。如果程序最后打印
Q00Q11111
,则表示它正在运行。感谢
dmckee
,帮助我格式化帖子。有小费吗?
最佳答案
在LoadQuadImageAux
中,基本上,您正在这样做:
ptr[0] = img->nw;
//these are now two separate pointers that point to the same location
ptr[0] = NewBlackQuadImage();
//this changes the location pointed to by ptr[0]
//it doesn't change the location pointed to by img->nw
//and it doesn't change the contents of the location pointed to by img->nw
您正在循环并更改保存在
ptr
数组中的指针值,但是您从未对通过引用传递的结构进行实际更改。我认为您想要做的只是:
img->nw = NewBlackQuadImage();
并且您使用
ptr
数组(大概是为了使循环更容易编码)会引起混乱。我的建议是完全放弃
ptr
并弄清楚如何编写代码以正确使用img
指针。