search

lmichel / TAP-annoter

The Unlicense
1 stars 3 forks source link

Créer mes objets TreeWalker mais avec + de méthodes #9

Open JoannVetter opened 3 years ago

JoannVetter commented 3 years ago

Comme dit ce matin, et après avoir fait fonctionné le code (enfin), j'ai commencé à le factoriser dans ma nouvelle classe "TreeWalkerMover", mais comme vous le savez peut-être il est impossible de cast un TreeWalker en TreeWalkerMover puisque le TreeWalkerMover est plus "spécifique" que le TreeWalker. A partir de la, je me disais qu'un simple appel au constructeur de TreeWalkerMover (qui est en fait un super();) résoudrait mon soucis. Problème : pour construire un TreeWalker on utilise pas un new TreeWalker(insérez paramètres quelconques) mais plutôt une méthode de la classe DocumentTransversal. Ainsi, je ne vois pas comment faire pour créer mes petits TreeWalkerMover comme je le souhaitais. Auriez vous une idée ? J'ai commit le code si jamais je ne suis pas assez clair. La partie concernée est dans le WalkerGetter, ligne 35.

lmichel commented 3 years ago

Je suis en réunion. Jr t’appelle des que je peux

Chuss/Salut/Bye from my IThing Laurent

Le 6 juil. 2021 à 15:39, JoannVetter @.***> a écrit :

 Comme dit ce matin, et après avoir fait fonctionné le code (enfin), j'ai commencé à le factoriser dans ma nouvelle classe "TreeWalkerMover", mais comme vous le savez peut-être il est impossible de cast un TreeWalker en TreeWalkerMover puisque le TreeWalkerMover est plus "spécifique" que le TreeWalker. A partir de la, je me disais qu'un simple appel au constructeur de TreeWalkerMover (qui est en fait un super();) résoudrait mon soucis. Problème : pour construire un TreeWalker on utilise pas un new TreeWalker(insérez paramètres quelconques) mais plutôt une méthode de la classe DocumentTransversal. Ainsi, je ne vois pas comment faire pour créer mes petits TreeWalkerMover comme je le souhaitais. Auriez vous une idée ? J'ai commit le code si jamais je ne suis pas assez clair. La partie concernée est dans le WalkerGetter, ligne 35.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub, or unsubscribe.

JoannVetter commented 3 years ago

On pourra regarder ça demain ? Je vais rentrer je pense, mais en tout cas tout fonctionne correctement maintenant (à part cet utilitaire) et le code est commenté

lmichel commented 3 years ago

Il faut que ton TreeWalkerMover soit composé d'un TreeWalker. Avec des accesseurs getXyz() qui encapsule les accès au TreeWalker tu dois pouvoir t'en sortir

JoannVetter commented 3 years ago

Rebonjour Laurent, j'ai beau essayer impossible d'arriver à faire un treeWalkerMover, on pourrait s'appeler pour en discuter si vous êtes disponible ?

lmichel commented 3 years ago

comme tu arrives à la fin de ton stage et que ce point n'est pas bloquant, je te propose de ne plus passer de temps dessus et de bien finaliser ton appender