Klick auf Button -> JFrame öffnet sich erneut. & Datei lässt sich nicht löschen

[jonas.r]

Hallo nochmal. Hab hier 2 kleine Probleme.

Problem Nr.1:
Ich habe ein Programm geschrieben, mit dem man seine zu erwartenden Stromkosten ausrechnen kann.
Auf dem Haupt-JFrame ist eine Liste mit Geräten, welche man hinzufügen, bearbeiten und löschen kann.
Wenn ich von diesem JFrame auf den Button "Neues Gerät hinzufügen" klicke, öffnet sich ein weiteres JFrame, in dem ich Name, Verbrauch und Laufzeit eines Gerätes in Textfelder eintragen kann. Beim klick auf einen "gerät speichern" button, erzeugt das Programm eine Textdatei und schreibt den Inhalt der Textfelder in sie rein.
Das alles funktioniert soweit bestens!

Der ActionListener, der zum "Gerät speichern" button gehört, führt neben der Erstellung der Datei, auch eine Methode "aktualisiereListe()" aus. Diese Methode befindet sich aber in der KLasse, in der sich auch das Hauptfenster befindet.
Ich erzeuge also eine neue Instanz der Hauptfenster-klasse und lass dann die Methode ausführen:

//Datei: NeuesGerätFenster.class

Hauptfenster hf = new Hauptfenster();
hf.aktualisiereListe();

Die Methode sieht wie folgt aus:

//Datei: Hauptfenster.class

public void aktualisiereListe(){
			leseVerzeichnisAus();	
			geräteListeJList.setListData(fileListOhneEndung);		
		}

Die Methode "leseVerzeichnisAus()" so:

//Datei: Hauptfenster.class

public void leseVerzeichnisAus(){
			//Verzeichnis auslesen:			
			 fileList = dir.list(new FilenameFilter() {					    
				public boolean accept(File d, String name) {
				return name.endsWith(".txt");
  				}
			 });
			 
			endungWeg();
		}

Nun aber öffnet sich das Haupt-JFrame (mit aktualisierter liste) nochmal! Nun hab ich also das alte Fenster und das neue!

Woran liegt das????:L


Problem Nr.2:
Außerdem lässt sich das nun erstellte Gerät nicht mehr löschen. Erst, wenn ich das Programm neu geöffnet habe. Ich habe auch versucht die Datei Manuell zu löschen, aber dann wird mir gesagt, dass das Programm sie gerade noch verwendet. Warum?

Hier die Methode zum Schreiben in eine Datei:

//Datei: DateiHandler.class

public void schreibeInDatei(String name, String verbrauch, String laufzeit, File f){
		
		FileWriter writer;
	    try {
			// new FileWriter(file ,true) - falls die Datei bereits existiert
	        // werden die Bytes an das Ende der Datei geschrieben	        
	        // new FileWriter(file) - falls die Datei bereits existiert
	        // wird diese überschrieben
	        writer = new FileWriter(f);
	        
	        // Text wird in den Strom geschrieben
	        writer.write(name.replaceAll(" ","_"));
	        writer.write(System.getProperty("line.separator"));    
	        writer.write(verbrauch);
	        writer.write(System.getProperty("line.separator"));
	        writer.write(laufzeit);
	        
	        //Der Strom wird übertragen und geschlossen
	        writer.flush();
	        writer.close();
	     }
	    catch (IOException e) {
	        e.printStackTrace();
	     }	    
	}

Ich würde mich sehr freuen, wenn ihr mir sagen könntet, was ich falsch gemacht habe=)

(Im Anhang hab ich das Programm als .jar File angehängt)

 

[Kevin94]

Ich hab leider für keines der Probleme eine Antwort, allerdings eine Anmerkungen.
Ganz am Anfang sei gesagt, das es sehr schwer dir zu helfen, wenn man nur diese Bruchstücke an Code vorgesetzt bekommt und in der .jar kein Source drin ist und man sich durch unkommentierten dekompilierten Code wühlen muss. Nebenbei: Ich bin zwar kein Verfechter der Umlaute-sind-böse Meinung, aber in Klassennahmen haben sie nichts zu suchen. Das kann dann doch mal böse enden.

Ausserdem konnte ich das erste Problem nicht nachstellen und den geposteten Code dazu nirgends in dem dekompilierten Code entdecken. Bei mir ergab sich nur das Problem, das die GUI nicht aktualisiert wurde.
Das zweite Problem hatte ich auch und den Grund dafür kann ich dir auch sagen, vermutlich ein nicht geschlossenen Stream, konnte ihn aber nirgends finden, liegt vermutlich an der Unübersichtlichkeit und dem Unwillen mehr als 10 min in die Suche zu stecken.

