[deb.] Passage d'arg. pointeur + mot const avec mes. Erreur

Bonjour,

Cette question fait plus ou moins suite a ma precedente question sur l'utilisation simultanee de plusieures class.
Probl de syntax decl de pl class

Je travaille pour le moment sur un code qui comprends plusieures class et ou une classe (A) utilise entre autre des instances d'une autre class (B)pour initialiser ses propre composants.

Pour l'instant je me limite au composant "t" de la class A qui est un pointeur
sur un array char.

Je veux ecrire une surcharge d'op. << pour imprime le contenu des carac. d'une instance et je l'ai redige de cette facon.

Code:

---

1 2 3 4 5 6 7 8 9 10 11

//Surcharge de l'operateur cout<< ostream& operator << (ostream& flux, const morceau& M){ const char* titr; cout << "\n Caracteristique du morceau : "; cout << "\n Titre : "; titr = M.get_Titre(); //Ligne 64 flux << titr; cout << "\n Duree : "; return flux; }//Fin Surcharge cout

---

Pour ce code jerecois le message d'erreur

Citation:

---

Compilateur: Default compiler
Building Makefile: "C:\Dev-Cpp\Devoir_3\Makefile.win"
Exécution de make...
make.exe -f "C:\Dev-Cpp\Devoir_3\Makefile.win" all
g++.exe -c morceauImpl.cpp -o morceauImpl.o -I"C:/Dev-Cpp/lib/gcc/mingw32/3.4.2/include" -I"C:/Dev-Cpp/include/c++/3.4.2/backward" -I"C:/Dev-Cpp/include/c++/3.4.2/mingw32" -I"C:/Dev-Cpp/include/c++/3.4.2" -I"C:/Dev-Cpp/include"

morceauImpl.cpp: In function `std::ostream& operator<<(std::ostream&, const morceau&)':
morceauImpl.cpp:64: error: passing `const morceau' as `this' argument of `const char* morceau::get_Titre()' discards qualifiers

make.exe: *** [morceauImpl.o] Error 1

Exécution terminée

---

Comme dit je n'arrive pas a comprendre ce que je dois faire avec ca.
Bien sur t est private

J'ai un selecteur ecrit ainsi avec son implementation

Code:

---

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

//retour du titre du morceau Selecteur public dans classe morceau const char* get_Titre(void);   et ici son implementation   //retour du titre du morceau const char* morceau::get_Titre(void){ return t; }   //Ici le constructeur utilise //Impl. construc. par init. morceau::morceau(char* ti, duree du){ t = ti; d = du; }

---

J'ai essaye la solution naive en ecrivant const partout ou cela avait l'air raisonnable, mais bon voila


D'avance merci

marc_3

Citation:

---

J'ai essaye la solution naive en ecrivant const partout ou cela avait l'air raisonnable, mais bon voila

---

Tu as oublié le plus important : getTitre(). En effet, cette fonction membre ne modifiant pas les données de la classe (c'est typiquement le cas pour tous les accesseurs de type get), elle doit être constante.

Code:

---

const char* get_Titre(void) const;

---

Merci pour la reponse ca marche c'est super. Par extention je veux faire la meme chose avec L'instance duree contenue dans mon objet morceau.

La seule chose que je sais faire est de reprendre l'experience precedente pour la developpe un peu plus loin jusqu'a ce que cela n'aille plus. Ici le temps de latence est trop court a mon gout.

En appliquant l'experience precedente j'ai ecrit cela

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29

J'ai toujours le meme constructeur avec l'objet duree que je cherche a reprendre pour affichage   //Impl. construc. par init. morceau::morceau(char* ti, duree du){ t = ti; d = du; }   Ici j'ai ma surcharge ou je veux maintenant charge duree en vue de reaffichage //Surcharge de l'operateur cout<< ostream& operator << (ostream& flux, const morceau& M){ const char* titr; const duree durdur; cout << "\n Caracteristique du morceau : "; cout << "\n Titre : "; titr = M.get_Titre(); //Ligne 64 flux << titr; cout << "\n Duree : "; durdur = M.get_duree(); //Ligne 68 ///////Ici idee question 1 int s, m; //decl var loc s = durdur.get_sec(); //init var loc avec selecteur de la classe durree m = durdur.get_min(); //init var loc avec selecteur de la classe durree //Affich m et s // /////// return flux; }//Fin Surcharge cout

---

Ici en suivant la decl de methode et l'implementation

Code:

---

1 2 3 4 5 6 7 8

//retour de l'instance duree decl de la methode const duree get_duree(void) const;   //retour de l'instance duree utilise dans l'objet const duree morceau::get_duree(void) const{ return d; }

---

L'idee est de charge l'instance de duree dans durdur et ensuite d'en extraire pour affichage les deux membres de donnees a travers deux selecteurs implante dans la classe durre.

Ca c'est la premiere question, est-ce une maniere de procede?

Pour la deuxieme,je traine toujours mes manques de connaissance de la syntax, et j'ai le message d'erreur suivant

Citation:

---

make.exe -f "C:\Dev-Cpp\Devoir_3\Makefile.win" all
g++.exe -c morceauImpl.cpp -o morceauImpl.o -I"C:/Dev-Cpp/lib/gcc/mingw32/3.4.2/include" -I"C:/Dev-Cpp/include/c++/3.4.2/backward" -I"C:/Dev-Cpp/include/c++/3.4.2/mingw32" -I"C:/Dev-Cpp/include/c++/3.4.2" -I"C:/Dev-Cpp/include"
morceauImpl.cpp: In function `std::ostream& operator<<(std::ostream&, const morceau&)':
morceauImpl.cpp:68: error: passing `const duree' as `this' argument of `duree& duree::operator=(const duree&)' discards qualifiers

