带指针的 C 深拷贝结构

Question

我是 C 新手，需要解决以下问题。

我有一个项目正在读取一个小的 bmp (512x512) 图像。我已经设法改变颜色并将其镜像（水平和垂直）。虽然我现在需要它转-90°。我无法使用的功能是 deepCopyBitmap()。

我在*copy->raster[i] 上不断收到以下错误：

indirection requires pointer operand ('PIXEL' (aka 'struct _pixel') invalid)

旋转 (512x512)

typedef struct _pixel {
    unsigned char blue;
    unsigned char green;
    unsigned char red;
} PIXEL;

typedef struct _bitmap {
    char file_path[PATH_MAX+1];
    char magic_number[3];
    unsigned int size;
    unsigned char application[5];
    unsigned int start_offset;
    unsigned int bitmapHeaderSize;
    unsigned int width;
    unsigned int height;
    unsigned short int depth;
    unsigned  char* header;
    PIXEL* raster;
} BITMAP;

void rotate(BITMAP* bmp) {
    int i;
    int j;
    PIXEL* originalPixel;
    BITMAP* originalBmp;

    deepCopyBitmap(bmp, originalBmp);

    for(j=1; j <= bmp->height; j++) {
        for(i=1; i <= bmp->width; i++) {
            originalPixel=getPixel(originalBmp->raster, bmp->width, bmp->height, j, i);
            setPixel(bmp->raster, bmp->width, bmp->height, (bmp->width + 1 - i), j, originalPixel);
        }
    }                  
}

void deepCopyBitmap(BITMAP* bmp, BITMAP* copy) {
    *copy = *bmp;
    if (copy->raster) {
        copy->raster = malloc(sizeof(*copy->raster));
        for (int i = 0; i < copy->height; i++) {
            copy->raster[i] = malloc(sizeof(*copy->raster[i]));
            memcpy(copy->raster[i], bmp->raster[i], sizeof(*copy->raster[i]));
        }
    }
}

更新

void deepCopyBitmap(BITMAP* bmp, BITMAP* copy) {
    copy = malloc(sizeof(BITMAP));
    *copy = *bmp;
    if (copy->raster) {
        size_t total_size = copy->height * copy->width * sizeof(PIXEL);
        copy->raster = malloc(total_size);
        memcpy(copy->raster, bmp->raster, total_size);
    }
}

Answer 1

你这里只分配一个PIXEL：

        copy->raster = malloc(sizeof(*copy->raster));

但是，您至少需要 copy->height PIXELs 才能使此迭代生效：

        for (int i = 0; i < copy->height; i++) {

这里，你再分配一个PIXEL：

            copy->raster[i] = malloc(sizeof(*copy->raster[i]));

但是，你可能打算复制copy->widthPIXELs，而不是一个：

            memcpy(copy->raster[i], bmp->raster[i], sizeof(*copy->raster[i]));
        }

你真正想做的是分配copy->height * copy->widthPIXELs，然后从原来的复制它们。

        size_t total_size = copy->height * copy->width * sizeof(PIXEL);
        copy->raster = malloc(total_size);
        memcpy(copy->raster, bmp->raster, total_size);