• Najnowsze pytania
  • Bez odpowiedzi
  • Zadaj pytanie
  • Kategorie
  • Tagi
  • Zdobyte punkty
  • Ekipa ninja
  • IRC
  • FAQ
  • Regulamin
  • Książki warte uwagi

Problemy z pamięcią- C. malloc

VPS Starter Arubacloud
0 głosów
289 wizyt
pytanie zadane 12 października 2015 w C i C++ przez Mateep Użytkownik (850 p.)

generalnie program ma liczycc wyznacznik macierzy kwadratowej, kompiluje się  ale nie można go odpalić bo ma jakiś problem ze zwalnianiem pamięci. dałby ktoś radę na to zerknąć?
 

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

void clean(double **matrix, int n)
{
	int i;
 	for(i = 0; i < n; i++)
    	{
        	free(matrix[i]);
    	}
    	free(matrix);
    
}

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


return matrixm;
}



double det(double ** matrix, int n)
{
  double** minor;
  double sum=0.0;
  double sign=1.0;
  int i,k=0,l,j=0,a;
  if(n==1) return matrix[0][0];
  if(n==2) return matrix[0][0]*matrix[1][1]-matrix[0][1]*matrix[1][0];
  
  minor=matrix_alloc(n-1);
 
 for(i=0;i<n;i++)
  {
	for(a=0;a<n;a++){
   	if(j!=a)
	{
		for(l=1;l<n;l++)
		{
			minor[k][l]=matrix[a][l];
			k++;
		}
	}}

  sum+=sign*matrix[0][i]*det(minor,n-1);
  sign = -sign;
}
clean(minor,n-1); 
  return sum;
}
int main()
{
	
	int n,i,j,k;
	double l,wyzna;	
	FILE *f;
	
	
	
	//otworzenie pliku
	f=fopen("matrix.txt","r");
	if(f==NULL)
	{
		printf("error");
		return 1;
	}
	//odczytanie rozmiaru macierzy
	fscanf(f,"%d",&n);

	
	
	double** matrix=matrix_alloc(n);

	if(matrix==NULL)
	{
		printf("problem z utworzeniem macierzy");
	}

	
	
	
	for(i=0;i<n;i++)
	{
		for(j=0;j<n;j++)
		{
		fscanf(f,"%lf",&matrix[i][j]);
						//printf("%.2f ",l);	
		}				//if(n)printf("\n");
	}
                                              // printf("%f",matrix[2][1]);
	
wyzna=det(matrix,n);
printf("%f",wyzna);
 
clean(matrix,n);	
fclose(f);

return 0;}

 

2 odpowiedzi

0 głosów
odpowiedź 13 października 2015 przez furas Maniak (53,800 p.)
Musiał byś napisać coś więcej - wrzucić komunikat o błędzie.

W tym momencie nie da się uruchomić i sprawdzić kodu bez pliku `matrix.txt`

Jakiś krótki przykład do testowania (taki do zapisania we własnym `matrix.txt`) też by ułatwił sprawę.

---

A tak na pierwszy rzut oka to jeśli w main zachodzi `matrix == NULL` to też należało by zakończyć program bo inaczej będzie próbować używać macierzy choć nie jest zaalokowane miejsce w pamięci.
0 głosów
odpowiedź 13 października 2015 przez draghan VIP (106,230 p.)
Problem chyba nie leży w zwalnianiu pamięci. ;)

Powstawiałem do Twojego kodu kilka testowych wydruków i zdaje się, że program nie wychodzi poza zewnętrzną pętlę for w funkcji det - więcej nawet: nie dochodzi do zakończenia jednego jej przebiegu. Sprawdź jeszcze raz dokładnie swój algorytm. ;)

Podobne pytania

0 głosów
1 odpowiedź 777 wizyt
pytanie zadane 14 października 2015 w Java przez natrov Gaduła (3,970 p.)
0 głosów
1 odpowiedź 1,661 wizyt
0 głosów
2 odpowiedzi 600 wizyt
pytanie zadane 25 października 2015 w C i C++ przez Mateep Użytkownik (850 p.)

92,453 zapytań

141,262 odpowiedzi

319,088 komentarzy

61,854 pasjonatów

Motyw:

Akcja Pajacyk

Pajacyk od wielu lat dożywia dzieci. Pomóż klikając w zielony brzuszek na stronie. Dziękujemy! ♡

Oto polecana książka warta uwagi.
Pełną listę książek znajdziesz tutaj.

Akademia Sekuraka

Akademia Sekuraka 2024 zapewnia dostęp do minimum 15 szkoleń online z bezpieczeństwa IT oraz dostęp także do materiałów z edycji Sekurak Academy z roku 2023!

Przy zakupie możecie skorzystać z kodu: pasja-akademia - użyjcie go w koszyku, a uzyskacie rabat -30% na bilety w wersji "Standard"! Więcej informacji na temat akademii 2024 znajdziecie tutaj. Dziękujemy ekipie Sekuraka za taką fajną zniżkę dla wszystkich Pasjonatów!

Akademia Sekuraka

Niedawno wystartował dodruk tej świetnej, rozchwytywanej książki (około 940 stron). Mamy dla Was kod: pasja (wpiszcie go w koszyku), dzięki któremu otrzymujemy 10% zniżki - dziękujemy zaprzyjaźnionej ekipie Sekuraka za taki bonus dla Pasjonatów! Książka to pierwszy tom z serii o ITsec, który łagodnie wprowadzi w świat bezpieczeństwa IT każdą osobę - warto, polecamy!

...