Erreur dans la javadoc ?

Bonjour,

Ne sachant pas comment procéder pour remonter ma remarque à Sun, je pose mon doute ici... Puisse quelqu'un de compétent faire suivre cette info... (je sais qu'il y en a sur ce forum). Bien sûr, sous réserve que je ne dise pas de bêtises... (je met en doute Sun, je doute donc de mes dires :lol:)

Constructeur TreeModelEvent

Citation:

---

To create a node changed event for the root node, specify the parent and the child indices as null.

---

Or, si j'écris ceci :

Code:

---

new TreeModelEvent(this, (Object[])null, null, maRacineModifiée);

---

Je me prends une NullPointerException :

Code:

---

1 2 3 4

Exception in thread "AWT-EventQueue-0" java.lang.IllegalArgumentException: path in TreePath must be non null and not empty. at javax.swing.tree.TreePath.<init>(TreePath.java:60) at javax.swing.event.TreeModelEvent.<init>(TreeModelEvent.java:114) at MaClasse.fireTreeChanged(MaClasse.java:111)

---

Or, si j'écris ceci :

Code:

---

new TreeModelEvent(this, new Object[]{maRacineModifiée}, null, null);

---

Mon évènement fonctionne parfaitement...

Donc il faudrait peut être remplacer ce texte dans la javadoc par :

Citation:

---

To create a node changed event for the root node, specify the children and the child indices as null.

---

Je ne dis pas de bêtises ? :oops:
C'est peut être connu et reconnu aussi, mais je n'ai rien trouvé sur le net...

Je retire ma "correction" puisque je ne sais pas trop pourquoi, je n'arrive plus à le faire fonctionner... J'ai du me tromper...

Bref, ce post peut rejoindre celui-ci : http://www.developpez.net/forums/sho...treemodelevent

Juste une idée, tu as essayé avec l'autre constructeur de TreeModelEvent ou on passe un TreePath au lieu de Object[] ? Comme c'est le constructeur du TreePath qui t'envoie l'exception, çà marche peut être

Et je passe quoi au paramètre du constructeur du TreePath ? null ?

... ce qui me génèrera l'erreur de la même façon...

Non, le problème est que dans la javadoc ils disent qu'on peut et qu'on doit mettre null pour un évènement sur la racine, mais dans les faits, on ne peut pas passer null...

Je suis en train de regarder le code de la classe DefaultTreeModel et le code de la méthode nodeChanged :

il apparaît que l'appel d'un changement sur la racine est équivalent au suivant :

Code:

---

new TreeModelEvent(this, new Object[]{laRacine}, new int[]{-1}, new Object[]{laRacine});

---

(d'après la source : http://developer.classpath.org/doc/j...el-source.html)


J'ai testé la même chose, effectivement cela fonctionne. Donc je confirme l'erreur de la javadoc (ma tentative de correction étant fausse)...

Par contre, je ne trouve pas comment générer un évènement si la racine est supprimée ou ajoutée...

Je continue de chercher...

Bon j'ai enfin réussi à signaler à mon JTree la modification et l'ajout d'une racine comme suit :

Code:

---

1 2 3

TreeModelEvent tme = new TreeModelEvent(this, new Object[]{maRacine}, new int[]{-1}, new Object[]{maRacine}); monTreeModelListener.treeStructureChanged(tme);

---

Par contre, pour la suppression de la racine, et donc retrouver un arbre vierge, je ne trouve pas... Si quelqu'un a une idée... :roll:

J'ai essayé cela :

Code:

---

1 2 3

TreeModelEvent tme = new TreeModelEvent(this, new Object[]{maRacineSupprimee}, new int[]{-1}, new Object[]{maRacineSupprimee}); monTreeModelListener.treeNodesRemoved(tme);

---

mais sans succès :

Citation:

---

Exception in thread "Thread-38" java.lang.ArrayIndexOutOfBoundsException: node has no children
at javax.swing.tree.DefaultMutableTreeNode.getChildAt(DefaultMutableTreeNode.java:228)
at javax.swing.tree.VariableHeightLayoutCache.treeNodesRemoved(VariableHeightLayoutCache.java:543)
at javax.swing.plaf.basic.BasicTreeUI$Handler.treeNodesRemoved(BasicTreeUI.java:3917)
at MaClasse.fireRootRemoved(MaClasse.java:127)

---

Citation:

---

Envoyé par Claythest

Et je passe quoi au paramètre du constructeur du TreePath ? null ?

... ce qui me génèrera l'erreur de la même façon...

---

Oui, null et non çà ne génèrera pas la même erreur: le constructeur avec Object[] se contente d'appeler le constructeur avec TreePath en lui passant new TreePath(Object[]). C'est ce new TreePath() qui génère l'exception, mais avec le deuxième constructeur il ne sera plus appelé.
Tu auras peut-être une autre erreur, à tester.

Citation:

---

Envoyé par NicoV

Oui, null et non çà ne génèrera pas la même erreur: le constructeur avec Object[] se contente d'appeler le constructeur avec TreePath en lui passant new TreePath(Object[]). C'est ce new TreePath() qui génère l'exception, mais avec le deuxième constructeur il ne sera plus appelé.
Tu auras peut-être une autre erreur, à tester.

---

Alors, si j'ai bien compris ce que tu me dis c'est que au lieu de faire ça :

Code:

---

new TreeModelEvent(this, (Object[])null, null, maRacineModifiée);

---

Je devrais faire ça :

Code:

---

1 2	TreePath tp = new TreePath((Object[])null); new TreeModelEvent(this, tp, null, maRacineModifiée);

---

C'est ça ?

Ce qui fait que au lieu de me prendre l'exception par l'appel au constructeur de TreePath dans le constructeur de TreeModelEvent, je me la prends dans mon code, lors de mon appel au constructeur TreePath ?

Je n'en vois pas l'intérêt... Quelque chose qui m'échappe... :lol:

Quoiqu'il en soit, le problème n'est plus là. Je sais comment signifier un évènement de modification ou d'ajout de la racine, mais pas de suppression...

Si quelqu'un le sait, merci de me le communiquer :)

Non, tu n'as pas compris, au lieu de :

Code:

---

new TreeModelEvent(this, (Object[])null, null, maRacineModifiée);

---

essaye

Code:

---

new TreeModelEvent(this, (TreePath)null, null, maRacineModifiée);

---

c'est à dire l'autre constructeur

Bonjour,

je ne connais pas trop, mais si tu essayais ceci :

Code:

---

new TreeModelEvent(this, new Object[] {null}, null, maRacineModifiée);

---

C'est à dire au lieu de passer un tableau null, passer un tableau initialisé qui contient un seul null ?

Si ça ne marche pas, il serait intéressant d'aller fouiller le code source (parce qu'ils doivent bien s'en servir quelque part), ou d'aller directement par là.