Sauvegarde d'un JTree et bouton d'ajout de JTextField dans JOptionPane

**Akame14** · 26/06/2017, 09h57

Bonjour,

J'ai deux questions.

Premièrement j'ai un JTree que je souhaite sauver, voici mon code pour le sauvegarder:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
 
 public void save(JTree tree){//ne garde pas les retours à la ligne
	 	DefaultMutableTreeNode node1;
	 	DefaultMutableTreeNode node;
	 	JFileChooser chooser = new JFileChooser();
	 	System.out.println("1");
	 		chooser.setApproveButtonText("Ok");
	 		chooser.showOpenDialog(null);
	 		if (chooser.showOpenDialog(null) == JFileChooser.APPROVE_OPTION){	
	 			System.out.println("2");
	 			try{
 
			 		FileWriter fw = new FileWriter(chooser.getSelectedFile());
			 		chooser.setApproveButtonText("Ok"); //intitulé du bouto
			 		System.out.println("3");
			 		BufferedWriter output = new BufferedWriter(fw);
			 		String texte = centre.getTextArea()+"\r\n";
					output.write(texte+"\r\n");
					System.out.println("4");
					DefaultTreeModel model = (DefaultTreeModel) tree.getModel();
					DefaultMutableTreeNode racine = (DefaultMutableTreeNode) model.getRoot();
					System.out.println("5");
					int a = racine.getChildCount();
					int b;
					System.out.println("6");
					for(int i = 1; i<=a;i++){
						System.out.println("7");
						node = (DefaultMutableTreeNode)racine.getChildAt(i);
						output.write(node.toString());
						b = node.getChildCount();
						for(int j =1;j<=a;j++){
							System.out.println("8");
							node1 = (DefaultMutableTreeNode)node.getChildAt(j);
							output.write(node1.toString());
							System.out.println("9");
						}
						output.write("\r\n");
						System.out.println("10");
					}
					output.flush();
					output.close();
					System.out.println("fichier enregistré");
	 			}
	 			catch(IOException ioe){
					System.out.print("Erreur : ");
					ioe.printStackTrace();
				}
			}
	 }

Sachant que mon arbre est sur trois niveaux. Voici l'erreur:

Code :

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

1
2
3
4
5
6
7
8
9
 
     [java] 1
     [java] Exception in thread "AWT-EventQueue-0" java.lang.NullPointerException
     [java] 2
     [java] 3
     [java]     at control.RWFile.save(Unknown Source)4
     [java]
     [java]     at control.MyActionListener.actionPerformed(Unknown Source)
     [java]     at javax.swing.AbstractButton.fireActionPerformed(AbstractButton.java:2022)

Un fichier vide est créé.

Deuxièmement, j'ai un JOptionPane qui contient un JPanel JOptionPane.showMessageDialog(null,req,"Blabla",JOptionPane.PLAIN_MESSAGE); lui-même contenant trois JTextField de base et un bouton permettant d'ajouter un JTextField à l'optionPane. J'ai ajouté des écouteurs dans ma classe Action Listener voici mon code:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
 
if(evt.getSource()==centre.getRequete()){
			System.out.println("1");
			Requete req = new Requete();//Probleme ici
			and = new ArrayList<String>();
			System.out.println("2");
			req.addListener(this);
			System.out.println("3");
			if(evt.getSource()==req.getButtonAdd()){
				System.out.println("5");
				field = new JTextField();
				req.add(field);
				and.add(field.getText());
			}
			JOptionPane.showMessageDialog(null,req,"Faire une requete",JOptionPane.PLAIN_MESSAGE);
}

Lorsque j'appuie sur mon bouton rien ne se passe.

Merci d'avance

**joel.drigo** · 26/06/2017, 12h24

Salut,

Quand tu as une NullPointerException, il te suffit de regarder la stack trace pour savoir où aller regarder et chercher ce qui pourrait être null dans la ligne de code concernée. Le mode debug peut également te permettre de plus facilement repérer ce qui est null. Pour nous, c'est déjà beaucoup plus difficile : on n'a pas tout ton code, les numéros de ligne sont probablement décalés et, bien sûr, on ne peut pas tester en debug ton application. Je dirais au pif que c'est tree qui est null, mais vraiment au pur pif.

Code :

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

