Discussion :

[Nyko17]

Bonjour. Je souhaiterais afficher sur l'ecran les informations d'un objet de type Vin * v1= new Vin(string nom, string annee, string quantité).

J'ai donc fais la surcharge de l'operateur d'injection <<, mais ça ne m'affiche que des truc bizarres en hexa, du genre 0x198f3s54.

Voici le code:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
std::ostream & operator << (std::ostream & os, Vin unVin)
{
             os << unVin.getsonNom() <<" "<< unVin.getsonAnnee() << " " << unVin->getsaQuantite() << '\n';
 
             return os;
 
}

Et voici un des accesseurs:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
std::string Vin::getsonNom() const
{
    return sonNom;
}

Une idée?
Merci.

[loicounet]

hello,

t'as tenté de placer ton objet en référence?

std::ostream & operator << (std::ostream & os, Vin &unVin)

dans mon bouquin c'est défini comme ça...ca vient peut être de la!

[zais_ethael]

Idéalement, il faut même que ce soit une référence constante :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

std::ostream& operator << (std::ostream& os, const Vin& unVin)

Si tu as mal défini ton constructeur de copie il se pourrait que ça vienne de la.

Au passage, si tous tes accesseurs sont définis tels que celui que tu nous as collé ce n'est pas très rapide. Il vaudrait mieux renvoyer des références constantes vers tes strings.

[Nyko17]

hello,

t'as tenté de placer ton objet en référence?

std::ostream & operator << (std::ostream & os, Vin &unVin)

dans mon bouquin c'est défini comme ça...ca vient peut être de la!

Oui. même résultat.

Idéalement, il faut même que ce soit une référence constante :
Code :

std::ostream& operator << (std::ostream& os, const Vin& unVin)

Si tu as mal défini ton constructeur de copie il se pourrait que ça vienne de la.

Au passage, si tous tes accesseurs sont définis tels que celui que tu nous as collé ce n'est pas très rapide. Il vaudrait mieux renvoyer des références constantes vers tes strings.

Je vais tester avec un const alors...

Je ne vois pas ce que tu veux dire pour les accesseurs. Oo

[koala01]

Salut,

Les "accesseurs", ce sont ces méthode qui renvoient une valeur propre à l'objet en cours de traitement, telle que
getSonNom, getSonAnee, getSaQuantité...

As tu remarqué que tu passe ton élément vin par valeur et que tu essaye d'obtenir le résultat de getSaQuantité comme s'il s'agissait d'un pointeur (il faut utiliser le . pour appeler getSaQuantité, et non le déréférencement de pointeur -> )

Enfin, l'idéal serait peut etre de déclarer la fonction amie dans la classe vin, histoire de pouvoir te passer du recours aux accesseurs...

Dans la classe Vin, un simple

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

friend std::ostream& operator<<(std::ostream&, const Vin&);

en accessibilité publique te permet d'écrire plutot

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
std::ostream& operator<<(std::ostream o, const Vin& v)
{
    o<<v.mNom<<" "<<v.mprenom<<" "<<v.mquantite;
    return o;
}

[zais_ethael]

Je vais tester avec un const alors...

Si ça ne marche pas avec une référence simple ce ne sera pas mieux avec une référence constante, le problème doit venir de l'objet que tu passes.
Tu as essayé d'afficher le résultat de tes getxxx() simplement dans ton main ?

[Davidbrcz]

Déférence tu bien ton objet ?
Car vu ce tu nous décrit, il me semble que tu affiche plus le contenu du pointeur que l'objet pointé.
Pour en être sûr, montre nous comment tu appelle operator<< pour ton objet.