Pas de retour de QString

Bonjour,

J'appelle depuis une class une autre class qui doit me retourner un QString. Mais il revient vide !

la classe de départ crée un nouvel ID3 :

Code:

---

1 2 3 4

Id3 *fichier_courant = new Id3(fic, this);   qDebug() << fichier_courant->Album() << fichier_courant->Artiste();

---

Les deux methodes de la class Id3 :

Code:

---

1 2 3 4

QString Id3::Album(){return this->album_name;}   QString Id3::Artiste(){return this->artiste_name;}

---

L'espace est alloué pour les deux variables, elles sont connues dans le constructeur Id3, mais pas à l'extérieur !! Ca sent la faute ici, mais je ne saisi pas où exactement.
Je sens la faute de débutant, mais où !!

Merci pour votre aide

4 lignes de code tout à fait valides, et on doit y trouver un problème dans un temps imparti ?

Vérifie que les chaines sont bien allouées, et qu'elle contiennent des données et pas juste des \0. Le debugger est ton amis pour résoudre ce genre de problème.

Après débogage, les string allouées dans le constructeur n'existe plus quand j'appelle les fonctions :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15

Id3::Id3 (QFile *song, QWidget *parent) : QWidget(parent) { .... .... QString *artiste_name =new QString(); QString *album_name = new QString(); .... .... }   QString Id3::Album(){return *album_name;}   QString Id3::Artiste(){return *artiste_name;}

---

Le prototype est le suivant:

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19

class Id3 : public QWidget {   Q_OBJECT   public: Id3 (QFile *song, QWidget *parent); QString Artiste(); QString Album();   private slots:   private: QGridLayout *layout; QTextEdit *sortie; QString *artiste_name; QString *album_name; };

---

Je n'arrive pas a saisir !!
Quelqu'un voit-il une erreur

Bonjour,

Moi je vois des QString alloué mais vide (et je crains qu'elles le resterons)
Maintenant, je me demande aussi pourquoi vous utilisez des pointeurs, au lieu de QString simplement.

Oh la,

Code:

---

QString *artiste_name =new QString();

---

Tu utilise le nom de type QString de nouveau dans le constructeur. Donc tu écrase la variable de classe au même nom. (C-à-d tu re-déclare la variable localement, tu ne fais pas référence à celle crée dans ton .h).

Et je te suggère de ne pas utiliser des pointeurs pour les types primitifs.

Donc

QString artist_name

et dans le cons:

artist_name = ""; //Ou autre traitement que tu ferais avant de le domper dans une var

Et oui merci, c'était bien la redéclaration qui n'allait pas !

Encore une erreur de débutant !

Je vais tenir compte de ton conseil concernant les pointeurs, je ne les avais pas utilisés au départ, et puis en cherchant on essaye tout, sauf la bonne chose !!

Merci