1
2
3
4
int a = racine.getChildCount();
					int b;
					System.out.println("6");
					for(int i = 1; i<=a;i++){

Comme dans quasiment tous les cas de listes, tableaux et assimilés en Java, les indices vont de 0 à n exclu, pas de 1 à n inclus !

Code :

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

1
2
3
chooser.setApproveButtonText("Ok");
	 		chooser.showOpenDialog(null);
	 		if (chooser.showOpenDialog(null) == JFileChooser.APPROVE_OPTION){

Pourquoi afficher deux dialogues ? Un seul suffit, surtout que tu n'exploites pas la réponse du premier.

Code :

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

1
2
FileWriter fw = new FileWriter(chooser.getSelectedFile());
chooser.setApproveButtonText("Ok"); //intitulé du bouto

Pourquoi changer le nom du bouton du FileChooser ici ? Non seulement il a déjà été changé pour la même valeur avant, mais ça fait longtemps que le dialogue n'est plus affiché et ne le sera plus ensuite.

Ta gestion de la fermeture de flux est à revoir : en cas d'exception, le fichier ne sera pas fermé (le flush() est inutile si tu fais un close()). L'utilisation d'un try-with-resource te simplifierait bien tout ça.

**Akame14** · 26/06/2017, 15h10

Je ne connaissais pas le try-with-resource. J'ai modifié mon code comme vous avez dit mais j'ai toujours une erreur et mon fichier est toujours vide.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
 
public void save(JTree tree){//ne garde pas les retours à la ligne
	 	DefaultMutableTreeNode node1;
	 	DefaultMutableTreeNode node;
	 	JFileChooser chooser = new JFileChooser();
	 	System.out.println("1");
	 		chooser.showOpenDialog(null);
	 		if (chooser.showOpenDialog(null) == JFileChooser.APPROVE_OPTION){	
	 			System.out.println("2");
	 			try{
 
			 		FileWriter fw = new FileWriter(chooser.getSelectedFile());
			 		System.out.println("3");
			 		BufferedWriter output = new BufferedWriter(fw);
			 		String texte = centre.getTextArea()+"\r\n";
					output.write(texte+"\r\n");
					System.out.println("4");
					DefaultTreeModel model = (DefaultTreeModel) tree.getModel();
					DefaultMutableTreeNode racine = (DefaultMutableTreeNode) model.getRoot();
					System.out.println("5");
					int a = racine.getChildCount();
					int b;
					System.out.println("6");
					for(int i = 0; i<a;i++){
						System.out.println("7");
						node = (DefaultMutableTreeNode)racine.getChildAt(i);
						output.write(node.toString());
						b = node.getChildCount();
						for(int j =0;j<a;j++){
							System.out.println("8");
							node1 = (DefaultMutableTreeNode)node.getChildAt(j);
							output.write(node1.toString());
							System.out.println("9");
						}
						output.write("\r\n");
						System.out.println("10");
					}
					output.flush();
					output.close();
					System.out.println("fichier enregistré");
	 			}
	 			catch(IOException ioe){
					System.out.print("Erreur : ");
					ioe.printStackTrace();
				}
			}
	 }

Et mon erreur:

Code :

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

1
2
 
 Exception in thread "AWT-EventQueue-0" java.lang.NullPointerException3

Je viens de voir que j'ai une ligne de code : FileWriter fw = new FileWriter(chooser.getSelectedFile()); mais l'utilisateur n'a pas à selectionné de fichier puisqu'il souhaite enregistrer et non sauver le fichier. De plus, l'optionPane s'affiche deux fois je pensais qu'en supprimant le double dialogue ca allait marcher mais non.

**joel.drigo** · 26/06/2017, 15h18

Envoyé par Akame14

Je ne connaissais pas le try-with-resource. J'ai modifié mon code comme vous avez dit mais j'ai toujours une erreur et mon fichier est toujours vide.

Je ne vois pas trop où tu aurais modifié ton code. Je ne vois pas de try-with-resource, je vois toujours 2 dialogues ouverts successivement. Ah, si, tu as modifié le parcours des nodes.

Envoyé par Akame14

Je viens de voir que j'ai une ligne de code : FileWriter fw = new FileWriter(chooser.getSelectedFile()); mais l'utilisateur n'a pas à selectionné de fichier puisqu'il souhaite enregistrer et non sauver le fichier.

Forcément, si aucun fichier n'est sélectionné, getSelectedFile() retourne null et la construction du FileWriter va planter. J'imaginais que au moins pour tes tests tu sélectionnais un fichier. Le JFileChooser n'empêche pas effectivement un approve et une selection nulle. Il faut traiter le cas dans le if. Attention, parce qu'on peut également choisir un dossier (donc à gérer aussi comme cas).
En revanche, je ne vois pas trop la différence que tu fais entre "enregistrer" et "sauver". Mais, en tout cas, pourquoi afficher un dialogue de choix de fichiers si ce n'est pour choisir un fichier (ou un dossier) ? Cela dit tu affiches un dialogue de sélection de fichier pour lecture (open), donc déjà là il y a un problème à ce niveau.

Envoyé par Akame14

De plus, l'optionPane s'affiche deux fois je pensais qu'en supprimant le double dialogue ca allait marcher mais non.

Bah, normalement oui, mais il faut le supprimer pour que ça le supprime.

**Akame14** · 26/06/2017, 15h30

D'accord mais qu'est-ce que je met dans mon FileWriter = new FileWriter? Etant donné que c'est la personne qui choisit le nom du fichier.
Concernant le try with ressource j'ai trouvé des choses comme ça:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
try {
     try (BufferedReader bufferedReader = new BufferedReader(new
         FileReader("C:/Users/jm/AppData/Local/Temp/monfichier.txt"))) {
        String ligne=null;
        while ((ligne = bufferedReader.readLine()) != null) {
             System.out.println(ligne);
        }
     }
}catch (IOException ioe) {
     ioe.printStackTrace();
}

je dois donc mettre l'initalisation de mon filewriter à la place du bufferedReader dans cet exemple?

**joel.drigo** · 26/06/2017, 18h02

Envoyé par Akame14

D'accord mais qu'est-ce que je met dans mon FileWriter = new FileWriter? Etant donné que c'est la personne qui choisit le nom du fichier.

La méthode getSelectedFile() retourne bien ce que l'utilisateur choisit. Mais ça peut être rien (null), un fichier ou un dossier. Cela peut être un fichier existant ou pas, un dossier existant ou pas.
Tu peux configurer le JFileChooser pour empêcher la sélection de dossier.

Donc globalement, il faudrait faire quelque chose comme ça :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
File file = chooser.getSelectedFile();
if ( file==null ) {
    // rien, on annule la sauvegarde (ou on pourrait redemander un fichier à l'utilisateur)
 
} else if ( file.isDirectory() ) {
    // c'est un dossier qui existe
    // soit tu annules, soit tu fais un fichier par défaut (mais attention à l'écrasement possible d'un fichier déjà existant, il faudra par exemple faire une boucle comme ça) :
    File targetFile = new File(file, "sans nom.text");
    boolean newFile = targetFile.createNewFile(); 
    for(int i=1;  !newFile; i++) {
        File targetFile = new File(file, "sans nom ("+i+").text");
        newFile = targetFile.createNewFile(); 
    } 
 
}
else if ( file.isFile() ) {
       // c'est une fichier qui existe : il faut éventuellement demander à l'utilisateur de confirmer qu'il veut bien remplacer le fichier
}
else {
       // le fichier n'existe pas
       file.getParentFile().mkdirs(); // on créé les dossiers au cas où ils n'existeraient pas déjà
       file.createNewFile(); // on créé éventuellement le fichier pour qu'il existe
}

Envoyé par Akame14

Concernant le try with ressource j'ai trouvé des choses comme ça:

Les deux trys peuvent être fusionnés, et on peut simplement faire :

Code :

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

1
2
3
4
5
6
try(BufferedReader reader = Files.newBufferedReader(file.toPath()) {
 
}
catch(IOException e) {
      ... traitement ...
}

Seulement, ce code est fait pour lire, pas pour écrire....

Envoyé par Akame14

je dois donc mettre l'initalisation de mon filewriter à la place du bufferedReader dans cet exemple?

donc, oui, pour lire c'est un peu la même chose sauf qu'à la place de reader on met writer, à peu de choses près (voir les options possibles).

Code :

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

1
2
3
4
5
6
try(BufferedWriter writer = Files.newBufferedWriter(file.toPath()) {
 
}
catch(IOException e) {
      ... traitement ...
}

Mais dans le bloc, évidemment, on ne fait pas une boucle pour lire des lignes, mais on écrit.

**Akame14** · 26/06/2017, 21h46

J'ai essayé ton code et j'ai fais en sorte de redemander si le fichier est null mais ce que je ne comprends c'est que l'utilisateur n'as pas a selectionné le nom du fichier, il faut juste qu'il rentre le nom du fichier qu'il veux enregistrer. J'ai aussi une erreur :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
 
 Exception in thread "AWT-EventQueue-0" java.lang.NullPointerException
     [java]     at control.RWFile.save(Unknown Source)
     [java]     at control.MyActionListener.actionPerformed(Unknown Source)
     [java]     at javax.swing.AbstractButton.fireActionPerformed(AbstractButton.java:2022)
     [java]     at javax.swing.AbstractButton$Handler.actionPerformed(AbstractButton.java:2348)
     [java]     at javax.swing.DefaultButtonModel.fireActionPerformed(DefaultButtonModel.java:402)
     [java]     at javax.swing.DefaultButtonModel.setPressed(DefaultButtonModel.java:259)
     [java]     at javax.swing.AbstractButton.doClick(AbstractButton.java:376)
     [java]     at javax.swing.plaf.basic.BasicMenuItemUI.doClick(BasicMenuItemUI.java:833)
     [java]     at javax.swing.plaf.basic.BasicMenuItemUI$Handler.mouseReleased(BasicMenuItemUI.java:877)
     [java]     at java.awt.Component.processMouseEvent(Component.java:6535)
     [java]     at javax.swing.JComponent.processMouseEvent(JComponent.java:3324)
     [java]     at java.awt.Component.processEvent(Component.java:6300)
     [java]     at java.awt.Container.processEvent(Container.java:2236)
     [java]     at java.awt.Component.dispatchEventImpl(Component.java:4891)
     [java]     at java.awt.Container.dispatchEventImpl(Container.java:2294)
     [java]     at java.awt.Component.dispatchEvent(Component.java:4713)
     [java]     at java.awt.LightweightDispatcher.retargetMouseEvent(Container.java:4888)
     [java]     at java.awt.LightweightDispatcher.processMouseEvent(Container.java:4525)
     [java]     at java.awt.LightweightDispatcher.dispatchEvent(Container.java:4466)
     [java]     at java.awt.Container.dispatchEventImpl(Container.java:2280)
     [java]     at java.awt.Window.dispatchEventImpl(Window.java:2750)
     [java]     at java.awt.Component.dispatchEvent(Component.java:4713)
     [java]     at java.awt.EventQueue.dispatchEventImpl(EventQueue.java:758)
     [java]     at java.awt.EventQueue.access$500(EventQueue.java:97)
     [java]     at java.awt.EventQueue$3.run(EventQueue.java:709)
     [java]     at java.awt.EventQueue$3.run(EventQueue.java:703)
     [java]     at java.security.AccessController.doPrivileged(Native Method)
     [java]     at java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(ProtectionDomain.java:76)
     [java]     at java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(ProtectionDomain.java:86)
     [java]     at java.awt.EventQueue$4.run(EventQueue.java:731)
     [java]     at java.awt.EventQueue$4.run(EventQueue.java:729)
     [java]     at java.security.AccessController.doPrivileged(Native Method)
     [java]     at java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(ProtectionDomain.java:76)
     [java]     at java.awt.EventQueue.dispatchEvent(EventQueue.java:728)
     [java]     at java.awt.EventDispatchThread.pumpOneEventForFilters(EventDispatchThread.java:201)
     [java]     at java.awt.EventDispatchThread.pumpEventsForFilter(EventDispatchThread.java:116)
     [java]     at java.awt.EventDispatchThread.pumpEventsForHierarchy(EventDispatchThread.java:105)
     [java]     at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:101)
     [java]     at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:93)
     [java]     at java.awt.EventDispatchThread.run(EventDispatchThread.java:82)

**joel.drigo** · 27/06/2017, 00h13

Envoyé par Akame14

J'ai essayé ton code et j'ai fais en sorte de redemander si le fichier est null mais ce que je ne comprends c'est que l'utilisateur n'as pas a selectionné le nom du fichier, il faut juste qu'il rentre le nom du fichier qu'il veux enregistrer.

C'est exactement pareil : qu'il saisisse un nom (de fichier qui existe ou n'existe pas) ou qu'il sélectionne un fichier dans la liste. On récupère le choix de l'utilisateur par getSelectedFile(), c'est tout.

J'ai aussi une erreur :
[CODE]
Exception in thread "AWT-EventQueue-0" java.lang.NullPointerException
[java] at control.RWFile.save(Unknown Source)
[/QUOTE]
Je ne sais pas ce qu'il y a dans la ligne qui cause l'erreur, donc je ne peux rien savoir de la cause de l'erreur. Si tu ne me montres pas la ligne qui cause l'erreur, je ne pourrais rien faire pour toi. Et quand bien même, comme je ne peux pas connaître tout le process qui a conduit à cette ligne, je ne pourrais faire que des suputations. Je te l'ai déjà dit, il te suffit d'exécuter ton code en mode debug, en pas à pas, pour voir directement toi-même la cause de l'erreur. La lecture et l'interprétation d'un code a ses limites.

**Akame14** · 27/06/2017, 08h24

D'accord mais comment executes-t-on en mode pas à pas? je n'ai jamais fais ça

**Akame14** · 27/06/2017, 08h51

Je ne sais pas si c'était la bonne commande mais j'ai trouvé ça:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
..\src\control\RWFile.java:95: warning: [unchecked] unchecked call to ArrayList(Collection<? extends E>) as a member of the raw type ArrayList
                        mdp = new ArrayList(Arrays.asList(allMdp));
                              ^
  where E is a type-variable:
    E extends Object declared in class ArrayList
..\src\control\RWFile.java:95: warning: [unchecked] unchecked conversion
                        mdp = new ArrayList(Arrays.asList(allMdp));
                              ^
  required: ArrayList<String>
  found:    ArrayList
2 warnings

**joel.drigo** · 27/06/2017, 09h04

C'est une trace de compile ça ! On ne risque pas d'y voir une erreur d'exécution.

C'est dans l'idée d'exécuter ton code en mode debug que tu as fait ça ? Tu ne connais pas le mode debug ? Tu utilises quel EDI ?

**Akame14** · 27/06/2017, 09h14

Eh bien j'utilise le Java normal JDK je ne sais pas trop si c'est ça l'EDI

**joel.drigo** · 27/06/2017, 09h19

L'EDI, c'est Eclipse, ou NetBeans, ou IntellijIdea... L'outil d'aide au développement, avec l'éditeur, tout ça.

**Akame14** · 27/06/2017, 09h25

Je crois que nous n'en utilisons pas ca me dit rien

**joel.drigo** · 27/06/2017, 09h28

Tu veux dire que tu codes avec un éditeur de texte, compile en console avec javac et exécute avec java.exe ? Je te conseille vivement d'utiliser un EDI, ça te simplifierait grandement la programmation.
Déboguer en console c'est possible certe, mais c'est ardu et fastidieux.

**Akame14** · 27/06/2017, 09h35

Je code avec sublime text et oui on executes dans la console, c'est la façon qu'on nous as appris en cours

**joel.drigo** · 27/06/2017, 09h45

C'est assurément très formateur de vous faire programmer avec un éditeur et de compiler en console. Mais contre productif (personne ne fait ça en entreprise, à part quelques furieux ou nostaligiques, et encore, ils vont plutôt utiliser VIM, voire plus roots encore

) M'enfin, on ferait mieux de vous apprendre à déboguer. Quand on a compilé 1 fois en console, on sait le faire. Cela ne sert pas à grand chose de faire un projet complet comme ça

Mais bon admettons.
Alors plus simple avant de passer au mode debug en console (!), ajouter une option de compilation pour avoir les numéros de lignes et références aux sources entre autres : -g au lieu de g:none, au moins tu auras dans ta trace le numéro de ligne où survient l'erreur !

**Akame14** · 27/06/2017, 10h04

Je doit être idiote mais ca ne marche pas. J'ai mis -g pendant la compilation et ca n'a pas mis d'erreur mais quand j'execute avec le -g ca me met

Code :

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

1
2
3
Unrecognized option: -g
Error: Could not create the Java Virtual Machine.
Error: A fatal exception has occurred. Program will exit.

**joel.drigo** · 27/06/2017, 11h39

Compilation = javac.

**Akame14** · 27/06/2017, 11h49

Lorsque je fais ca: javac -g -d ../class ../src/view/* je n'ai aucune erreur

Sauvegarde d'un JTree et bouton d'ajout de JTextField dans JOptionPane

Composants Java

Discussions similaires

Partager

Partager