Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Open
JoannVetter opened this issue Jul 6, 2021 · 5 comments
Open

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

JoannVetter opened this issue Jul 6, 2021 · 5 comments

Comments

@JoannVetter
Copy link
Collaborator

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
Copy link
Owner

lmichel commented Jul 6, 2021 via email

@JoannVetter
Copy link
Collaborator Author

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
Copy link
Owner

lmichel commented Jul 6, 2021

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
Copy link
Collaborator Author

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

@lmichel
Copy link
Owner

lmichel commented Jul 7, 2021

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants