Discussion :

[sylvain303]

Bonjour,

Mon C++ est un peut rouillé alors je ne vois pas encore bien où peut être le problème sur cet exemple de code opensource, que j'essaye de compiler avec g++

J'ai condensé le code dans un seul fichier pour répéter l'erreur que voici :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
g++ -c -Wall -Wno-unknown-pragmas -Wno-format -O3 -std=gnu++11 -DTIXML_USE_STL -I ~/code/ruby/gosu/MonkSVG/thirdparty/boost_1_55_0/ test_smart_ptr.cpp -o test_smart_ptr.o
test_smart_ptr.cpp: In function ‘int main(int, char* const*)’:
test_smart_ptr.cpp:33:34: error: no matching function for call to ‘MonkSVG::SVG::initialize(SVGHandler*)’
  svg_parser.initialize( &handler );
                                  ^
test_smart_ptr.cpp:33:34: note: candidate is:
test_smart_ptr.cpp:20:8: note: bool MonkSVG::SVG::initialize(MonkSVG::ISVGHandler::SmartPtr)
   bool initialize( ISVGHandler::SmartPtr handler ) { return true; }
        ^
test_smart_ptr.cpp:20:8: note:   no known conversion for argument 1 from ‘SVGHandler*’ to ‘MonkSVG::ISVGHandler::SmartPtr {aka boost::shared_ptr<MonkSVG::ISVGHandler>}’

le code source ici http://pastie.org/8809379#1

Pour moi, étant donné que SVGHandler dérive de MonkSVG::ISVGHandler, il devrait être éligible comme un boost::shared_ptr<ISVGHandler> (ligne 10) ?

après le code était prévu pour xcode sur mac et j'ai essayé de le compilé sous linux.
Le code original, une fois remis en place avec les contraintes GNU/linux, donne le même genre d'erreur. Le code est un peu ancien, j'ai pris la dernière version de boost.

J'ai essayé de remplacer les lignes 33 34 par…

Code :

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

1
2
3
 
	MonkSVG::ISVGHandler::SmartPtr handler(new SVGHandler);
	svg_parser.initialize( handler );

le compilateur aime encore moins…

Si vous voyez un truc flagrant, je suis preneur.

Cordialement,
Sylvain.

[JolyLoic]

Pour l'exemple que tu as mis, je n'aime pas plus ce que tu as écrit que le compilateur, et je suis bien content qu'il te le refuse :
La construction d'un shared_ptr à partir d'un pointeur est explicite. Ce qui signifie que tu ne peut pas l'utiliser pour convertir à la volée un pointeur en shared_ptr. Et c'est une très bonne chose ici. Un shared_ptr a pour objectif de gérer la durée de vie de l'objet pointé, en le détruisant quand il n'y en a plus besoin. Il est propriétaire (avec propriété partagée) de cet objet. Or là, la durée de vie de handler est déjà gérée : C'est une variable locale qui sera détruite dès que l'on sortira du scope ! Tu vas donc faire un delete d'une variable gérée dans la pile, c'est une bonne chose que le compilateur te montre explicitement ce que tu allais faire !

La seconde écriture m' l'air plus correcte (mais je ne suis pas encore totalement réveillé). Tu peux me donner l'erreur de ton compilateur ?

Sachant que je ne trouve pas cette écriture très belle quand même :
- Quand on veut créer un shared_ptr et l'objet pointé simultanément, il vaut mieux utiliser [std/boost]::make_shared (plus simple, plus exception safe, et plus performant).
- J'ai des doutes sur l’intérêt de passer un shared_ptr en argument de initialize, plutôt qu'une simple référence. Mais il faudrait plus d'infos sur ce que cette fonction est sensée faire pour confirmer/infirmer ces doutes.

[sylvain303]

Bonjour,

Pour l'exemple que tu as mis, je n'aime pas plus ce que tu as écrit que le compilateur, et je suis bien content qu'il te le refuse :
La construction d'un shared_ptr à partir d'un pointeur est explicite. Ce qui signifie que tu ne peut pas l'utiliser pour convertir à la volée un pointeur en shared_ptr.

Je ne suis pas l'auteur du code, j'essaye juste de le faire fonctionner. Je ne sais pas même pas encore si l'utilisation des shared_ptr a un intérêt dans ce cas.

La seconde écriture m' l'air plus correcte (mais je ne suis pas encore totalement réveillé). Tu peux me donner l'erreur de ton compilateur ?

Je vais relire l'erreur, on dirait une erreur de linker seulement. C'est peut être bon finalement… J'ai pas le temps tout de suite mais je regarderai un peu plus tard.

- J'ai des doutes sur l’intérêt de passer un shared_ptr en argument de initialize, plutôt qu'une simple référence. Mais il faudrait plus d'infos sur ce que cette fonction est sensée faire pour confirmer/infirmer ces doutes.

Alors de ce que je comprends ce code qui est un parser simplifié de SVG, récupère un handler qui doit fournir une série de méthodes concrètes qui géreront, à la convenance du développeur, les informations extraites du SVG.

Dans le pastie j'ai écourté le code. Mais si j'ai bien compris, le MonkSVG::ISVGHandler définit une interface ou une classe abstraite qui donne les protypes des types d'appels à implémenter pour le parser (class SVG). Je vois l'intérêt de cette forme de flexibilité, mais je ne peux pas encore juger de la pertinence de ce passage avec un smart_ptr…

Note bien, que ce n'est pas ma question de la pertinence de ce code. Je vais m'adresser à l'auteur pour ça.

Merci de tes remarques, ça confirme mes soupçons et m'aiguille sur une bonne réflexion.
Sylvain.

[sylvain303]

http://pastie.org/8812830

Voici le message du compilateur, c'est un warning :

Code :

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

1
2
/home/sylvain/code/ruby/gosu/MonkSVG/thirdparty/boost_1_55_0/boost/checked_delete.hpp:34:5:
warning: deleting object of polymorphic class type ‘SVGHandler’ which has non-virtual destructor might cause undefined behaviour [-Wdelete-non-virtual-dtor]

bref pas top tout ça…

[JolyLoic]

Il a encore une fois bien raison ton compilateur, tu devrait l'en remercier

Ajoute ~ISVGHandler() {} dans la classe ISVGHandler. Voir : http://cpp.developpez.com/faq/cpp/?p...UCTEUR_virtuel

[sylvain303]

Il a encore une fois bien raison ton compilateur, tu devrait l'en remercier

Merci. C'est fait déjà. Mais il est timide et refuse les compliments…

Code :

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

1
2
3
4
g++ --merci
g++: error: unrecognized command line option ‘--merci’
g++: fatal error: no input files
compilation terminated.