Кодирование проблемы с помощью 2-го массива структур в другой структуре в C

Кажется, вы можете создавать массивы переменной длины, поэтому вы работаете в системе C99 или в системе, которая ее поддерживает. Но не все компиляторы их поддерживают. Если вы хотите их использовать, вам не нужно объявление указателя arr в вашей структуре. Предполагая, что нет массивов переменной длины, давайте посмотрим на соответствующие части вашего кода:

/*data for each image*/
typedef struct
{ 
    int height;
    int width;
    pixelStruct *arr; /*pointer to 2-d array of  pixels*/
} imageStruct;

arr - это указатель на pixelStruct , а не на двумерный массив пикселей. Конечно, вы можете использовать arr для доступа к такому массиву, но комментарий вводит в заблуждение и намекает на недоразумение. Если вы действительно хотите объявить такую ​​переменную, сделайте что-то вроде:

pixelStruct (*arr)[2][3];

и arr будет указателем на «массив 2 из массива 3 из pixelStruct», что означает, что arr указывает на двумерный массив. Это не совсем то, что вам нужно. Честно говоря, вы не об этом заявляете, так что все хорошо. Но ваш комментарий предполагает неправильное понимание указателей в C, и это проявляется позже в вашем коде.

На этом этапе вам будет полезно прочитать хорошее введение в массивы и указатели в C, и действительно хорошее C для умников: массивы и указатели Криса Торека. В частности, убедитесь, что вы понимаете первую диаграмму на странице и все, что есть в определении функции f .

Поскольку вы хотите иметь возможность индексировать arr в Естественным образом используя индексы «столбец» и «строка», я предлагаю вам объявить arr как указатель на указатель. Итак, ваша структура становится:

/* data for each image */
typedef struct
{ 
    int height;
    int width;
    pixelStruct **arr; /* Image data of height*width dimensions */
} imageStruct;

Затем в вашей функции ReadImage вы выделяете необходимую память:

int i;
thisImage.arr = malloc(thisImage.height * sizeof *thisImage.arr);
for (i=0; i < thisImage.height; ++i)
    thisImage.arr[i] = malloc(thisImage.width * sizeof *thisImage.arr[i]);

Обратите внимание, что для ясности я не проверял ошибки на malloc . На практике вам следует проверить, вернул ли malloc NULL , и принять соответствующие меры.

Предполагая, что вся память выделена успешно, теперь вы можете прочитать свое изображение в thisImage.arr (точно так же, как вы делали для imageArr в исходной функции).

Как только вы закончите с thisImage.arr , обязательно освободите его:

for (i=0; i < thisImage.height; ++i)
    free(thisImage.arr[i]);

free(thisImage.arr);

На практике вам может потребоваться обернуть указанные выше части выделения и освобождения в соответствующие функции, которые выделяют и освобождают объект arr , и заботятся о проверке ошибок.

int i;
thisImage.arr = malloc(thisImage.height * sizeof *thisImage.arr);
for (i=0; i < thisImage.height; ++i)
    thisImage.arr[i] = malloc(thisImage.width * sizeof *thisImage.arr[i]);

Обратите внимание, что для ясности я не проверял ошибки на malloc . На практике вы должны проверить, вернул ли malloc NULL , и принять соответствующие меры.

Если вся память выделена успешно, теперь вы можете прочитать свое изображение в thisImage.arr (точно так же, как вы делали для imageArr в исходной функции).

Как только вы закончите с thisImage.arr , обязательно освободите его:

for (i=0; i < thisImage.height; ++i)
    free(thisImage.arr[i]);

free(thisImage.arr);

На практике вам может потребоваться обернуть указанные выше части выделения и освобождения в соответствующие функции, которые выделяют и освобождают объект arr , и заботятся о проверке ошибок.

int i;
thisImage.arr = malloc(thisImage.height * sizeof *thisImage.arr);
for (i=0; i < thisImage.height; ++i)
    thisImage.arr[i] = malloc(thisImage.width * sizeof *thisImage.arr[i]);

Обратите внимание, что для ясности я не проверял ошибки на malloc . На практике вы должны проверить, вернул ли malloc NULL , и принять соответствующие меры.

Если вся память выделена успешно, теперь вы можете прочитать свое изображение в thisImage.arr (точно так же, как вы делали для imageArr в исходной функции).

Как только вы закончите с thisImage.arr , обязательно освободите его:

for (i=0; i < thisImage.height; ++i)
    free(thisImage.arr[i]);

free(thisImage.arr);

На практике вам может потребоваться обернуть указанные выше части выделения и освобождения в соответствующие функции, которые выделяют и освобождают объект arr , и заботятся о проверке ошибок.