Discussion :

[Torrasque]

Bonjours,

J'ai fait des modifications pour un projet open source.

Actuelement, le programme (ou plutôt jeu) affiche du texte sur la console grâce a :
info->AddLine("xxx"); et *info << "xxx";

Mes modifications m'oblige à les changer respectivement en :
guicontroller->AddText("xxx"); et *guicontroller << "xxx"

Mais comme les développeurs veulent que je fasse le moins de modifications possible à leur code et que je garde la compatibilité avec ce qu'ils ont maintenant, j'utilise #define NEW_GUI , puis #ifdef NEW_GUI ou j'en ai besoin.

Malheureusement, comme info->AddLine et *info sont utilisé partout dans le code, utiliser des #ifdef partout rendrait le tout illisible.


J'ai donc éssayer :

Code :

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

1
2
3
4
5
 
#ifdef NEW_GUI
#define info->Addline(x) guicontroller->AddText(x)
#define *info *guicontroller
#endif

Mais cela ne marche pas. (je ne m'y connais pas beaucoup en précompilation.)

Peut-être que #define AddLine(x) guicontroller->AddText(x) pourrai marcher, mais cela ne m'arrange pas car je devrais modifier le code a beaucoup d'endroits. Et cela ne marcherait de toute facon pas pour #define *info *guicontroller

J'aimerai juste que le précompilateur remplace le texte de gauche par celui de droite.
Est-ce que je demande est possible , ou doit-je chercher une autre solution?

Merci

[Laurent Gomila]

Tu ne peux pas simplement remplacer AddText par AddLine et ainsi faire un "#define info guicontroller" ?

Si ce n'est pas possible, tu peux faire une classe adapteur dont tu crées une instance info, et qui renvoie sur guicontroller. Mais c'est de la bidouille.

Je ne sais pas exactement quelle implication tu as dans ce projet, mais est-ce que tu ne peux pas essayer de mettre sur pied une solution "propre", quitte à devoir redévelopper certaines choses ?

[JolyLoic]

Je n'aime pas trop les define, ils risques de déborder... J'ai l'impression que info comme guicontroller sont des variables globales. Si tel est le cas, voici ce que je ferais (avec GuiController le type de guicontroller) :

- Ajouter une fonction AddLine dans la GuiController pour raison de compatibilité, qui appellerait AddText
- Définir :

Code :

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

1
2
GuiController guicontroller;
GuiController &info = guicontroller;

[Torrasque]

Merci des réponses, je vais éssayer.

Loulou24 :

Je ne sais pas exactement quelle implication tu as dans ce projet, mais est-ce que tu ne peux pas essayer de mettre sur pied une solution "propre", quitte à devoir redévelopper certaines choses ?

Ben, pour l'instant, je ne suis pas trop impliqué dedans. Et j'ai aussi peur de faire des modifications irréversibles sans être capable des les supporter (bugs caché, truc important qui manque, etc...) car je sais ou sont mes limites en programmation.