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.

[Nyko17]

J'ai essayé avec const, ça ne marche pas non plus.
Donc je me suis résigné à ne pas utiliser de pointeurs, je vais me contenter d'objets Vin simples, ça marche bien comme ça... De toutes façons, je n'avais aucune utilitée à utiliser des pointeurs, c'etait juste pour mieux comprendre leur fonctionnement. (C'est raté d'ailleurs ).
Enfin je suis toujours ouvert pour comprendre comment l'affichage avec un pointeur.
Pour faire simple et ne s'occuper que de ce qui nous intéresse, j'ai ça:


Vin.hpp

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
class Vin
{
  string sonNom;
  string sonAnnee;
  string saQt;
 
  public:
          Vin(std::string nom, std::string annee, std::string quantite); 
          string getNom() const;
          string getAnnee() const;
          string getQt() const;
};

Vin.cpp

Code :

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

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

ListeVins.hpp

Code :

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

1
2
3
4
5
6
7
class ListeVins
{
 
  public:
  std::list<Vin*> sesVins;
 
};

ListeVins.cpp

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
std::ostream & operator << (std::ostream & os, Vin&  unVin)
{
             os << unVin.getLeNom() <<" "<< unVin.getLaAnnee() << " " << unVin.getLaQt() << '\n';
 
             return os;
 
}
 
//Voici un exemple d'affichage:
 
for(int i=0; i<nbrVin; i++)
        {
 
              getline(uneListeVins, nom); //lecture depuis un fichier texte
              getline(uneListeVins, date);
              getline(uneListeVins, qt);
              Vin* v=new Vin(nom, date, qt); // creation d'un Vin*
              sesVins.push_back(v); //insertion dans la list
              std::cout << v << '\n'; // CRASH
              //toString(v); //affichage de secours (marche)
              delete v;
              uneListeVins.ignore();
        }

[Davidbrcz]

Code :

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

std::cout << *v << std::endl;

c'est mieux non ?

[Nyko17]

J'essaierai.

Donc *v ça fait quoi, ça passe un Vin& au lieu d'un Vin* ?

[zais_ethael]

Donc *v ça fait quoi, ça passe un Vin& au lieu d'un Vin* ?

En gros oui.
Ça déréférence l'objet.

[Nyko17]

Ok, alors dernière question. Imaginons que je veuille non pas passer un Vin& mais un Vin. Est possible? Si oui comment?

[Pierre.M]

C'est possible en déclarant ton op comme ça:

Code :

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

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

et en l'appelant:

Code :

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

cout << *v << endl;

mais c'est pas franchement conseillé...

[Nyko17]

Ok Ok, c'etait juste pour savoir. Bon je testerai tout ça ce soir, et je vous tiens au courant
merci encore.

[Nyko17]

Bon, merci ça marche, il fallait bien mettre *v.