[Risolto] [C++]Passaggio puntatore a funzione

[cent]

Salve a tutti ragazzi. Sto preparando un programmino per l'esame di informatica e non riesco a risolvere un problema.
Praticamente ho creato una funzione che crea una lista dinamica; questa viene invocata passandogli dei parametri per reference. Tali parametri sono tutti nulli, dato che verranno riempiti dalla funzione stessa al suo termine.
Se passo delle semplici variabili, tutto funziona; se passo invece un puntatore (dove verrà memorizzato l'indirizzo della testa della lista), questo torna sempre NULL:

Codice:

/////////////////////////////////////////////////////////////////////////////////////////////////////////
///Funzione che legge il file "database/registroAbbonamenti.txt" e lo memorizza in una lista dinamica///
/////////////////////////////////////////////////////////////////////////////////////////////////////////
void leggiAbbonamenti(int &numAbb, int &codiceAbb, abbonamento *testaListaAbb){
[...]
    return;
}
/////////////////////////////////////////////////////////////////////////////////////////////////////////
///Fine funzione///
///////////////////

int main(){
    int numAbb, codiceAbb;
    abbonamento *testaListaAbb=NULL;
    leggiAbbonamenti(numAbb, codiceAbb, testaListaAbb);
system("PAUSE");
}

E' sbagliato il passaggio oppure c'è proprio qualcosa nella funzione??
Grazie mille
Roberto

[kelev]

Prova così:

Codice:

/////////////////////////////////////////////////////////////////////////////////////////////////////////
///Funzione che legge il file "database/registroAbbonamenti.txt" e lo memorizza in una lista dinamica///
/////////////////////////////////////////////////////////////////////////////////////////////////////////
void leggiAbbonamenti(int numAbb, int codiceAbb, abbonamento *testaListaAbb){
[...]
    return;
}
/////////////////////////////////////////////////////////////////////////////////////////////////////////
///Fine funzione///
///////////////////

int main(){
    int numAbb, codiceAbb;
    abbonamento *testaListaAbb=NULL;
    leggiAbbonamenti(numAbb, codiceAbb, &testaListaAbb);
system("PAUSE");
}

[cent]

No, ci avevo provato anche io ma l'errore è il seguente:

163 K:\Parcheggio.cpp cannot convert `abbonamento**' to `abbonamento*' for argument `3' to `void leggiAbbonamenti(int&, int&, abbonamento*)'

[kelev]

Puoi postare il programma che compili?

[cent]

Questa è la fuzione principale, il main è quello sopra:

Codice:

void leggiAbbonamenti(int &numAbb, int &codiceAbb, abbonamento *testaListaAbb){
ifstream fin;
char nome[100], residenza[100], telefono[20], buffer1[31], file[100], codice[100], buffer2[31], credito[100];
int codice2;
abbonamento *current=NULL;
abbonamento *head=NULL;
abbonamento *temp=NULL;

strcpy(file,"database/registroAbbonamenti.txt");
fin.open(file);
if(fin.fail()){
cout<<"Errore (1) apertura file!\n";
}
    int t=0;
while(!fin.eof()){
//estrazione campi
//se il primo carattere è diverso da "/", allora errore lettura del file
fin.getline(buffer1,30);t++;
fin.getline(nome,99);t++;
fin.getline(residenza,99);t++;
fin.getline(telefono,99);t++;
fin.getline(codice,99);t++;
fin.getline(credito,99);t++;
fin.getline(buffer2,30);t++;
codiceAbb=atoi(codice);
codice2=atoi(codice);
if (buffer1[0]!='/' || buffer2[0]!='/'){
            cout<<"Errore (2) di lettura del file. File danneggiato! Righe::  "<<t<<"\n";
//cout<<"Buffer1::  -"<<buffer1<<"-   nome::   -"<<nome<<"-   residenza::   -"<<residenza<<"-   telefono::   -"<<telefono<<"-   codice::   -"<<codice<<"-\n";
break;
}else{
      //aggiungo alla struttura dati
      //cout<<"Aggiungo i dati alla lista\n";
      temp=new abbonamento;
      strcpy(temp->nome,nome);
      strcpy(temp->residenza,residenza);
      strcpy(temp->telefono,telefono);
      strcpy(temp->codice,codice);
      strcpy(temp->credito,credito);
      temp->next=NULL;
      if(head==NULL){
                head=temp;
                //current=head;
              }else{
                    current=head;
                    while(current->next!=NULL)
                          current=current->next;
                    current->next=temp;
              }
       
          }
}//fine ciclo while
cout<<"Righe totali file:  "<<t<<"\n";
cout<<"Elenco abbonamenti: \n";
    current=head;
    testaListaAbb=head;
int i=0;   
    while(current!=NULL){
           cout<<"\n--------------------\n";
           cout<<current->nome<<"\n";
           cout<<current->residenza<<"\n";
           cout<<current->telefono<<"\n";
           cout<<current->codice<<"\n";
           cout<<current->credito<<"\n";
           current=current->next;
       i++;
    }
    cout<<"\n--------------------\n";
    numAbb=i;
    return;
}

[kelev]

Nel passaggio di parametri un puntatore:
- Parametri formali => void ciao(int *x, int *y) ;
- Parametri attuali => ciao(&x,&y).
 
Quindi da così:

Codice:

void leggiAbbonamenti(int &numAbb, int &codiceAbb, abbonamento *testaListaAbb){

a così:

Codice:

void leggiAbbonamenti(int numAbb, int codiceAbb, abbonamento *testaListaAbb){

[menelicte]

Se devi modificare l'indirizzo della lista, devi passarlo per riferimento, usando quindi un puntatore di puntatore.

Prova a modificare così, usando

Codice:

(*testaListaAbb)

al posto di

Codice:

testaListaAbb

all'interno della funzione:

Codice:

/////////////////////////////////////////////////////////////////////////////////////////////////////////
///Funzione che legge il file "database/registroAbbonamenti.txt" e lo memorizza in una lista dinamica///
/////////////////////////////////////////////////////////////////////////////////////////////////////////
void leggiAbbonamenti(int numAbb, int codiceAbb, abbonamento **testaListaAbb){
[...]
    return;
}
/////////////////////////////////////////////////////////////////////////////////////////////////////////
///Fine funzione///
///////////////////

int main(){
    int numAbb, codiceAbb;
    abbonamento *testaListaAbb=NULL;
    leggiAbbonamenti(numAbb, codiceAbb, &testaListaAbb);
system("PAUSE");
}

[cent]

No, non devo modificarlo, devo solo ottenerlo per utilizzarlo in un'altra funzione.

@xajx: non funziona anche apportando le modifiche sugeritemi.

[cent]

Ok, ora funziona.
Grazie a tutti

[kelev]

Cosa hai corretto?

[cent]

Ciao, allora ho messo un doppio asterisco al nome della variabile (puntatore) quando definisco la funzione; all'interno della funzione, quando vado a modificare il puntatore in questione metto solo un asterisco.
Quando richiamo la funzione, passo il puntatore senza * e &.

Ciao ciao

[kelev]

Ah ok grazie