c - C编程问题:在函数中传递，创建和返回结构。

这个函数不起作用，我不明白为什么。它编译得很好，程序似乎也在运行，但是经过仔细检查和调试，我发现：

newImg->x = b;
newImg->y = a;

实际上不起作用而且会引起问题。我尝试使用newImg=img进行复制，但这不允许我以后更改newImg的值。它们保持不变。我还尝试修改img的值，然后执行newImg，但是调试表明newImg得到了极端值。
结构如下：

typedef struct
{
     unsigned char grayscale;
} PGMPixel;

typedef struct
{
     int x, y;
     PGMPixel *data;
} PGMImage;

功能如下：

static PGMImage *rotatePGM(PGMImage *img)
{
    PGMImage *newImg;


    // Memory allocation for pgm
    newImg = (PGMImage *)malloc(sizeof(PGMImage));
    if (!newImg)
    {
         fprintf(stderr, "Unable to allocate memory\n");
         exit(1);
    }

    //memory allocation for pixel data
    newImg->data = (PGMPixel*)malloc(newImg->x * newImg->y * sizeof(PGMPixel));
    if (!newImg)
    {
         fprintf(stderr, "Unable to allocate memory\n");
         exit(1);
    }

    int a = img->x;
    int b = img->y;
    newImg->x = b;
    newImg->y = a;

    int u = a - 1;
    int v = b - 1;
    int i = 0;
    int j = 0;

    if(newImg)
    {
        for (i = 0; i < a; i++)
        {
            for (j = 0; j < b; j++)
            {
                img->data[(j*a)+(u-i)].grayscale = img->data[(i*b)+j].grayscale;
            }
        }
    }
    return newImg;
}

如果有帮助的话，我将使用MinGW GCC和windows 8。

最佳答案

线路

newImg->data = (PGMPixel*)malloc(newImg->x * newImg->y * sizeof(PGMPixel));

是错误的-它在初始化之前使用newImg->x和newImg->y。您应该使用img中的值来代替

newImg->data = malloc(img->x * img->y * sizeof(PGMPixel));

我对那条线路又做了一个小改动-you don't need to cast the return from malloc
在后面的行中还使用了错误的PGMPixel实例

img->data[... = img->data[...

（大概应该是您分配给的newImg->data）

关于c - C编程问题:在函数中传递，创建和返回结构。，我们在Stack Overflow上找到一个类似的问题：https://stackoverflow.com/questions/17774272/