15 replies to this topic

[toyo]

Sono abbastanza sicuro che sarà una cazzata, ma ci sto perdendo il pomeriggio.

Allora, una funzione/metodo che prende parametri passati come const & è comoda per risparmiare memoria (evita costruttori di copia) quando dobbiamo passare uno o più parametri che non saranno modificati dalla funzione.


Ok, sempre usati, sempre andato tutto bene, però ora:

la firma del metodo è

void inserisciGruppo( const string &, const Account* &  );

l'invocazione è

Account* Legami::crea_account_polimorfo(string line){



[....cose....]



this->inserisciGruppo(idgruppo,tmp);



}

idgruppo è una string creata dentro crea_account_polimorfo, tmp è un Account*, sempre creato in crea_account_polimorfo

errore:

scheletro_logica.cpp:73:46: error: no matching function for call to ‘Legami::inserisciGruppo(std::string&, Account*&)’

scheletro_logica.cpp:26:6: note: candidate is: void Legami::inserisciGruppo(const std::string&, const Account*&)

in pratica: non ho la funzione che prende string& e Account*&, ho quella che prende const string& e const Account&.

Perchè? Non dovrebbe funzionare?



Inb4 errore del cazzo.

[MadJackal]

Prime cose che mi viene in mente: chiami per caso metodi NON const della classe Account nella funzione inserisciGruppo (se ci usi dentro this, sono dichiarati non const al 100%)? Hai dichiarato la variabile che passi alla funzione come const Account*?

Nota che per passare un argomento per reference, senza quindi richiamare il copy-constructor, il const non è necessario. E c'è differenza tra una const reference ed una reference "normale", che mi sa che è la cosa che ti ha gabbato.


Secondariamente, che compilatore usi? Mi sa tanto di un GCC olderrimo...

[TigerShark]

scusa, ma se sei dentro la classe perche' usi this-> ?

[toyo]

Prime cose che mi viene in mente: chiami per caso metodi NON const della classe Account nella funzione inserisciGruppo (se ci usi dentro this, sono dichiarati non const al 100%)?

no, nella funzione inserisciGruppo vengono chiamati metodi non const, ma della classe Gruppo

Hai dichiarato la variabile che passi alla funzione come const Account*?

no, è dichiarata come Account* tmp; poi in uno switch c'è la new

Nota che per passare un argomento per reference, senza quindi richiamare il copy-constructor, il const non è necessario. E c'è differenza tra una const reference ed una reference "normale", che mi sa che è la cosa che ti ha gabbato.

vero, me la sono riguardata, il const serve per permettere di passare oggetti anonimi come reference, l'avevo lasciato per quello. L'ho tolto e funziona, grazie .
Però non capisco ancora perché prima non funzionasse, in quanto i 2 parametri passati erano due oggetti indirizzabili, non oggetti anonimi.

Secondariamente, che compilatore usi? Mi sa tanto di un GCC olderrimo...

come ide uso netbeans, il compilatore è il g++ standard della macchina

[toyo]

scusa, ma se sei dentro la classe perche' usi this-> ?

boh, è un vezzo che mi porto dalle superiori. L'ho tolto dai, ma alla fine non cambiava niente.

[toyo]

Ok ora c'è una roba che mi fa alquanto bestemmiare

void Legami::inserisciGruppo( string& s, Account* &  member){

    vector::iterator it;

    it= gruppi->begin();

    [ ... cose commentate ... ]

    }

così dà segmentation fault

void Legami::inserisciGruppo( string& s, Account* &  member){

    vector::iterator it;

    //it= gruppi->begin();

    [ ... cose commentate ... ]

    }

così no

gruppi è un vector*

al primo colpo è vuoto, quindi il begin dovrebbe restituirimi l'iteratore past-the-end


invece segmentation fault

[toyo]

che mona, non l'avevo inizializzato nel costruttore


fa bene scrivere le cose però

[toyo]

Sono abbastanza sicuro che sarà una cazzata, ma ci sto perdendo il pomeriggio.

Allora, una funzione/metodo che prende parametri passati come const & è comoda per risparmiare memoria (evita costruttori di copia) quando dobbiamo passare uno o più parametri che non saranno modificati dalla funzione.


