Petit soucis java.lang.NullPointerException

Version imprimable

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:

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:

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); }
:(

essai ca :

Code:

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");}	
	}

05/07/2007, 13h52
chtig

Ca ne sera pas mieux c'est le ".getPath()" qui fait planter car ".getSelectedFile()" renvoie null ...
05/07/2007, 13h58
kha_yassine

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

ensuite sur le .getPath()
05/07/2007, 14h03
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?!:evilred:

bon du coup j'ai retesté un autre truc
Code:

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

05/07/2007, 14h56
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)
05/07/2007, 15h08
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...

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:

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.