Alokacja pamięci. problem

Question

Witam piszę program i mam tam alokację pamięci tablicy2 wymiarowej. Chciałbym aby allokacja wykonywała się w funkcji a nie w mainie. jak mam to napisać , czy da się aby argumentem był tylko rozmiar tablicy???

oto moja alokacja:

//alokacja odpowiedniej ilosci wierszy
	double**matrix=(double**)malloc(n*sizeof(double*));
	if ( matrix == NULL )
	{
		printf("blad przy alokacji pamieci ilosci wierszy");
		return 1;
	}
	
	
	for(i=0;i<n;++i)
	{
		matrix[i]=(double*)malloc(n*sizeof(double));
		if (matrix[i]==NULL)
		{
			printf("blad przy alokacji pamieci");
			clean(matrix,i);
			return 1;
		}

 

Answer 1

Jasne, że się da. Skopiuj kod do funkcji, dodaj pobieranie wielkości jako parametru funkcji i zwracaj z funkcji wskaźnik na stworzoną tablicę.

Comments

niby wiem o co CI chodzi ale jakoś mi to nie chce się kompilować.

---

Ale daje jakiś komunikat o błędzie ?

Stwierdzenie "nie chce się kompilować" nic nie wnosi do dyskusji.

---

funkcja powinna być typu double * ?

i ma zwracać wskaźnik na tą moją tablice.
a później wywołanie jak?
nie do końca wiem dlatego nie chcialbym walnąć jakiejś głupoty :)

---

Zmienna matrix jest typu double** więc funkcja też musi być tego typu.

double** macierz(int n){...}

Funkcja konczy się linią

return matrix;

Wywołanie:

double** nowa_macierz = macierz(5);

---

Wersja w C.

Musiałem pozbyć się clean() i przenieść deklarację int i

Jeśli coś się nie uda to funkcja zwraca NULL aby dało się to rozpoznać w main().

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

double** macierz(int n) {

    int i;
    
    //alokacja odpowiedniej ilosci wierszy
    double** matrix = (double**)malloc(n*sizeof(double*));
    if ( matrix == NULL )
    {
        printf("blad przy alokacji pamieci ilosci wierszy");
        return NULL;
    }
     
     
    for(i = 0; i < n ; i++)
    {
        matrix[i] = (double*)malloc(n*sizeof(double));
        if( matrix[i] == NULL )
        {
            printf("blad przy alokacji pamieci");
            matrix[i] = 0;
            return NULL;
        }
    }

    return matrix;
}

int main() {

    double** m = macierz(5);

    if( m == NULL ) {
        printf("problem z utworzeniem macierzy");
    } else {
        printf("macierz utworzona");
    }

    //TODO: nalezalo by macierz skasowac za pomoca free
    
    return 0;
}

 

---

wielkie dzięki! jeszcze mam pytanie , bo ja tam wtym alokowaniu mam return 1; jeżeli alokacja źle nastąpi no i w kompilacja wywala mi warninga:

return makes pointer from integer without a cast [enabled by default]
da się coś z tym zrobić?

---

Funkcja macierze() zwraca wartość, którą przypisuję do zmiennej m typu double**
Liczba 1 nie jest wartością typu double** więc kompilator się burzy. Mówi, że niejawnie zamienia liczbę integer na coś co jest typu double** (aby mógł przypisać tą wartość do zmiennej m). Nie burzył by się gdyby tam była jawna zamiana - "rzutowanie" (ang. cast)

return (double**)1;

NULL jest wartością typu double** dlatego lepiej tu zwracać NULL jako oznaczenie błędu. Zwykle tego typu funkcje jako błąd zwracają właśnie NULL.

Przykładowo malloc w razie problemu zwraca NULL.

---

Bardzo dziękuję za pomoc teraz wszystko jest jasne i klarowne. może uda mi się kiedyś malloca ogarnąć :)