Swing JDialog "flutet" den Arbeitsspeicher

[Cromm Cruach]

Hallo,

dieser Post bezieht sich ein Stück weit auf diesen Beitrag Auf beendingung des Formular warten

Das im Thema genannte Problem habe ich gelöst. Es werden nun alle Formulare nach einander angezeigt! Bisher habe ich diese Stapelverarbeitung im kleinen getestet habe gerade einen Test mit 145 Einträgen gestartet!

Hier nochmal der Code der die Fomulare anzeigt und am ender wieder zerstört!

public class OpenDialogSwingWorker extends SwingWorker<Void, Void>{

	FileAnalysis f = null;
	int i;
	
    public OpenDialogSwingWorker(FileAnalysis f, int i){
        this.f = f;
        this.i = i;
    }

    @Override
    public Void doInBackground() throws Exception {
		StringTokenizer t = new StringTokenizer(f.filenames.get(i), "[].", false);
		
		String quality = t.nextToken();
		String title = t.nextToken();
		title = title.substring(1);
		
		MovieDataForm mdf = new MovieDataForm(f, "Daten hinzufügen");
		MovieSearch ms = new MovieSearch(mdf);
		
		ms.searchPanel.searchField.setText(title);
		ms.searchPanel.searchButton.doClick();
		
		mdf.tabPane.fileInfoPanel.qualityCombo.setSelectedItem(quality);
		mdf.tabPane.fileInfoPanel.locationCombo.setSelectedItem(f.locationBox.getSelectedItem());
		mdf.tabPane.fileInfoPanel.sizeField.setText(f.size.get(i));
		mdf.tabPane.setSelectedComponent(mdf.tabPane.picturePanel);			
    	
    	return null;
    }
}

Am Ende der verarbeitung werden die Formulare mit

dispose();

"zerstört". Leider bleiben daten aus der Verarbeitung im Speicher und werden nicht vom GarbageCollectior vernichtet. Bei jeder Stapelverarbeitung wächst der Platz, der vom Programm im Arbeitsspeicher belegt wird um etwa 160.000 Kb, so dass nach wenigen einträgen 1,2 GB belegt sind.

in dem Formular werden unter anderem Bilder angezeigt. Dies geschieht in diesem Code Teil.

		pictureBox = new JRadioButton[cleanPictures.length];
		pictureLabel = new JLabel[cleanPictures.length];
		ImageIcon[] img = new ImageIcon[cleanPictures.length];
		
		for(int i = 0; i < cleanPictures.length; i++){
			
			img[i] = new ImageIcon(new URL(cleanPictures[i][1]));
			img[i].setImage(img[i].getImage().getScaledInstance(140, 210, Image.SCALE_DEFAULT));
			
			pictureLabel[i] = new JLabel(img[i]);
			pictureLabel[i].setSize(140, 210);
			
			
			pictureBox[i] = new JRadioButton();
			pictureBox[i].setSize(20, 20);
			pictureBg.add(pictureBox[i]);
			
			pictureListPanel.add(pictureLabel[i]);
			pictureListPanel.add(pictureBox[i]);
		}
		
		
	}

Die URLs stammen aus einem XML-Dokument das zu beginn im Formular eingelesen wird. Die Anzahl der Bilder die geladen werden sind von Eintrag zu Eintrag unterschiedlich (können schon mal 20 bilder sein). Und der wachsende Arbeitspeicher gängt im wesentlichen von der Bilderanzahl ab (soviel konnt ich herausfinden). Das Problem ist das die Bilder (sowie ich das sehe) nach beendigung des Formulars im Arbeitsspeicher bleiben.

Leider weis ich nicht warum. Müsste der GarbageCollector nicht merken das sie nicht mehr benötigt werden und den Platz wieder freigeben? Oder Übersehe ich hier etwas.

Danke im vorraus!

GRuß

 

[Camino]

Soweit ich weiss, entfernt der Garbage Collector nicht mehr benötigte Objekte nicht sofort, sondern wird irgendwann mal losgeschickt und schaut nach, was weg kann.

 

[Cromm Cruach]

Allerdings hat er bei mir auch nach 30 min noch nicht aufgeräumt

 

[Ebenius]

Was das Speicherproblem angeht: Nimm doch mal einen Profiler (vielleicht Oracles VisualVM) und schau in Deine Anwendung herein, während sie läuft. Dann siehst Du wo Deine Objekte hängen.

Oder päckele mal den Quelltext Deiner gesamten Anwendung in ein ZIP/JAR und mach es verfügbar. Aus den Schnipseln lässt sich nicht herausfinden, wo die Daten hängen bleiben.

