Discussion :

[yrlac]

Bonjour a tous,

débutant en java je suis un peu perdu.

Voila mon soucis, j ai créée une fenêtre avec differents menu, dont un menu ouvrir (via JFileChooser).
Bon jusque la ca va.

Mais ensuite dans les autres menu je voudrais effectuer des actions sur le fichier ouvert (comme le lire par exemple)
ceci marche bien mais a condition qu'il y ait bien un fichier ouvert!

si on choisit lire et qu'il n'y a pas de fichier ouvert on obtient une "java.lang.NullPointerException"

j'ai essayer de la lever de plusieurs façon mais je seche.

je mets juste la partie "lecture"

Code :

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

1
2
3
4
5
6
7
8
 
 public void lecture (ActionEvent e) {
	 String p=fileChooser.getSelectedFile().getPath();
	 if (p==null){
 
		throw new NullPointerException("pas de fichier");}
 		lireFichier(p);
	}

ce qui ne marche pas

j'ai réflechi un peu (pas assez ^^) et j'ai tenté de lever l exception avant :

Code :

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

1
2
3
4
5
6
7
8
 
 public void lecture (ActionEvent e) {
if (fileChooser.getSelectedFile().getPath()==null){
		throw new NullPointerException("pas de fichier");}
String p= fileChooser.getSelectedFile().getPath();
		lireFichier(p);
 
}

[kha_yassine]

essai ca :

Code :

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

1
2
3
4
5
6
7
public void lecture (ActionEvent e) {
	 String p=fileChooser.getSelectedFile().getPath();
	try { 
              lireFichier(p);
              } catch (NullPointerException e)
                  {System.out.println("pas de fichier");}	
	}

[chtig]

Ca ne sera pas mieux c'est le ".getPath()" qui fait planter car ".getSelectedFile()" renvoie null ...

[kha_yassine]

Alors il faut tester sur le .getSelectedFile() s'il retourn null

ensuite sur le .getPath()

[yrlac]

j'ai essayer et effectivement ça ne marche pas non plus...
du coup j'ai testéle .getSelectedFile() ca ne fonctionne pas non plus?!

[yrlac]

bon du coup j'ai retesté un autre truc

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

public void lecture (ActionEvent e) {
	File fichier=fileChooser.getSelectedFile();
	try{
		test(fichier);
		}	
		
	catch	(NullPointerException e1)
		{System.out.println("pas de fichier");}
		
	}
	
public void test (File f){
	String p=fileChooser.getSelectedFile().getPath();
	try { 
              lireFichier(p);
              } 
	catch (NullPointerException e)
                  {System.out.println("pas de fichier");}
}

Mais ça ne marche pas plus...
de meme je comprend pas pourquoi le faite de mettre e a la place de e1 me déclenche une erreur

[chtig]

"Ca marche pas" ce n'est clairement pas du langage Java !
Merci d'apprendre les rudiments pour déclarer un problème. Et quand tu as des erreurs, la première chose est de les lire (cf ta question e1 vs e)

[yrlac]

disons ce que je comprend pas c'est pourquoi ca passait dans le code de Kha_yassine et pas dans celui la?

lire l erreur je peux le faire ça y a pas de soucis (surtout qu'Eclipse te l indique clairement) c'etait juste pour essayer de comprendre...

[vincent63]

Ah mon avis, si ça ne marche pas, c'est parce que tu n'as toujours pas blindé ton code.

Ta méthode Test(fichier) appelle toujours fileChooser.getSelectedFile().getPath() (ce qui parrait curieux puisque tu as déjà passé le fichier en paramètre). Mais à aucun moment tu ne testes la non nullité de ton object file.

Essaie de mettre un truc comme ça :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
	public void lecture(ActionEvent e) {
		File file = fileChooser.getSelectedFile();
		if (file == null) {
			System.err
					.println("L'utilisateur n'a pas choisi de fichier, on sort de la méthode");
			return;
		}
		String path = file.getPath();
		lireFichier(path);
 
	}

Ca devrait aller mieux.