JFileChooser & FileReader

[Dark Void]

Guten Tag,
ich bräuchte Hilfe bei einem Code, der per FileChooser eine Datei auswählt, alle Zeilen darauf ausliest und in einen String packt.

				 				 try
				 {
					 String string = "";
					 FileReader reader;

					 JFileChooser fc = new JFileChooser();
					 fc.setFileSelectionMode(JFileChooser.FILES_AND_DIRECTORIES);
					 int returnVal = fc.showOpenDialog(fc);
					 
					 if (returnVal == JFileChooser.APPROVE_OPTION)
					 {
						 reader = new FileReader(fc.getSelectedFile());
					 }
					 BufferedReader br = new BufferedReader(new FileReader(fc.getSelectedFile()));
					 
					 String ausgelesen;
					while ((ausgelesen = br.readLine()) != null)
					 {
						 string = string + ausgelesen;
					 }
					System.out.println(string);
					 br.close();
				}
				 
				 catch (IOException o)
				{o.printStackTrace();}
			}

Der String string sollte also nach der while Schleife den selben Text enthalten, wie die ausgewählte Text Datei. Es wird aber zuvor "null" ausgegeben.

EDIT:
hat sich geklärt, ich musste nur per leerem String initialisieren.

 

[FrozenBrain]

Dein Quelltext macht so wie er ist an einigen Stellen kaum Sinn.

Zum einen solltest du, da du ja nur Dateien und keine Ordner auslesen willst, als FileSelectionMode FILES_ONLY anstelle von FILES_AND_DIRECTORIES verwenden.

Dann öffnest du einen FileReader wenn eine Datei ausgewählt wurde, machst aber trotzdem weiter egal ob eine ausgewählt wurde oder nicht. Somit solltest du deinen gesamten Code nach der Bedingung zwischen die geschweiften Klammern packen.

Einen zweiten FileReader zu initialisieren ist überflüssig, nimm dafür besser direkt den "reader" den du schon vorher erzeugt hast.

Deine While-Schleife dürfte soweit passen, achte aber darauf, dass readLine() die Zeile ohne Zeilenumbruch am Ende zurückgibt.

Wenn ich weiter in's Detail gehen soll, sag bescheid

 

[Dark Void]

So sollte es jetzt passen:

				 try
				 {
					 String string = "";

					 JFileChooser fc = new JFileChooser();
					 fc.setFileSelectionMode(JFileChooser.FILES_ONLY);
					 fc.showOpenDialog(null);

					 BufferedReader br = new BufferedReader(new FileReader(fc.getSelectedFile()));
					 
					 String ausgelesen;
					while ((ausgelesen = br.readLine()) != null)
					 {
						 string = string + ausgelesen;
					 }	
					System.out.println(string);
				}
				 
				 catch (IOException o)
				{o.printStackTrace();}

 

[FrozenBrain]

Du solltest eventuell schon prüfen, ob der Nutzer auch wirklich eine Datei ausgewählt hat oder nicht. Dazu hatte deine If-Anweisug aus deinem ersten Post schon gestimmt, nur hättest du mehr von deinem Code zwischen die geschweiften Klammern packen müssen. Ansonsten sollte dein Code aber so stimmen, ja

 

[Mujahiddin]

Dein BufferedReader wird nirgends geschlossen.

Wenn du Java7 verwendest, benutze die neuen Methoden. Da kannst du auch Encoding mit angeben und platformunabhängig Sonderzeichen in die Dateien schreiben:

JFileChooser fc = new JFileChooser();
fc.setFileSelectionMode( JFileChooser.FILES_ONLY );
fc.showOpenDialog( null );

File f = fc.getSelectedFile();
if( f != null ) {
	
	String string = "";
	try {
		string = new String( Files.readAllBytes( f.toPath() ) /* , Charset.forName("MyEncoding") */);
	} catch( IOException e ) {
		e.printStackTrace();
	}
	System.out.println( string );
}