Discussion :

[Sub]

Bonjour à tous.

Je suis programmeur amateur et j'ai voulu me créer un logger pour mes programmes. Je voulais pouvoir l'utiliser de la même façon que std::cout, lui passer du texte et des manipulateurs comme std::endl;

Mon problème c'est que dès mes premiers tests je me suis heurté à un petit problème :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
#include <iostream>
 
class CLog : public std::ostream
{
};
int main(void)
{
    CLog Log;
    Log.rdbuf(std::cout.rdbuf());
    Log << "Test" << std::endl;
 
    return EXIT_SUCCESS
}

Cela compile et voilà ce que j'obtiens à l'exécution :

Test
abnormal program termination

Le problème vient du std::endl puisque je n'ai plus le message d'erreur en l'enlevant. Par contre je n'ai aucune de la raison de cette erreur, qu'est-ce que j'ai foiré ?

Merci d'avance.

[Laurent Gomila]

Et en restaurant le streambuf de Log avant de quitter le programme ? Parce que là potentiellement tu vas avoir deux flux qui vont détruire le même streambuf.

[Sub]

Bon je crois avoir trouvé la solution à ce problème. Je vous la donne, je pense que cela peut être intéressant.

Bon tout d'abord je rappel que la programmation est un passe-temps pour moi, donc ne me lyncher pas si je dis des conneries .

J'ai trouvé dans ce liens : http://www.tacc.utexas.edu/services/...g/cre_2288.htm
qu'il fallait passer au constructeur un pointeur vers un streambuf.

The scheme for deriving from the stream classes is slightly different in that you must always provide a pointer to a stream buffer. This is because the stream classes do not contain a stream buffer, as the file or string stream classes do.

Donc voilà ce que j'aurais dû faire :

Code :

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

1
2
3
4
5
class CLog : public std::ostream
{
    public :
        CLog(std::streambuf* stream) : std::ostream(stream) {}
};

Après une petite escapade du côté du code de la SL, voilà ce que j'ai pu découvrir :
Le constructeur de ostream appel une fonction nommée init() (nom évocateur ?) de sa classe mère ios. Cette fonction prend en paramètre le fameux pointeur sur streambuf. Or rdbuf() n'appelle pas cette fonction elle.

D'ailleur le code suivant compile et fonctionne sans afficher de 'abnormal program termination' :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
class CLog : public std::ostream
{
    public :
        void Initialisation(std::streambuf *stream) {init(stream);}
};
 
int main(void)
{
    CLog Log;
    Log.Initialisation(std::cout.rdbuf());
    Log<< "test" << std::endl;
 
    return EXIT_SUCCESS;
}

Preuve qu'il est donc conseillé de fournir directement un pointeur sur stream au constructeur, à moins de vouloir se taper une fonction "Initialisation()" puisque init() est protégée.

Cependant ceci compile et fonctionne également :

Code :

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

1
2
3
4
5
class CLog : public std::ostream
{
    public :
        CLog(void) : std::ostream(0) {}
};

Apparemment l'important est que la fonction init() de la classe ios soit appelée même si on ne lui passe qu'un pointeur null. Le constructeur par défaut de ostream ne fait pas appel à cette fonction lui. Il est également protégé, ce qui empêche d'instancier un objet de type ostream.

Donc la fontion init() fait des trucs louches avec les streambuf qui sont nécessaire puisque que std::endl fait mumuse avec eux également (il les flush automatiquement, comme ça, sans demander, le malotru), d'où l'engueulade par le programme à l'exécution. D'ailleurs un std::ends fonctionne très bien avec ma classe foireuse de mon premier message.

Voilàààà, vous pouvez me balancer les pierres dessus pour les conneries que j'ai pu dire maintenant

Laurent Gomila : Alors ta solution ne fonctionnait pas mais c'est également un problème dont il faut se soucier apparemment. Dans le cas où je partage le flux de std::cout, il faut juste que je m'assure que mon objet Log soit détruit avant std::cout, ce dernier étant global cela ne devrait pas être trop dur.