Dann ein paar allgemeine Anmerkungen:
Null-Layout, wirchlich? Ich hoffe du kannst einen GUI-Builder zu deiner Entschuldigung vorbringen. Wer tut sich denn sowas freiwillig an, es gibt doch genügend LayoutManager, die das einfacher und mit übersichtlicherem Code erledigen.
Das zweite ist

extends JFrame

. Dazu kann ich nur auf diesen Blogeintrag verlinken.
Dann die Art der Speicherung: Jedes Gerät als einzlene Datei zu speichern ist ja schon unelegant, ab das ganze noch hat codiert in einem Unterverzeichnis von C: wäre für mich als User definitiv ein Grund das Programm schnell wieder zu löschen. Du kannst genausogut alle Geräte in einer Datei speichern, erfordert beim Parsen etwas mehr Aufwand, ist aber auch schnell erledigt. Als alternative Speicherorte bietet sich

user.home

an.
Und zu letzt noch die Hauptklasse: Eine Klasse, in der nur die Main-Methode steht, in welcher nur eine einzige Anweisung steht, kann man sich auch sparen. Die Main-Methode hättest du genausogut auch in Hauptfenster stecken können.

 

[Lissy]

Hallo,
also ich bin mir nicht sicher, aber überprüfe mal ob du bei deinem JFrame die Methode

 f //Name deines JFrames*//.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);

hast. Anders hat das rote Kreuz oben bei deinem Fenster keine Funktion (lässt sich also nicht schließen). Das ist bei Java so, wieso weiß ich auch nicht.

 

[bERt0r]

Hauptfenster hf = new Hauptfenster();

Da ich wahnsinnig gut englisch, übersetzt ich mal für dich: new bedeutet neu. Wenn du dem Computer sagst neues Hauptfenster bitte, dann tut der das auch. Wenn du auf dein Hauptfenster zugreifen willst, musst du deinem zweiten Frame eine Referenz darauf übergeben, z.B mit

class EingabeFenster extends JFrame
{
       public EingabeFenster(Hauptfenster mamaFenster)
{
}
}
EingabeFenster fenster=new EingabeFenster(this);

 

[jonas.r]

[OT]Erstmal vorweg:
Es ist wirklich nett von dir, dass du dir die Mühe gemacht hast das Programm durchzugucken und sogar die .jar datei entpackt hast.
Wie du schreibst wirkt allerdings ziemlich genervt, weswegen ich etwas verwirrt bin. Ist doch immerhin ein Anfänger-Unterforum und da sind so "einfache" Fehler doch normal oder nicht?

Also ich habe versucht die Codeschnipsel aufs Wesentliche zu reduzieren, wenn das nicht reicht, hab ich überhaupt nichts dagegen, hier noch mehr davon, oder auch alles, mit in den Post einzubringen.

Mit dem Stream weiß ich nicht genau was du meinst. Meinst du den Datenstrom, der in die neu erstellte Datei fließt? Der wird doch mit

writer.close()

in der Methode

schreibeInDatei()

geschlossen, oder irre ich mich?
Und warum unterstellst du mir zu faul zu sein? Natürlich habe ich vorher lange Zeit nach einer Problemlöung gesucht, bevor ich jemand anderen um Hilfe frage!

Ich hab vorher schon viel mit Layout-Managern rumprobiert, aber das Ergebnis war für mich auf Grund von meiner unverständlichkeit nicht zufrieden stellend. Deswegen habe ich kein Layout Manager verwendet, sondern selber abhängigleiten beschrieben.

Das mit dem extends JFrame kannt ich bis jetzt nur so. (Wenn du die Antwort auf den Blogeintrag auch gelesen hast, solltest du wissen, dass es offt so gelehrt wird. So auch bei mir in der Schule.)

Alle weiteren Probleme, die dich ja anscheinend persönlich stören, sind auch nur auf Grund meines Unwissens als Anfänger vorhanden.

Dass es einen Befehl wie

user.home

