BufferWriter als methode

[Java_Fan]

Hallo freunde;

ich bin mit BufferWriter nicht so flüssig. Ich will den openfile() methode mehrmals aufrufen. wenn ich BufferWriter auf null setze bekomme exception und wenn nicht bekomme ich fehler !! was mache ich falsch??

liebe Gruße
G

private static void outputChar(int i, int last, char c) throws IOException {
		BufferedWriter bw = null;		
		
		for (int j = last; j < i; j++)
		{
		openfile(bw);			
		bw.write(c); -----> hier bricht das program
		}
		bw.close();
	}

	public static Object openfile(BufferedWriter bw) throws IOException {	
		File f1 = new File("c:/test1.txt");
		FileWriter fw = new FileWriter(f1);
		bw = new BufferedWriter(fw);
		
        return bw;
	}

Exception ist :

Exception in thread "AWT-EventQueue-0" java.lang.NullPointerException
at suchAlgoritmus.outputChar(suchAlgoritmus.java:106)
at suchAlgoritmus.underline(suchAlgoritmus.java:94)
at suchAlgoritmus.auslesen(suchAlgoritmus.java:69)
at suchAlgoritmus.setFileName(suchAlgoritmus.java:25)
at Start$1.actionPerformed(Start.java:33)
at javax.swing.AbstractButton.fireActionPerformed(AbstractButton.java:1849)
at javax.swing.AbstractButton$Handler.actionPerformed(AbstractButton.java:2169)
at javax.swing.DefaultButtonModel.fireActionPerformed(DefaultButtonModel.java:420)
at javax.swing.DefaultButtonModel.setPressed(DefaultButtonModel.java:258)
at javax.swing.plaf.basic.BasicButtonListener.mouseReleased(BasicButtonListener.java:236)
at java.awt.Component.processMouseEvent(Component.java:5517)
at javax.swing.JComponent.processMouseEvent(JComponent.java:3135)
at java.awt.Component.processEvent(Component.java:5282)
at java.awt.Container.processEvent(Container.java:1966)
at java.awt.Component.dispatchEventImpl(Component.java:3984)
at java.awt.Container.dispatchEventImpl(Container.java:2024)
at java.awt.Component.dispatchEvent(Component.java:3819)
at java.awt.LightweightDispatcher.retargetMouseEvent(Container.java:4212)
at java.awt.LightweightDispatcher.processMouseEvent(Container.java:3892)
at java.awt.LightweightDispatcher.dispatchEvent(Container.java:3822)
at java.awt.Container.dispatchEventImpl(Container.java:2010)
at java.awt.Window.dispatchEventImpl(Window.java:1791)
at java.awt.Component.dispatchEvent(Component.java:3819)
at java.awt.EventQueue.dispatchEvent(EventQueue.java:463)
at java.awt.EventDispatchThread.pumpOneEventForHierarchy(EventDispatchThread.java:242)
at java.awt.EventDispatchThread.pumpEventsForHierarchy(EventDispatchThread.java:163)
at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:157)
at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:149)
at java.awt.EventDispatchThread.run(EventDispatchThread.java:110)

 

[Ebenius]

Deine Probleme haben nichts mit dem BufferedWriter zu tun.

In Deinem Code setzt du in Zeile 3 bw auf null und dann verwendest Du die null-Referenz in Zeile 8. Klar, dass dabei eine NullPointerException kommt.

 

[Java_Fan]

Das Stimmt. Aber wenn ich diese Zeile lösche bekome ich fehler !!!!

 

[Ebenius]

Weniger Ausrufezeichen, mehr Aussage bitte.

 

[SlaterB]

> openfile(bw);

->

bw = openfile(bw);

bzw. openfile() besser ganz ohne Parameter

 

[Ebenius]

bzw. openfile() besser ganz ohne Parameter

Slater, das geht schief.

 

[SlaterB]

inwiefern?

 

[Java_Fan]

ich habe es jetzt so verändert:

private static void outputChar(int i, int last, char c) throws IOException {
		BufferedWriter bw = null;		
		
		for (int j = last; j < i; j++)
		{
		bw = (BufferedWriter)openfile(bw);			
		bw.write(c);		
		System.out.print(c);
		}
		//bw.close();
	}

und so funktioniert. Aber ich muss den file auch zu machen können oder? wo muss den bw.close() hin? [/quote]

 

[SlaterB]

> bw = (BufferedWriter)openfile(bw);

dieser Aufruf könnte doppelt einfacher

bw = openfile();
lauten, wenn du nur den Parameter weglassen und den richtigen Rückgabetyp setzen würdest

-----

zum close() kann niemand was sinnvolles sagen,
setze dann close(), wann du es für richtig hälst,

allgemein ist es sehr fragwürdig, für nur ein oder wenige Zeichen gleich eine Datei zu öffnen,

 

[Java_Fan]

danke euch alle! mit bw.close() wie du gesagt hast muss ich weiter rätzeln

 

[Verjigorm]

close() möglichst in den finally Block setzen

hierzu am besten try/catch drumrum und die Exception durchreichen, ich denke du brauchst die für irgendwas, wenn du schon throws schreibst

 

[0x7F800000]

wozu ist überhaupt diese merkwürdige methode da? und wo wir schon dabei sind: warum sind da 2 unnötige temporäre variablen drin?

      bw = new BufferedWriter(new FileWriter(new File("c:/test1.txt")));

warum schreibst du nicht direkt hin was du machst? funktionalitätslose dummy-methoden machen den code weder übersichtlicher, noch schneller...

 

[SlaterB]

wenn openfile() in 20 Methoden benutzt wird, dann macht die extra Methode schon Sinn,
wobei dann aber die 20 Methoden für sich wohl bereits das größere Problem sind

 

[0x7F800000]

naja... einen einzeiligen aufruf eines bekannten Konstruktors durch einen einzeiligen aufruf einer unbekannten eigenbau-methode zu ersetzen ist merkwürdig. Und außerdem glaube ich dem nicht, dass der OP in 20 völlig unterschiedlichen Situationen dieselbe testdatei aufmachen muss ???:L