make.exe: *** [morceauImpl.o] Error 1

Exécution terminée

---

C'est la deuxieme question quesque je dois faire lacontre?

D'avance merci

marc_3

Citation:

---

Ca c'est la premiere question, est-ce une maniere de procede?

---

Plus ou moins, sauf qu'au lieu de faire un retour par valeur pour ton accesseur get_duree (dont le retour n'a plus lieu d'être const puisqu'il s'agit de toute façon d'une copie), mieux vaut faire un retour par référence constante ; ça t'évitera une copie et les effets de bords qui peuvent en découler.

Code:

---

const duree& get_duree(void) const;

---

Code:

---

1 2 3 4 5 6

ostream& operator << (ostream& flux, const morceau& M) { ... const duree& durdur = M.get_duree(); ... }

---

Citation:

---

C'est la deuxieme question quesque je dois faire lacontre?

---

Tu as déclaré durdur en tant que constante, et tu tentes de le modifier par la suite (durdur = M.get_duree()). Cf. ce que je viens de raconter avec les références constantes.

loulou,

j'ai plus le temps maintenant avec ca. Je vais reprendre avec demain.
Merci pour le coup de main

Marc_3

Bonjour,
nouvel obstacle j'essaie de finir cette histoire de chargement des variables privee de mon objet duree, qui sont ns nombreSecondes et nm nombreMinute.

Comme dit plus haut j'ai dans ma class duree les selecteurs suivant avec leurs implementations:

Code:

---

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

//Decl methode // est retourné: le nb de sec de IC const int get_Sec(void) const;   // est retourné: le nb de min de IC const int get_Min(void) const;   //Implementation //retour de duree en secondes du morceau const int duree&::get_durSec(void) const { //Ligne 69 return ns; }   //retour de duree en secondes du morceau const int duree&::get_durMin(void) const { //Ligne 74 return nm; }

---

Ca c'est le message d'erreur que j'ai avec

Citation:

---

g++.exe -c DureeImpl.cpp -o DureeImpl.o -I"C:/Dev-Cpp/lib/gcc/mingw32/3.4.2/include" -I"C:/Dev-Cpp/include/c++/3.4.2/backward" -I"C:/Dev-Cpp/include/c++/3.4.2/mingw32" -I"C:/Dev-Cpp/include/c++/3.4.2" -I"C:/Dev-Cpp/include"

DureeImpl.cpp:69: error: expected init-declarator before '&' token
DureeImpl.cpp:69: error: expected `,' or `;' before '&' token
DureeImpl.cpp:74: error: expected init-declarator before '&' token
DureeImpl.cpp:74: error: expected `,' or `;' before '&' token
make.exe: *** [DureeImpl.o] Error 1
Exécution terminée

---

Ici en suivant c'eest ce que j'ai ecris comme surcharge d'operateur qui a besoin de ce morceau de code.

Code:

---

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

//Surcharge de l'operateur cout<< ostream& operator << (ostream& flux, const morceau& M){ const char* titr;   cout << "\n Caracteristique du morceau : "; cout << "\n Titre : "; titr = M.get_Titre(); flux << titr; cout << "\n Duree : "; const duree& durdur = M.get_duree(); //const int s, m; //decl var loc const int s& = durdur.get_Sec(); //init var loc avec selecteur de la classe durree const int m& = durdur.get_Min(); //init var loc avec selecteur de la classe durree //Affich m et s, encore a faire     return flux; }//Fin Surcharge cout

---

Quest_1) Bon evidement maintenant que je suis j'aimerai bien sortir de cette histoire avec une solution qui marche et j'aimerai bien savoir quoi faire avec. Comment faire pour resoudre le mes. d'erreur?

Quest_2) Mais en reflechissant, je me demande si je n'aurai pas mieux fait, au tout debut, de faire une autre surcharge a l'interieur de la class duree et appele cette surcharge ensuite depuis l'interieur de ma class morceau. Est-ce que ce serait mieux, et possible?

D'avance merci

marc_3

Citation:

---

Quest_1) Bon evidement maintenant que je suis j'aimerai bien sortir de cette histoire avec une solution qui marche et j'aimerai bien savoir quoi faire avec. Comment faire pour resoudre le mes. d'erreur?

---

Code:

---

1 2 3

const int duree&::get_durSec(void) const { //Ligne 69 return ns; }

---

duree&::get_durSec, ça n'a pas de sens (le & doit suivre le type du retour, pas le nom de la classe !).

C'est, au choix :

- const int duree::get_durSec() const
- const int& duree::get_durSec() const
- int duree::get_durSec() const

Le premier n'est jamais utilisé (comme je te l'ai dit, tu vas renvoyer une copie et non une référence sur la donnée membre, inutile donc qu'elle soit constante).

Le second est à utiliser systèmatiquement, sauf pour les types primitifs (comme c'est le cas ici).

Donc, utilise la troisième :wink:

Citation:

---

Quest_2) Mais en reflechissant, je me demande si je n'aurai pas mieux fait, au tout debut, de faire une autre surcharge a l'interieur de la class duree et appele cette surcharge ensuite depuis l'interieur de ma class morceau. Est-ce que ce serait mieux, et possible?

---

Effectivement, c'est ce qu'il vaut mieux faire.

OK je vais reprendre le tout depuis le debut en reprenant l'archtecture des classes, j'ai trop tourne en rond sur ce truc et je ne sais plus a qu'elle version j'en suis. En meme temp je vais ferme cette question.

Merci pour l'aide

Marc_3