falsche Pfadangabe bei FileChooser

[Corcovado]

Hallo,
gibts ne einfache Moeglichkeit (ne Methode oder so) bei nem JFileChooser.showOpenDialog() nach zu pruefuen ob ein File leer ist oder einen Pfad irrsinnig, der also zB nicht existiert? Wenn man zB "asdfasdfasdf" in dem Fenster eingibt. Damit man irgendwie drauf reagieren kann, also bspw szPathIn (unten) einfach auf "" setzt.

public String readPath(){ 
    	String szPathIn = "";
    	JFileChooser filechooser = new JFileChooser();	
	int hInput = filechooser.showOpenDialog(null);
	File file = null;
	if( hInput == JFileChooser.APPROVE_OPTION){
		file = filechooser.getSelectedFile();
		szPathIn = file.getAbsolutePath();
	}
    	return szPathIn;
}

Ich hoffe das ist so ueberhaupt sinnvoll implementiert....wenn nicht bitte ich um sachdienliche Hinweise

 

[thE_29]

Hab zz keine Zeit mir das genauer anzusehen, aber im Notfall, mach ne while Schleife und am Ende machst du ne Überprüfung ob der String der eingeben wurde auch existieren kann und falls net, die Schleife, sprich den FileChooser Dialog wiederholen!

 

[mic_checker]

Du meinst ne Überprüfung ob die angegebene Datei existiert?

Warum legst du nicht einfach ein File Objekt an, guckst mit exists() ob die Datei existiert und falls ja is gut, ansonsten halt net ...

Falls die Datei net existiert, kannst du unten deinen String ja auf "" setzen.

 

[Corcovado]

Hm, ich habs so versucht (Die fehlenden Deklarationen sind die gleichen wie in obigem Beispiel, nur halt klassenweit deklariert).
Das mit exist() hab ich versucht, es klappt bedingt, ich bekomme immer noch haufenweise Exceptions wenn ich beim Filechooser Fenster auf Cancel gehe, was kann ich da tun ?

public String readPath(){
    	filechooser = new JFileChooser();	
	hInput = filechooser.showOpenDialog(null);
	file = null;
	if( hInput == JFileChooser.APPROVE_OPTION){
		file = filechooser.getSelectedFile();
		szPathIn = file.getAbsolutePath();
	}
	
	if(((file.getAbsoluteFile()).length() != 0) && file.exists()){
		return szPathIn;
	}else{
		szPathIn = "";
		return szPathIn;
	}
}

PS:
Ich bekomme eine NullPointerException, ich glaube das is wichtig !?

 

[Corcovado]

Habs selber rausgefunden, man kann die if-else-Schachtelungen anders machen, um ein "Cancel", eine falsche Eingabe oder eine sinnvolle Eingabe abzufangen.

import java.io.File;

import javax.swing.JFileChooser;

public class File09 {
	private JFileChooser filechooser;
	private int hInput;
	private File file;
	private String szPathIn;
	
	public File09(){
		try{
			System.out.println("read path: #" + readPath() + "#");
		}catch(NullPointerException e){
			// reset vals
			szPathIn = "";
			e.printStackTrace();
		}
	}
	
	// FileChooser Window
	public String readPath() throws NullPointerException{
    	System.out.println("readPath()");//
    	filechooser = new JFileChooser();	
		hInput = filechooser.showOpenDialog(null);
		file = null;
		if( hInput == JFileChooser.APPROVE_OPTION){
			file = filechooser.getSelectedFile();
			szPathIn = file.getAbsolutePath();	
			if(file.exists()){
				System.out.println("readPath() file.exists() == true");//
			}else{
				System.out.println("readPath() file.exist() == false");//
				szPathIn = "";
			}
		}else{
			System.out.println("readPath() canceled");//
			szPathIn = "";
		}
		
		System.out.println("readPath() - end");//
		return szPathIn;
    }
	  
	/////////////////////////////////////////////////////////////////////
	public static void main(String[] args) {
		File09 f = new File09();
	}
}

Danke trotzdem fuer Eure Antworten !!!

 

[mic_checker]

readPath() könntest du auch kürzer machen:

private String szPathIn = "";
...   
public String readPath() {
       System.out.println("readPath()");//
       filechooser = new JFileChooser();   
      hInput = filechooser.showOpenDialog(null);
      file = null;
      if( hInput == JFileChooser.APPROVE_OPTION){
         file = filechooser.getSelectedFile();
         szPathIn = file.getAbsolutePath();   
         if(!file.exists())           
            szPathIn = "";                  
      }    

      return szPathIn;
    }

Sollte eigentlich genauso klappen..

Das "throws NullPointerException" musst du auch nicht mit angeben...

 

[Corcovado]

Ich hab gewusst, dass das jetz kommt

Habs auch schliesslich so angewendet. Aber zu meinem Verstaendnis is es halt oft fuer mich noch leichter die Dinge ausfuehrlichst in Einzelteile zu zerlegen und eher naiv zu implementieren (allein um mir meine Kommentare ausgeben zu lassen oder sie reinzuschreiben). Wenn ich dann mit so einer naiven Implementierung rumspiele entdecke ich oft erst wie man sie vereinfachen kann. Danke trotzdem.

 

[mic_checker]

Ich teste auch immer mit jede Menge System.out... wenn etwas nicht richtig funktioniert Die Überprüfung auf Cancel kannst du dir aber durch die Zuweisung von "" zu szPathIn oben sparen....Und wie gesagt muss man NullpointerException nicht explizit angeben bei throws.