Dynamiczne tablice dwuwymiarowe w osobnej funkcji w C

Question

Chciałbym by dynamiczna alokacja pamięci odbywała się w osobnej funkcji.

Napisałem taką funkcję

void Alokacja_pamieci_tablica_dwuwymiarowa(int ***tablica_alokowana,int wymiar1,int wymiar2)
{
   **(tablica_alokowana)=(int*)malloc(wymiar1*sizeof(int));

        int i;
        for(i=0;i<wymiar1;i++)
        {
            *(tablica_alokowana[i])=(int*)malloc(wymiar2*sizeof(int));

    }

}

W mainie jest wywoływana w ten sposób

int main()
{ int **obraz;

    Alokacja_pamieci_tablica_dwuwymiarowa(&obraz,3,2);
return 0;
}

Pewnie błąd jest gdzieś w wskaźnikach ,ale trochę się w nich pogubiłem .

Proszę o wskazanie błędu i ewentualne wytłumaczenie co jest nie tak.

 

Answer 1

**(tablica_alokowana)=(int*)malloc(wymiar1*sizeof(int));

Dlaczego wynik malloc castujesz na int* ? Ewidentnie to ma być int**, bo tablica 2d. Podobnie ilość alokowanych bajtów - dlaczego sizeof(int) skoro chcesz zaalokowac pamięć na `wymiar1` wskaźników? 

Reszta wydaje mi się ze jest ok.

Answer 2

Jakiś wyjątek z jednego z projektów...

#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <assert.h>

static void alloc_2dim_table(int ***table, size_t height, size_t width) {
	size_t row;

	*table = malloc(height * sizeof(int *));
	assert(*table != NULL);

	for(row = 0; row < height; ++row) {
    		(*table)[row] = malloc(width * sizeof(int));
	}
}

static void dealloc_2dim_table(int ***table, size_t height) {
	size_t row;
	for(row = 0; row < height; ++row) {
		free((*table)[row]);
	}
	free(*table);
}

static void fill_zero_2dim_table(int ***table, size_t height, size_t width) {
	size_t row;
	for(row = 0; row < height; ++row) {
		memset((*table)[row], 0, width * sizeof(int));
	}
}

static void show_2dim_table(int ***table, size_t height, size_t width) {
	size_t row;
	size_t col;
	for(row = 0; row < height; ++row) {
		for(col = 0; col < width; ++col) {
			printf("%6d ", (*table)[row][col]);
		}
		(void) putchar('\n');
	}
}

int main(void) {
    int **table;
    const size_t width = 10;
    const size_t height = 10;

    alloc_2dim_table(&table, height, width);
    fill_zero_2dim_table(&table, height, width);
    show_2dim_table(&table, height, width);

    dealloc_2dim_table(&table, height);

    return EXIT_SUCCESS;