gibt wusste ich bis jetzt nicht und habe deswegen das, was am meisten auf jeden Fall vorhanden ist (nämlich einfach C://) verwendet. Aber danke für den Tip, das fand ich auch nicht so Schick.
[/OT]

Zum Thema direkt:
Den Fehler des ersten Problems hab ich schon entdeckt. Beheben konnte ich ihn aber noch nicht.
In dem ActionListener öffne ich eine neue Instanz des Hauptfensters. Also macht er natürlich auch einen neuen Frame auf.

Das 2. Problem bleibt immernoch ungelöst, ich würde mich freuen, wenn mir jemand da noch helfen könnte.

Liebe Grüße

Edit: Während ich geschrieben habe, haben hier noch 2 weitere geantwortet, aber das hat sich mit meinem Post jetzt glaub ich auch erledigt

 

[Kevin94]

Mal vorneweg: Es war definitiv nicht meine Absich, schroff oder genervt zu klingen oder dir zu unterstellen das du faul bist.

Codeschnipsel: Die Menge an Code im Einganspost war eigentlich richtig, ich hätte es halt schön gefunden, wenn der richtige Quellcode auch in der .jar gewesen wäre.

Stream: Das ist der Englische Überbegriff für für Datenströme aller Art. In deinem Fall wären das FileWriter und der Scanner zum lesen (Denn muss man auch schliesen). Wenn ich mich nicht irre schliest du den nicht immer. Am einfachsten hat man die Kontrolle darüber, wenn man mit dem try-with-Resources Konstrukt das dem Compiler überlässt.

Das die angesprochenen Problem allein auf Unwissen und nicht auf Absicht basieren, habe ich nie bestritten. Genau dafür ist das Forum da: Solche Fehler anzusprechen.

Und

user.home

ist kein Befehl sondern eine System-Variable, die man über

System.getProperty("user.home")

abfragen kann.

 

[jonas.r]

Leider schließe ich den Scanner immmer, sofort nach verwendung ( mit scan.close()). Wär schön gewesen, wenn es das wäre. Ich benutze ihn zwei mal. Einmal beim auslesen aus den Dateien und einmal zum verändern der Daten. Der FileWriter wird auch geschlossen ( mit writer.close(), direkt nachdem der Strom gecshrieben wurde)

@Kevin94: Nebenbei: Das mit dem extends JFrame und der Main-Methode hab ich geändert

 

[jonas.r]

Kann niemand helfen?

Also das Problem nochmal in der Zusammenfassung:

-Ich erstelle eine Datei und schreibe einen Inhalt rein. Nach dem Schreiben mit einem FileWriter wird der writer sofort wieder mit dem Befehl

fileWriter.close();

geschlossen.
-Nun will ich diese Datei löschen, was aber nicht funktioniert. Erst nach ca. 40sek geht es dann.
-Auch das manuelle löschen aus dem Ordner wird solange von Windows blockiert.
-Andere Dateien kann ich mit dem Programm löschen, wenn sie bei dem letzten Ausführen des Programmes erstellt, oder ihre erstellung ca 40 sek her ist.

Die Methode zum Erstell der Datei geht so:

//Datei: DateiHandler.class

public void macheDatei(File f){

	      
	    if (!f.exists()){
			try {
				new Formatter(f);   					
			} 
			catch (FileNotFoundException e) {
				e.printStackTrace();
			}
	    }
	}

Die Methode zum schreiben geht so:

//Datei: DateiHandler.class

public void schreibeInDatei(String name, String verbrauch, String laufzeit, File f){
		
		FileWriter writer;
	    try {
			// new FileWriter(file ,true) - falls die Datei bereits existiert
	        // werden die Bytes an das Ende der Datei geschrieben	        
	        // new FileWriter(file) - falls die Datei bereits existiert
	        // wird diese überschrieben
	        writer = new FileWriter(f);
	        
	        // Text wird in den Strom geschrieben
	        writer.write(name.replaceAll(" ","_"));
	        writer.write(System.getProperty("line.separator"));    
	        writer.write(verbrauch);
	        writer.write(System.getProperty("line.separator"));
	        writer.write(laufzeit);
	        
	        //Der Strom wird übertragen und geschlossen
	        writer.flush();
	        writer.close();
	     }
	    catch (IOException e) {
	        e.printStackTrace();
	     }	    
	}

Die Methode zum löschen funktioniert an sich ja, deswegen lass ich die raus.

Die Methode in dem ActionListener zum speichern der Datei sieht so aus:

//Datei: NeuesGerätFenster.class

class ErzeugeGerätListener implements ActionListener{
	
		public void actionPerformed(ActionEvent e) {
			String eingabeName= eingabeNameJtext.getText();
			File f = new File (dir+"//" + eingabeName +".txt");
			DateiHandler dh= new DateiHandler();
			dh.macheDatei(f);
			dh.schreibeInDatei(eingabeNameJtext.getText(), eingabeVerbrauchJText.getText(), eingabeLaufzeitJtext.getText(), f);
			
			Hauptfenster hauptFenster = Hauptfenster.hauptFenster;
			hauptFenster.aktualisiereListe();
			frame.dispose();
		}
	}

Weiß jemand was ich falsch mache?