JPanel mit Grafik von HDD über JTree

[pixel-shock]

Hi zusammen,

habe jetzt die Forensuche komplett durch, aber nichts so richtig gefunden, was mir hilft.

Folgender Aufbau:

- JFrame mit BorderLayout
- LEFT ein JTree (funzt einwandfrei)
- CENTER ein JPanel (heißt bei mir contentPanel) in einer JScrollPane

Folgende Funktion:

- Klick auf JTree durchsucht angeklickten Ordner nach jpg, gif, png
- Alle gefundenen Grafiken sollen in weitere JPanels gepackt werden, welche dann wiederum in das contentPanel gepackt werden

Folgendes Problem:


- java.lang.OutOfMemoryError: Java heap space
- Ich weiß noch nicht so richtig, wie ich es am besten angehe, also das Image als ImageIcon in ein JLabel legen oder als Grafik direkt zeichnen lassen? Unter das Bild sollen noch diverse kleine Buttons und eine ComboBox.

package source;

import java.awt.BorderLayout;
import java.awt.Image;
import java.awt.MediaTracker;
import java.io.File;
import java.io.IOException;

import javax.imageio.ImageIO;
import javax.swing.ImageIcon;
import javax.swing.JLabel;
import javax.swing.JPanel;

public class PhotoContent extends JPanel {
	
	private Image img;
	private JLabel imageLabel;
	private int imgWidthOrg;
	private int imgHeightOrg;
	private int imgWidthNew;
	private int imgHeightNew;
	private int difference;
	
	
	public PhotoContent(String filePath) {
		this.setLayout(new BorderLayout());		
		this.imageLabel = new JLabel("", JLabel.CENTER);
		this.add(this.imageLabel, BorderLayout.CENTER);
		this.showImage(filePath);
	}
	
	private void showImage(String filePath) {
		try {
			this.img = new ImageIcon(ImageIO.read(new File(filePath))).getImage();
			MediaTracker mt = new MediaTracker(this);
			mt.removeImage(img);
			mt.addImage(img, 0);
			try {
				mt.waitForAll();
		        this.imgWidthOrg = this.img.getWidth(this);
		        this.imgHeightOrg = this.img.getHeight(this);

		        if (this.imgWidthOrg > 150) {
		        	this.imgWidthNew = 150;
	        		this.difference = (this.imgWidthNew * 100) / this.imgWidthOrg;
	        		this.imgHeightNew = (this.imgHeightOrg * this.difference)/100;	        		
	        		if (this.imgHeightNew > 150) {
	        			this.imgHeightNew = 150;
	        			this.difference = (this.imgHeightNew * 100) / this.imgHeightOrg;
	        			this.imgWidthNew = (this.imgWidthOrg * this.difference) / 100;
	        		}	        		
		        }
		        this.img = this.img.getScaledInstance(this.imgWidthNew, this.imgHeightNew, Image.SCALE_FAST);
				this.imageLabel.setIcon(new ImageIcon(img));
			} catch (InterruptedException e) {
				e.printStackTrace();
			}
		} catch (IOException e1) {
			e1.printStackTrace();
		}
	}

}

Zudem müsste ich das Laden und Resizen glaube ich in einem Thread laufen lassen ... aber dann spuckt Java mir noch schneller die heap Exception aus.

Würde mich sehr über Hinweise und Tips freuen.

LG
Tino

 

[pixel-shock]

Hmm ... ich habe jetzt noch das Netz durchsucht, aber irgendwie haben alle das selbe Problem nur ohne wirkliche Lösung.
Ich kann doch aber nicht von den ganzen Bildern, die gefunden werden erst Thumbnails erstellen lassen und dann diese anzeigen lassen und beim Verzeichniswechsel dann wieder löschen?

LG
Tino

 

[r.w.]

Hallo Tino,

vorab muss ich sagen, dass ich mich selbst noch als Java-Einsteiger betrachte
und vor einem ganz ähnlichen Problem stehe. Vielleicht wird hier ja uns beiden
geholfen.

Bei Deinem Code fallen mir spontan 2 Dinge auf:

1. Wenn ich mich nicht irre, ist bei Verwendung von ImageIO.read() der Mediatracker nicht nötig.
2. Speicherst Du das geladene Bild in Deiner Klasse doppelt ab.
Einmal in Originalgrösse in img und einmal skaliert in imageLabel.

Damit ist Dein Problem sicher nicht komplett gelöst, aber wahrscheinlich läuft der Speicher
so erst etwas später über. ;-)

