[C] - problema double pointer a file

[Re_Kotc]

ciao a tutti, sto cercando di far funzionare questo programmino (non l'ho scritto io è di un amico ) ma crasha sempre, il main è questo:

Codice PHP:

int main()
{
      FILE ** in;
      FILE **  out;
      printf("%p\n",in);
      char * filein = "PIDin.txt" ;
      char * fileout = "DataOut.data" ;     //openCommChannels(args[1],args[2],in,out);
      openCommChannels(filein , fileout , in , out);
      printf("metà\n");//
      FILE * a,* b;
      printf("%p\n",in);
      printf("pre assegn\n");
      a=(*in);
      b=(*out);
      closeCommChannels(a,b);
      return 0;    
    } 


la funzione openCommChannels è definita così:

Codice PHP:

//apre i canali di comunicazione per l'ingresso e l'uscita      
int openCommChannels(char const * inputFileName , char const * outputFileName , FILE ** ifpp , FILE ** ofpp){
    //printf("inizio openComm");
    FILE * x;
    FILE * y;
    x=fopen(inputFileName ,"r");
    y=fopen(outputFileName , "w");
    printf("pre controllo su ifpp\n");
    if(x== NULL){
                   fprintf(stderr,"Impossibile aprire il file da cui leggere");
                   return -1;
                   }
                   else {
                        printf("pointer pre %p",ifpp);
                        //ifpp = &x;
                        
                        *ifpp = x;
                        printf("pointer post %p",ifpp);
                        //printf("ok");
                        }
    if(y== NULL){
                   fprintf(stderr,"Impossibile aprire il file su cui scrivere");  
                   return -1;
                   }     
                   else
                       *ofpp = y;
    return 0;                       
    }; 


se lo eseguo l'output del programma è:

76c0db07
pre controllo su ifpp
pointer pre 76c0db07

e poi il tutto crasha...quindi il problema dovrebbe essere qui

*ifpp = x;

e non capisco perchè!

avete suggerimenti? grazie mille

[wingman87]

Sì, il problema è proprio lì, perché ifpp non punta a nulla, o meglio punta ad un'area di memoria che non hai allocato. Lo stesso vale per ofpp.
Per risolvere dove dichiari in e out dovresti allocare il puntatore:

FILE ** in=(FILE**)malloc(sizeof(FILE*));
FILE **  out=(FILE**)malloc(sizeof(FILE*));

E quando non li usi più dovresti deallocarli con free.
Tra l'altro non ho guardato proprio bene ma mi sembra che un semplice puntatore fosse sufficiente (invece di un doppio puntatore) in questo caso potresti fare:

FILE * in;
FILE * out;

e quando richiami la funzione fai:

openCommChannels(filein , fileout , &in , &out);

[Re_Kotc]

grazziiiiieeeeee mi hai risolto un problema!! sei stato utilissimo grazie!

[Re_Kotc]

mmm solo una domanda...ma perchè se utilizzo il doppio puntatore devo allocare la memoria prima mentre con il puntatore singolo no?

[wingman87]

Dunque, non è facilissimo da spiegare (non per me almeno):
Il doppio puntatore può contenere l'indirizzo in memoria di un puntatore singolo.
Il puntatore singolo invece può contenere un indirizzo di memoria in cui si trova un valore.
Quindi se ho **p
1) p può contenere l'indirizzo di un'area di memoria contenente un puntatore singolo
2) *p è il valore del puntatore singolo puntato da p
3) **p è il valore puntato dal puntatore singolo puntato da p
Quando dichiari FILE **p stai allocando lo spazio solo per il doppio puntatore che quindi non contiene ancora l'indirizzo di un puntatore e quindi non puoi usare 2) e 3)
Se allochi lo spazio per un puntatore e metti il risultato (la malloc restituisce un indirizzo) in p potrai usare 2).
A questo punto non puoi ancora usare 3) ma in openCommChannels la fopen restituisce un puntatore a FILE (quindi un indirizzo) e lo mette in *p

*ifpp = x;

Quando invece usi il puntatore singolo hai *p
1) p può contenere un indirizzo di memoria in cui si trova un valore.
2) *p è il valore puntato da p
Quando dichiari FILE *p allochi lo spazio per il singolo puntatore e quindi puoi usare 1) ma questo non contiene ancora un indirizzo di memoria valido. L'indirizzo che ti serve ti viene restituito dalla fopen e a quel punto potrai usare anche 2)

[tomminno]

Originariamente inviato da Re_Kotc

mmm solo una domanda...ma perchè se utilizzo il doppio puntatore devo allocare la memoria prima mentre con il puntatore singolo no?

Dichiarare FILE ** in quel modo nel main è indice di scarsa comprensione dei puntatori
Per rispondere alla tua domanda: con il puntatore singolo la memoria viene allocata dalla funzione fopen, ma con il puntatore doppio chi la inizializza?
Prova a pensare (o a debuggare che forse ti spiegherebbe tante cose): a cosa punta nel main FILE ** in; ?
Ad un area di memoria passatemi il termine "sbagliata" o meglio non inizializzata (** è un puntatore a un puntatore).
Quando provi ad assegnarne il valore del "puntatore puntato" all'interno di openCommChannels stai cercando di modificare un puntatore non valido (*ifpp rappresenta appunto questo puntatore non inizializzato), ovvero stai cercando di modificare un'area di memoria che non puoi modificare.

La soluzione che ti è stata suggerita era corretta infatti a te nel main serve un FILE* che deve essere modificato all'interno di openCommChannels, qui è corretto l'uso di FILE** in quanto te devi modificare il valore della variabile in contenuta nel main e in C per fare questo serve appunto un puntatore a puntatore.