Grundsätzliche Anmerkung: Dein SwingWorker baut Swing-Elemente in [c]doInBackground()[/c] auf. Genau das sollte er nicht machen. Eigentlich müsstest Du die langlaufenden Arbeiten (die wahrscheinlich aus dem Konstruktor [c]MovieDataForm(FileAnalysis, String)[/c] aufgerufen werden) in [c]doInBackground()[/c] machen und dann alles was die GUI verändert in der Methode [c]done()[/c] des SwingWorkers.

Ebenius

 

[bERt0r]

Das Problem stellt sich gar nicht, wenn du deiner MovieDataForm eine setData Funktion spendierst und dann einfach eine Instanz im Speicher hältst die du immer wieder mit neuen Daten bestückst anstatt jedes mal ein neues Objekt zu erzeugen.

 

[Cromm Cruach]

Laut dem Profiler wird der Speicher von String-Objekten gefüllt.

Aktuell sind alle meine Klassenvariablen public. Ich weis das das unprofessionell ist und bin auch dabei das zu ändern und mit gettern und settern zu arbeiten. Könnte das eine Ursache für das Speicherproblem sein?

Ansonsten nutzt meine Programm ne Menge DB-Abfragen. Wie sieht es das mit den ResultSets aus? Werden die iwann von selbst gelöscht oder muss man die iwie schließen oder so?!

 

[Cromm Cruach]

So ich habe meinen kompletten Code umgebaut! Alles so gemacht wie hier gefordert. Das Formular wird nun nicht mehrmals geladen sondern nur einmal und dann werde nur die Daten im Formular ersetzt.

Gebracht hat es nicht der Arbeitsspeicher wird immer noch geflutet und zwar sobald diese Funktion aufgerufen wird:

	public void setPicture() throws MalformedURLException{
		
		String[][] pictures = tmdbMl.getImage();
	
		window.tabPane.picturePanel.pictureListPanel.removeAll();
		window.tabPane.picturePanel.pictureListPanel.setLayout(new GridLayout(0,4,10,10));
		
		splashScreen.setDetailLabel("Importiere Bilddaten ...");
		
		Vector<String> url = new Vector<String>();
		for(int i = 0; i< pictures.length;i++){
			if(pictures[i][0]!= null){
				url.add(pictures[i][1]);
			}
		}
			
		window.tabPane.picturePanel.pictureBox = new JRadioButton[url.size()];
		window.tabPane.picturePanel.pictureLabel = new JLabel[url.size()];
		ImageIcon[] img = new ImageIcon[url.size()];
		
		splashScreen.setProgressBarMaximum(url.size());
		splashScreen.setProgressBar(0);
		
		for(int i = 0; i < url.size(); i++){
			
			img[i] = new ImageIcon(new URL(url.get(i)));
			img[i].setImage(img[i].getImage().getScaledInstance(140, 210, Image.SCALE_DEFAULT));
			
			window.tabPane.picturePanel.pictureLabel[i] = new JLabel(img[i]);
			window.tabPane.picturePanel.pictureLabel[i].setSize(140, 210);
			
			
			window.tabPane.picturePanel.pictureBox[i] = new JRadioButton();
			window.tabPane.picturePanel.pictureBox[i].setSize(20, 20);
			window.tabPane.picturePanel.pictureBg.add(window.tabPane.picturePanel.pictureBox[i]);
			
			window.tabPane.picturePanel.pictureListPanel.add(window.tabPane.picturePanel.pictureLabel[i]);
			window.tabPane.picturePanel.pictureListPanel.add(window.tabPane.picturePanel.pictureBox[i]);
			
			splashScreen.setProgressBar(splashScreen.getProgressBar()+1);
		}
		
    	window.tabPane.picturePanel.pictureScrollPane.getViewport().setView(window.tabPane.picturePanel.pictureListPanel);

Ich bin mir nun zu 100% sicher, das es die Bilder sind. genau dann wenn sie geladen werden steigt der Speicherbedarf. und nach dem Schließen des Formulars werden sie nicht mehr aus Arbeitsspeicher genommen. Ich hab einfach keine ahnung wo der Fehler liegen soll.

 

[bERt0r]

Warum hast du ein String[][] mit dem namen pictures? Du speicherst doch nicht die Pixeldaten als Strings, oder?

 

[Cromm Cruach]

Nein das sind informationen zum Bild:

picture[0] = Id von Bild i
picture[1] = URL zu Bild i

 

[Ebenius]

Bzgl. DB-Abfragen: Danach einfach immer in dieser Reihenfolge aufräumen:

1. ResultSet.close()

2. Statement.close()

3. [c]Connection.close()[/c] (sofern die selbe Connection nicht nachgenutzt wird)

Je nach Treiber-Implementation ist 1. zwingend erforderlich oder wird implizit bei 2. mit gemacht. Besser man macht's immer selbst.

Ebenius