VG ROlf

 

[pixel-shock]

Hi,

das werde ich mal probieren, DANKE für den Tip

LG
Tino

 

[pixel-shock]

Hmm ... hilft leider auch nicht ... sobald die Fotos im Ordner etwas größer sind schießt er sich gleich ab.

LG
Tino

 

[Verjigorm]

was ist etwas "größer" ?

import java.awt.BorderLayout;
import java.awt.Image;
import java.io.File;
import java.io.IOException;
import javax.imageio.ImageIO;
import javax.swing.ImageIcon;
import javax.swing.JLabel;
import javax.swing.JPanel;

public class PhotoContent extends JPanel {
   
   private Image img;
   private JLabel imageLabel;
   private int imgWidthOrg;
   private int imgHeightOrg;
   private int imgWidthNew;
   private int imgHeightNew;
   private int difference;
   
   
   public PhotoContent(String filePath) {
      this.setLayout(new BorderLayout());      
      this.imageLabel = new JLabel("", JLabel.CENTER);
      this.add(this.imageLabel, BorderLayout.CENTER);
      this.showImage(filePath);
   }
   
   private void showImage(String filePath) {
         try {
			this.img = ImageIO.read(new File(filePath));
		} catch (IOException e) {
			e.printStackTrace();
		}

              this.imgWidthOrg = this.img.getWidth(this);
              this.imgHeightOrg = this.img.getHeight(this);

              if (this.imgWidthOrg > 150) 
              {
                 this.imgWidthNew = 150;
                 this.difference = (this.imgWidthNew * 100) / this.imgWidthOrg;
                 this.imgHeightNew = (this.imgHeightOrg * this.difference)/100;                 
                 if (this.imgHeightNew > 150) 
                 {
                    this.imgHeightNew = 150;
                    this.difference = (this.imgHeightNew * 100) / this.imgHeightOrg;
                    this.imgWidthNew = (this.imgWidthOrg * this.difference) / 100;
                 }                 
              }
              
              this.img = this.img.getScaledInstance(this.imgWidthNew, this.imgHeightNew, Image.SCALE_FAST);
            this.imageLabel.setIcon(new ImageIcon(img));
      }     
}

klappt bei mir mit 15MB Bilder ohne Probleme, größere Bilder hab ich nicht
Verbraucht zwischen 12 und 18MB Heapspace

 

[pixel-shock]

Größer heißt, dass das Problem kommt, sobald alle Bilder im Ordner eine Auflösung über 1000px haben.
Ich habe es grade mit einem Ordner mit insgesamt 16MB Fotos getestet und der Fehler kommt immernoch.

Woran kann das dann liegen, wenn es bei dir ohne Probleme funzt?

LG
Tino

 

[Verjigorm]

Der Ordner is ja erstmal egal, geht ja um das einzelne Bild.
Meine Bilder hier haben ne AUflösung von 2560x1024

Haste den neuen Code von mir benutzt?

Der Stacktrace zur HeapSpace-Exception würde auch helfen.
Wieviele solcher Bilder lädst du gleichzeitig? etc...

 

[pixel-shock]

Also die einzelnen Bilder sind unter 1MB.

Jub hab den Code benutzt.

Die Objekte "PhotoContent" werden so erzeugt.

			File[] files = new File(path).listFiles();
			if (files != null) {
			  for (int i = 0; i < files.length; i++) {
				  if (files[i].isDirectory()) {
				     if (files[i].canRead() && !files[i].isHidden() && !isEmpty(files[i])) {	
						TreeNode subDir = new TreeNode(files[i], false);
						selectedNode.add(subDir);
				     } 
				  } else {					 
					  for (int j=0;j<this.allowedFileTypes.length;j++) {
						  if (files[i].getName().endsWith(this.allowedFileTypes[j])) {						
							PhotoContent pcontent = new PhotoContent(files[i].getAbsolutePath());
							this.fileViewPanel.add(pcontent);
						  }
					  }
				  }
			  }
			}

LG
Tino

 

[Verjigorm]

oO wie umständlich ...

mach dir nen ordentlichen FileFilter für dein listFiles
Vermutlich ist dein Ordner riesig groß und es kracht bei listFiles() ...

 

[pixel-shock]

Ja mit dem FileFilter hast du Recht, mein Ordner ist allerdings grade mal 25 Bilder groß.

Ich frage mich aber grade, ob es was ändert, wenn ich immer nur ein Foto lade und wenn das fertig ist erst das nächste starte.

LG
Tino