Java heap space

[erZ]

Tag auch

Also ich habe einen jFileChooser, wo ich enorme Mengen von Dateien auswählen will. Wenn ich da über einen bestimmten Wert geben, kommt schon mal ne Exception Aber das ist noch nicht mal das eigentliche Problem.

Ich lese Dateien so ein:

    private void jFileChooser1ActionPerformed(java.awt.event.ActionEvent evt) {                                              

        if(evt.getActionCommand().equals("ApproveSelection")) { //Öffnen
            Main.choosedFiles = jFileChooser1.getSelectedFiles(); //das ist egal
            try {
                //Abbilden der Files auf den Vector fileImages in hf.fileImages
                for(int i=0;i < Main.choosedFiles.length;i++) {
                    Main.hf.fileImages.add(ImageIO.read(Main.choosedFiles[i])); //<-- hier liegt das Problem!
                }
                //rest-code

                
            } catch (java.io.IOException e) {
                //tritt eh nicht ein :D
            }
            this.dispose();
        }
        
        if(evt.getActionCommand().equals("CancelSelection")) { //Abbrechen
            this.dispose();
        }
    }

Und wenn ich da große Dateien oder viele, je nach dem angebe und dann auf öffnen klicke, kommt:

• 
  Exception in thread "AWT-EventQueue-0" java.lang.OutOfMemoryError: Java heap space
  at java.awt.image.DataBufferByte.<init>(DataBufferByte.java:58)
  at java.awt.image.ComponentSampleModel.createDataBuffer(ComponentSampleModel.java:397)
  at java.awt.image.Raster.createWritableRaster(Raster.java:938)
  at javax.imageio.ImageTypeSpecifier.createBufferedImage(ImageTypeSpecifier.java:1169)
  at javax.imageio.ImageReader.getDestination(ImageReader.java:2879)
  at com.sun.imageio.plugins.jpeg.JPEGImageReader.readInternal(JPEGImageReader.java:943)
  at com.sun.imageio.plugins.jpeg.JPEGImageReader.read(JPEGImageReader.java:915)
  at javax.imageio.ImageIO.read(ImageIO.java:1422)
  at javax.imageio.ImageIO.read(ImageIO.java:1282)
  at twainprojekt.Durchsuchen.jFileChooser1ActionPerformed(Durchsuchen.java:47)
  at twainprojekt.Durchsuchen.access$000(Durchsuchen.java:7)
  at twainprojekt.Durchsuchen$1.actionPerformed(Durchsuchen.java:22)
  at javax.swing.JFileChooser.fireActionPerformed(JFileChooser.java:1717)
  at javax.swing.JFileChooser.approveSelection(JFileChooser.java:1627)
  at javax.swing.plaf.basic.BasicFileChooserUI$ApproveSelectionAction.actionPerformed(BasicFileChooserUI.java:914)
  at javax.swing.AbstractButton.fireActionPerformed(AbstractButton.java:1995)
  at javax.swing.AbstractButton$Handler.actionPerformed(AbstractButton.java:2318)
  at javax.swing.DefaultButtonModel.fireActionPerformed(DefaultButtonModel.java:387)
  at javax.swing.DefaultButtonModel.setPressed(DefaultButtonModel.java:242)
  at javax.swing.plaf.basic.BasicButtonListener.mouseReleased(BasicButtonListener.java:236)
  at java.awt.AWTEventMulticaster.mouseReleased(AWTEventMulticaster.java:272)
  at java.awt.Component.processMouseEvent(Component.java:6038)
  at javax.swing.JComponent.processMouseEvent(JComponent.java:3260)
  at java.awt.Component.processEvent(Component.java:5803)
  at java.awt.Container.processEvent(Container.java:2058)
  at java.awt.Component.dispatchEventImpl(Component.java:4410)
  at java.awt.Container.dispatchEventImpl(Container.java:2116)
  at java.awt.Component.dispatchEvent(Component.java:4240)
  at java.awt.LightweightDispatcher.retargetMouseEvent(Container.java:4322)
  at java.awt.LightweightDispatcher.processMouseEvent(Container.java:3986)
  at java.awt.LightweightDispatcher.dispatchEvent(Container.java:3916)
  at java.awt.Container.dispatchEventImpl(Container.java:2102)

Hat jemand eine Idee woran das liegt bzw was hilfreicher wäre - wie ich das beheben kann? Mitn sleep vllt, aber das einlesen dauert so schon recht lang

Hinweise und Lösungen sind gern gesehen

MfG erZ

 

[The_S]

Naja, die JVM hat halt nur 64MB zur Verfügung. Wenn du zu viele bzw. zu große Bilder hast, dann ist der Speicher halt schnell voll. Optimierungsvorschläge:

1.) Verwende eine ArrayList anstelle eines Vectors.
2.) Geb der ArrayList einen Initialisierungswert mit der (geschätzten) Anzahl der Bilder bei der Initialisierung mit.
3.) Wenn du nicht alle Bilder gleichzeitig benötigst, speicher dir im Vector lediglich den Pfad und lade die Bilder dann bei bedarf.

und wenn das alles nicht hilft/nicht möglich ist, musst du wohl den Heap der JVM erhöhen (-Xmx bzw. -Xms).

Und noch ein allgemeiner Tipp:

Nicht immer alles mit statischen Aufrufen machen! Es gibt nicht umsonst die Möglichkeit Objekte zu erzeugen .

 

[erZ]

Ah super danke

Na ja das mit statischen Aufrufen ist so ne Sache. Mir kommts so vor, dass man bei großen Projekten paar Dinge am besten Lösen kann mit statischen Aufrufen. Zur Zeit hab ich so 98% dynamik drin und ich versuch auch schön alles per Laufzeit (nur das notwenigste) zu lösen. Aber bei zB dem Hauptfenster, wo ich fast überfall von zugreifen muss - fand ich die statische Idee besser, als den Spaß immer als parameter zu übergeben (oder ich bin noch zu unerfahren )

Danke auf jedenfall nochmal, erZ

 

[The_S]

Ich denke wohl eher, du bist zu unerfahren. Kannst dir ja mal wenn du lust hast MVC und das Observer-Pattern angucken .

 

[byte]

Dann solltest Du zumindest mit Singletons arbeiten anstatt mit direkten statischen Zugriffen. Wobei Singletons eigentlich auch schon wieder viel zu sehr verleiten, das Pattern zu abusen. Da entsteht dann auch schnell ein wirr verdrahtes Kneul an Modulen. :roll:

 

[The_S]

Bevor er sich als Anfänger mit dem Anti-Pattern schlechthin beschäftigt, lieber gleich richtig programmieren .