Ok, sempre usati, sempre andato tutto bene, però ora:

la firma del metodo è

void inserisciGruppo( const string &, const Account* &  );

l'invocazione è

Account* Legami::crea_account_polimorfo(string line){



[....cose....]



this->inserisciGruppo(idgruppo,tmp);



}

idgruppo è una string creata dentro crea_account_polimorfo, tmp è un Account*, sempre creato in crea_account_polimorfo

errore:

scheletro_logica.cpp:73:46: error: no matching function for call to ‘Legami::inserisciGruppo(std::string&, Account*&)’

scheletro_logica.cpp:26:6: note: candidate is: void Legami::inserisciGruppo(const std::string&, const Account*&)

in pratica: non ho la funzione che prende string& e Account*&, ho quella che prende const string& e const Account&.

Perchè? Non dovrebbe funzionare?



Inb4 errore del cazzo.

[MadJackal]

Prime cose che mi viene in mente: chiami per caso metodi NON const della classe Account nella funzione inserisciGruppo (se ci usi dentro this, sono dichiarati non const al 100%)? Hai dichiarato la variabile che passi alla funzione come const Account*?

Nota che per passare un argomento per reference, senza quindi richiamare il copy-constructor, il const non è necessario. E c'è differenza tra una const reference ed una reference "normale", che mi sa che è la cosa che ti ha gabbato.


Secondariamente, che compilatore usi? Mi sa tanto di un GCC olderrimo...

[TigerShark]

scusa, ma se sei dentro la classe perche' usi this-> ?

[toyo]

Prime cose che mi viene in mente: chiami per caso metodi NON const della classe Account nella funzione inserisciGruppo (se ci usi dentro this, sono dichiarati non const al 100%)?

no, nella funzione inserisciGruppo vengono chiamati metodi non const, ma della classe Gruppo

Hai dichiarato la variabile che passi alla funzione come const Account*?

no, è dichiarata come Account* tmp; poi in uno switch c'è la new

Nota che per passare un argomento per reference, senza quindi richiamare il copy-constructor, il const non è necessario. E c'è differenza tra una const reference ed una reference "normale", che mi sa che è la cosa che ti ha gabbato.

vero, me la sono riguardata, il const serve per permettere di passare oggetti anonimi come reference, l'avevo lasciato per quello. L'ho tolto e funziona, grazie .
Però non capisco ancora perché prima non funzionasse, in quanto i 2 parametri passati erano due oggetti indirizzabili, non oggetti anonimi.

Secondariamente, che compilatore usi? Mi sa tanto di un GCC olderrimo...

come ide uso netbeans, il compilatore è il g++ standard della macchina

[toyo]

scusa, ma se sei dentro la classe perche' usi this-> ?

boh, è un vezzo che mi porto dalle superiori. L'ho tolto dai, ma alla fine non cambiava niente.

[toyo]

Ok ora c'è una roba che mi fa alquanto bestemmiare

void Legami::inserisciGruppo( string& s, Account* &  member){

    vector::iterator it;

    it= gruppi->begin();

    [ ... cose commentate ... ]

    }

così dà segmentation fault

void Legami::inserisciGruppo( string& s, Account* &  member){

    vector::iterator it;

    //it= gruppi->begin();

    [ ... cose commentate ... ]

    }

così no

gruppi è un vector*

al primo colpo è vuoto, quindi il begin dovrebbe restituirimi l'iteratore past-the-end


invece segmentation fault

[toyo]

che mona, non l'avevo inizializzato nel costruttore


fa bene scrivere le cose però

[toyo]

ho scoperto qual'era il problema iniziale, che avevo per il momento lasciato stare passando i puntatori per copia


in pratica, per prendere un riferimento puntatore costante a Tipo, non bisogna scrivere (come "intuitivo") cost Tipo * &, perchè questo è un riferimento puntatore a Tipo costante, e quindi non può prendere un oggetto anonimo, ma solo un lvalue.

Bisogna invece scrivere Tipo* const & (riferimento puntatore costante a Tipo), così prende anche oggetti anonimi (rvalue senza lvalue)

se si vuole prendere un riferimento puntatore costante a Tipo costante invece è const Tipo * const &

[matti157]

ahhhhhhhhh ora si che si ragiona