Discussion :

[Bouteille]

Bonjour,

récemment, j'ai fait quelque chose qui marche, mais qui, à mon avis, est assez moche.
J'ai une classe CMyStream qui dérive de std::ostream et qui redéfinit quelques petites choses...

Or dans mon code, quand je veux utiliser mon stream, je ne peux pas faire ça :

Code :

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

1
2
CMyStream monStreamTest(parametres_Constructeur);
monStreamTest << "Salut tout le monde" << std::endl;

En effet, l’opérateur "<<" n'est pas redéfini dans ma classe (ben oui je vais pas m’embêter) et je ne peux pas profiter de celui de ostream par polymorphisme (argh c'est pourtant ça que je voudrais, je suis feignant moi) car monStreamTest n'est ni un pointeur ni une référence.

A l'inverse, ceci marche très bien :

Code :

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

1
2
3
4
ostream* monStreamTest=new CMyStream(parametres_Constructeur);
(*monStreamTest) << "Salut tout le monde" << std::endl;
// ...
delete(monStreamTest); // oui oui le destructeur est virtuel toussa toussa ;)

Mais franchement je ne trouve pas ça super mignon (le déréférencement) et pas naturel a taper. Un simple "monStreamTest" serait tellement plus esthétique.

Alors j'ai trouvé ça :

Code :

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

1
2
3
ostream& monStreamTest=*(new CMyStream(parametres_Constructeur));
monStreamTest << "Salut tout le monde" << std::endl;
// ...

Ça marche, c'est plus joli à utiliser... Mais franchement j'ai un peu peur de me faire taper dessus... Je ne pense pas que les références aient été prévues pour ça, et je suppose que je dois deleter mon allocation en mémoire dynamique mais j'ai peur que le programme m'en veuille de venir souiller sa référence comme ça... Et je ne saurais même pas la syntaxe correcte..
Bref je soupçonne le gros truc dégueulasse.

En gros, j'aimerais votre avis, et si ça vous irrite les yeux, dites moi ce que je peux faire (une solution de débutant tellement plus évidente, élégante et sure..)

Merci pour votre aide et dsl pour le choc !

[koala01]

Salut,

A vrai dire, tant que tu utilises le type réel de ta classe, il n'y a meme pas besoin de recourir à l'allocation dynamique de la mémoire .

Tu pourrais donc parfaitement écrire un code proche de

Code :

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

1
2
CMyStream  monStreamTes(parametres_Constructeur);
monStreamTest << "Salut tout le monde" << std::endl;

Par la suite, si tu le transmet à une fonction qui prend une référence sur un ostream, tu n'auras, a priori, pas de problème non plus (pour autant que les conditions d'héritage soient remplies )

Le seul cas où tu auras, efffectivement, besoin de l'allocation dynamique, c'est si tu veux utliser un ostream dont le type réel sera déterminé à l'exécution, mais je présume, si tu as pris la peine de faire dériver ta classe de ostream, que ce cas risque d'être relativement peu fréquent

[Jean-Marc.Bourguet]

J'ai une classe CMyStream qui dérive de std::ostream et qui redéfinit quelques petites choses...

Generalement, c'est mal fait.

Or dans mon code, quand je veux utiliser mon stream, je ne peux pas faire ça :

Code :

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

1
2
CMyStream monStreamTest(parametres_Constructeur);
monStreamTest << "Salut tout le monde" << std::endl;

C'est signe que c'est mal fait. Jouons aux devinettes, parmi les choses que tu as definie dans CMyStream il y a un operator<< et celui masque celui de la classe de base. Donc il suffit d'ajouter

Code :

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

1
2
 
using std::ostream::operator<<;

dans ta definition de CMyStream.

(Ca reste mal fait, j'ai du mal a voir les cas ou definir un operator<< dans un descendant de ostream est la chose a faire).

[Bouteille]

Woups je crois que je me suis mal exprimé. Merci pour vos commentaires.

En réalité, je ne redéfinis pas du tout l’opérateur << de ostream dans ma classe, au contraire je n'y touche pas. Pour les détails vous avez tout le code dans le dernier message ici (ça a juste un peu été amélioré depuis mais le mécanisme d'héritage n'a pas bougé) : Ajouter une en-tete à un ostream

Je touche juste au streambuf. Je ne voudrais du coup pas masquer l’opérateur de ostream mais en tirer bénéfice en l'utilisant pour CMyStream.

Or quand j'essaye dans mon programme le premier encart que j'ai posté, j'ai à la compilation :

In function 'int main()':
erreur: invalid operands of types 'CMyStream()' and 'const char [20]' to binary 'operator<<'
||=== Build finished: 1 errors, 0 warnings ===||

Je dois passer a coté de quelque chose d’évident sur l'héritage.. J’espère... En attendant je vais relire le cours sur le site

[Jean-Marc.Bourguet]

C'est plus simple avec le message d'erreur. Tu as declare

Code :

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

1
2
 
CMyStream foo();

en pensant definir une variable de type CMyStream alors que tu as declare une fonction qui retourne un CMyStream.

[Bouteille]

Ah...
Hum...

Ben oui ca marche... Je crois que ce n'est pas la première fois en plus.
Désolé de vous avoir fait perdre votre temps.
Merci...

Au passage si vous avez des remarques sur limplémentation du CMyStream dans l'autre message je suis preneur.