[C]confusione sui puntatori..

[santaclause83]

non riesco a capire come mai la firma della funzione di allocazione di questa struct inizi con un tipo e per compilare serenamente alla fine ne voglia restituire il puntatore...

typedef struct matrice {
int r, c; // numero di righe e di colonne
char nomemat[N];
double **vett; // Puntatore al vettore contenente gli elementi
}t_nmat;

1)la firma della funzione di allocazione di una matrice è questa

t_nmat alloca_mat(char *nome,char elementi[]);

2)dentro la funzione lo spazio per la struct lo alloco in questo modo

t_nmat *nuova;//matrice da allocare



nuova = (t_nmat *)malloc(sizeof(t_nmat));

3)inizializzo i vari campi e in teoria secondo cosa mi dice la firma dovrei restituire un tipo t_nmat,invece il compilatore non da problemi solo se gli restituisco un *t_nmat

infatti :
return *nuova;

ma in questo modo la firma iniziale non dovrebbe essere tipo * nomefun(...)?

[andbin]

Originariamente inviato da santaclause83

1)la firma della funzione di allocazione di una matrice è questa

t_nmat alloca_mat(char *nome,char elementi[]);

In questo modo la struttura viene restituita per valore (copia), cosa perfettamente lecita in ANSI "C".

t_nmat m;
m = alloca_mat (......);

[santaclause83]

nel caso voglia passare direttamente il riferimento alla struttura creata invece?

potresti delucidarmi sulla firma della funzione e il tipo restituito in quel caso,anche secondo come ho impostato le cose sopra?

grazie mille,nicholas.

[andbin]

Originariamente inviato da santaclause83

nel caso voglia passare direttamente il riferimento alla struttura creata invece?

potresti delucidarmi sulla firma della funzione e il tipo restituito in quel caso,anche secondo come ho impostato le cose sopra?

Se vuoi restituire una struttura usando il valore di ritorno (perché si potrebbe anche restituire una struttura tramite un puntatore nei parametri), si può fare in 2 modi:

A) Per valore

t_nmat alloca_mat(char *nome,char elementi[])
{
    t_nmat m;
    ....
    return m;
}

B) Per indirizzo

t_nmat *alloca_mat(char *nome,char elementi[])
{
    t_nmat *pm;
    pm = (t_nmat*) malloc (sizeof(t_nmat));
    ....
    return pm;
}

[santaclause83]

ok,grazie,ho modificato la funzione di allocazione di memoria per facilitarmi anche con la funzione di de_allocazione,e mi dà un segmentation fault che non mi so spiegare, inoltro i passi salienti del codice:

// Struttura dati che contiene la matrice
typedef struct matrice {
int r, c; // numero di righe e di colonne
char nomemat[N];
double **vett; // Puntatore al vettore contenente gli elementi
}t_nmat;


la funzione per allocare spazio ha questa intestazione qua,restituisce
un puntatore a struct,è un pò lunga,ma con delle stampe di cortesia e funzioni varie mi sono assicurato che gli elementi della struct venissero
inizializzati sul serio:

t_nmat* alloca_mat(char*nome,char elementi[]);


PROBLEMA:devo scrivere la funzione di deallocazione dello spazio

SOLUZIONE brevettata:

void dealloca(int righe,t_nmat* mat){

int i;

for (i=0; i<righe; i++) {
printf("libero il vettore numero %d",i);
free(mat->vett[i]);
}
free(mat->vett);
free(mat->nomemat);
free(mat);
}

nel main scrivo quanto segue:

int main(void){

t_nmat* insiememat[20];


char dati1[N] = "1 2 3 4 ; 5 6 7 8 ; 1 2 4 6 ;"; //la matrice deve essere inizializzata da tastiera,e per prova creo un array di char con gli elementi dentro

insiememat[0] = alloca_mat("gigi",dati1);

dealloca(insiememat[0]->r,insiememat[0]);

return 0;
}

hai mica un idea sul perchè la funzione di deallocazione restituisca un segmentation fault non appena entri nel ciclo for???

[-fidel-]

Originariamente inviato da santaclause83

hai mica un idea sul perchè la funzione di deallocazione restituisca un segmentation fault non appena entri nel ciclo for???

Come allochi la struttura mat ed i singoli elementi di vett?
In più, il segmentation fault avviene sulla prima free che incontri?

[santaclause83]

si la free avviene sul primo elemento che incontro,faccio fare delle stampe di controllo alla funzione e vedo che fa solo in tempo a stampare la prima riga,prima ancora di deallocare il tutto.-
cmque le varie componenti della struct le inizializzo così:

t_nmat *nuova;//matrice da allocare


nuova = (t_nmat *)malloc(sizeof(t_nmat));

strcpy(nuova->nomemat,nome); //dò il nome alla matrice


..nella funzine faccio un ciclo dove da una stringa tiro fuori gli elementi della matrice,il numero righe e il numero colonne...

/*assegnamento del numero righe e numero colonne della nuova matrice*/

nuova->r = m;

nuova->c = n;


/*inizializzazione dello spazio riservato agli elementi della matrice tramite il puntatore di t_nmat*/


nuova->vett = calloc(m,sizeof (*nuova->vett));
for (i=0; i<m; i++) {


nuova->vett[i] = calloc(n, sizeof (**nuova->vett));

}

//inizializzazione degli elementi della matrice,in els[] ho messo i numeri letti dalla stringa iniziale,li vado a inserire nello spazio appena allocato dai vettori della matrice

for(i=0;i<m;i++){

for(j=0;j<n;j++){

nuova->vett[i][j] = els[count];

count++;

}



}

[-fidel-]

Direi che è un problema di cast.

sostituisci:

nuova->vett[i][j] = els[count];

con

nuova->vett[i][j] = (double)els[count];

Inoltre, els[] è un array statico o creato sempre con malloc/calloc?
se crei con malloc:

double *els;
els = (double*)malloc(num_elements * sizeof(double));

[santaclause83]

els[] comunque l'ho dichiarato come array di double

in compenso ho fatto dei passi avanti,nel senso che

1)nella funzione alloca_mat mi faccio restituire un puntatore alla struttura e non la struttura puntata

2)alla funzione "dealloca" passo adesso tramite il puntatore l'esatta area di memoria dove risiede la struct

con delle stampe di controllo riesco controllare la free su almeno i vettori della matrice,ma al momento in cui devo liberare l'array di char del nome,ritorrna il segmentation fault.

faccio vedere la funzione chiamata dal main,la funzione implementata e l'output ricevuto:


da main:

t_nmat* insiememat[20]; //array di puntatori a matrice

char dati1[N] = "1 2 3 4 ; 5 6 7 8 ; 1 2 4 6 ;";
char dati2[N] = "3.4 2.5 3 6 8 ; 4 6 89 12 10 ;";

insiememat[0] = alloca_mat("gigi",dati1);
insiememat[1] = alloca_mat("marco",dati2);

dealloca(insiememat[0]->r,*insiememat[0]);
return 0;


funzione dealloca:

void dealloca(int righe,t_nmat mat){

int i;

printf("sto per deallocare l'area di memoria di %s occupata dai vettori\n",mat.nomemat);

for (i=0; i<righe; i++) {
printf("liberando il vettore numero %d\n",i+1);
free(mat.vett[i]);
}
printf("liberando l'area di memoria occupata dal nome\n",i);
free(mat.nomemat);
printf("liberando l'area di memoria occupata dalla struttura\n",i);
free(&mat);
}


OUTPUT:

sto per deallocare l'area di memoria di gigi occupata dai vettori
liberando il vettore numero 1
liberando il vettore numero 2
liberando il vettore numero 3
liberando l'area di memoria occupata dal nome
Segmentation fault

[santaclause83]

vabbè lascia stare la sbadataggine del copia/incolla sulle stampe di controllo dopo il ciclo che hanno il contatore come parametro,cmque ho verificato non dipende da quello....

[-fidel-]

Guarda, ho fatto un semplice programminosfruttando la tua struttura e funziona benissimo:

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

typedef struct matrice
{
   int r, c; // numero di righe e di colonne
   char nomemat[20];
   double **vett; // Puntatore al vettore contenente gli elementi
} t_nmat;

int main()
{
   t_nmat *nuova;
   int i, j;

   nuova = (t_nmat *)malloc(sizeof(t_nmat));
   nuova->vett = calloc(10, sizeof (*nuova->vett));
   for (i = 0; i < 10; i++)
   {
      nuova->vett[i] = calloc(20, sizeof (**nuova->vett));
      for (j = 0; j < 20; j++)
         nuova->vett[i][j] = (double)j;
   }
   // Stampa valori
   for (i = 0; i < 10; i++)
   {
      for (j = 0; j < 20; j++)
         printf("%.1f ", nuova->vett[i][j]);
      printf("\n");
   }
   // Libero memoria.
   for (i = 0; i < 10; i++)
      free(nuova->vett[i]);
   free(nuova->vett);
   free(nuova);
   return 0;
}

...

[santaclause83]

ok grazie,cmque il problema adesso rimane liberare memoria per l'array di char "nomemat",è lì che mi restituisce segmentation fault...

dici che mi basta fare la free sui vettori e sulla struct?

[santaclause83]

questione risolta!!

invece di dicharare nomemat come array di dimensione N lo dichiaro come puntatore a stringa

char *nomemat;

e quando inizializzo la struct alloco lo spazio per il nomemat con una malloc di N char

nuova->nomemat = (char *)malloc(sizeof(char)*N);


grazie e alla prossima/e!