java swing jscrollpane

[tom (Gast)]

Hallo allerseits,

folgendes Problem:
Ich habe einen Java-JFrame, der eine JScrollPane enthält, die wiederum 
ein großes JPanel enthält:

...
public MyFrame(){
   JPanel p = new JPanel();
   JScrollPane sp = new ScrollPane(p,JScrollPane.VERTICAL_SCROLLBAR_AS_NEEDED,JScrollPane.HORIZONTAL_SCROLLBAR_AS_NEEDED);
   sp.getHorizontalScrollBar().addAdjustmentListener(irgendeinAdjustmentListener);
   sp.getVerticalScrollBar().addAdjustmentListener(irgendeinAdjustmentListener);
}
...

Dann wird das JPanel "bemalt", was relativ viel Zeit in Anspruch nimmt, 
da es sich um eine große Farbmatrix handelt.
Wenn ich jetzt scrolle, wird die

adjustmentValueChanged()

 Methode des AdjustmentListeners der aufrufenden JScrollBar aufgerufen, 
die ein

repaint()

 des JPanel zur Folge hat.

Zum Problem:
Lasse ich bei jedem repaint() des JPanel die gesamte Farbmatrix neu 
malen, zeigt er beim scrollen alles schön ABER es dauert zu lang, bis 
das gesamte JPanel neu gemalt ist.
Lasse ich das JPanel nicht neu malen beim scrollen, so wird dort, wohin 
ich srolle nur weißer Hintergrund angezeigt.

Wie schaffe ich es auf einfachem Wege, dass das die Farbmatrix des 
JPanel beim Scrollen überall zu sehen ist, ohne bei jedem Scrollen das 
gesamte JPanel neu zeichnen zu müssen?

[Markus M. (mark_m)]

Du musst wahrscheinlich das "Scrollable interface" implementieren.

Grüsse

[Läubi .. (laeubi)]

tom schrieb:
> Wie schaffe ich es auf einfachem Wege, dass das die Farbmatrix des
> JPanel beim Scrollen überall zu sehen ist, ohne bei jedem Scrollen das
> gesamte JPanel neu zeichnen zu müssen?

Am einfachsten indem man (nicht nur beim scrollen) das Bild auf ein 
BufferedImage zeichnet und immer wenn sich etwas geändert hat dieses neu 
malt und (im AWT Thread) repaint() aufruft.
Dieses Bild kann man dann in der Paintmethode gemütlich ausgeben, und 
muss auch nicht ständig neuzeichnen. Während das Bild gemalt wird, 
könnte man sogar einen Loadingindicator zeigen...

[tom (Gast)]

Hallo Markus,
danke für die schnelle Antwort!

Zuerst mal klappt deine Lösung ziemlich gut.
Es gibt nur noch ein Problem:
Scrolle ich vertikal, so klappt alles super.
Scrolle ich aber horizontal, wird nur der letzte "Block" der Farbmatrix 
in horizontaler Richtung hinten dran gesetzt.
Scrolle ich DANACH wieder vertikal, passiert das gleiche in vertikaler 
Richtung.

Konstruktor JFrame:

...
CanvasPanel canvasPanel = new CanvasPanel();
ScrollPaneListener spListener = new ScrollPaneListener();
scrollPane = new JScrollPane(canvasPanel,JScrollPane.VERTICAL_SCROLLBAR_AS_NEEDED,JScrollPane.HORIZONTAL_SCROLLBAR_AS_NEEDED);        scrollPane.getHorizontalScrollBar().addAdjustmentListener(spListener);        scrollPane.getVerticalScrollBar().addAdjustmentListener(spListener);
...
[\code]

ScrollPaneListener:
[code]
class ScrollPaneListener implements AdjustmentListener{
   public void adjustmentValueChanged(AdjustmentEvent e){
      // nur wenn gescrollt wurde
      if(scrollPane.getHorizontalScrollBar().getValueIsAdjusting() || scrollPane.getVerticalScrollBar().getValueIsAdjusting()
         canvasPanel.setDrawMode(CanvasPanel.NONE);
   }
}

CanvasPanel (implementiert Interface Scrollable):

...
public void paint(Graphics g){
   if(drawMode==NONE) // wenn gescrollt wird
      return;
   else{
      ...
      // sonst male Farbmatrix
      // sehr zeitaufwendig
   }
}
...
// Scrollable Methoden
public Dimension getPrefferedScrollableViewportSize(){
   return this.getPreferredSize();
}

public int getScrollableUnitIncrement(Rectangle rect, int orientation, int dir){
   if(orientation==SwingConstants.HORIZONTAL)
      return horStepSize;
   return vertStepSize;
}

public int getScrollableBlockIncrement(Rectangle rect, int orientation, int dir){
   if(orientation==SwingConstants.HORIZONTAL)
      return horBlockSize;
   return vertBlockSize;
}

public int getScrollableTracksViewportWidth(){
   return false;
}

public int getScrollableTracksViewportHeight(){
   return false;
}
...

[tom (Gast)]

Hallo Läubi,

danke für den Vorschlag.
Hab leider noch nie mit BufferedImage gearbeitet.
Hast du ein Codeschnipsel, das mir deine Idee verdeutlicht?

Danke

[Markus M. (mark_m)]

Man könnte jetzt lange das Vorgehen Diskutieren. Ich denke ein Beispiel 
bringt dir mehr.

Oracle stellt ein schönes Tutorial bereit:
http://docs.oracle.com/javase/tutorial/uiswing/components/scrollpane.html#scrollable

Ich hoffe Du kommst mit Englisch einigermassen zurecht.

Grüsse

[tom (Gast)]

Also ich habe es gelöst mit Läubis Weg und es funktioniert sehr gut.

CanvasPanel:

...
BufferedImage buffImage = new BufferedImage(breite,hoehe,BufferedImage.TYPE_INT_RGB);
...
@Override
public void paintComponent(Graphics g){

   super.paintComponent(g);

   if(buffImage==null)
      return;
   
   if(drawMode==NONE){ // wenn nur gescrollt wird
      g.drawImage(buffImage,0,0,null);
      return;
   }

   Graphics gImage = buffImage.createGraphics();

   /****
   Hier jetzt alle Zeichenaufrufe auf dem Objekt gImage!
   z.B. gImage.drawLine(0,0,10,10);
   ****/

   g.drawImage(buffImage,0,0,null);
}
...

[Läubi .. (laeubi)]

Wenn du jetzt das Zeichnen noch außerhalb des AWT Threads machst geht es 
noch besser ;-)
Das geht z.B. mit einer AtomicReference in etwa so:

AtomicReference<BufferedImage> buffImage = new AtomicReference();
...
@Override
public void paintComponent(Graphics g) {
   BufferedImage img = buffImage.get();
   super.paintComponent(g);
   if(img==null)
      return;
   g.drawImage(img,0,0,this);
}

public void setData(List<..> meineDaten,...) {
   BufferedImage img = new BufferedImage(breite,hoehe,BufferedImage.TYPE_INT_RGB);
   Graphics img = buffImage.createGraphics();

   /****
   Hier jetzt alle Zeichenaufrufe auf dem Objekt gImage!
   z.B. gImage.drawLine(0,0,10,10);
   ****/
   buffImage.set(img);
}

Die Signatur von setData ggf. an deine Bedürfnisse anpassen. Wenn du 
dann in einem Swingworker in der execute Methode setData aufrufst und in 
done dann repaint bleibt deine GUI reaktiv und blockiert auch nicht beim 
ersten Aufruf, ebenso nicht beim scrollen.

[tom (Gast)]

Super, vielen Dank!

Thread closed

[tom (Gast)]

Vllt noch mal eine Frage zu dem SwingWorker.
Wie benutze ich den? Die Oracle API hat mir nicht wirklich 
weitergeholfen.

Mein Programm soll folgendes machen:
Es gibt ein Haupt-JPanel, darin befindet sich eine JScrollPane und ein 
JButton. Die JScrollPane hält das CanvasPanel.
- Wird jetzt der Button gedrückt, werden die Daten geändert und das Bild 
im CanvasPanel mit diesen neuen Daten neu gemalt.
- Zusätzlich gibt es im Haupt-JPanel einen per JButton aktivierbaren 
Timer, der die neuen Daten aller x ms automatisch berechnet und das 
CanvasPanel automatisch aktualisiert.

Hab halt bisher immer in der actionPerformed() des Timers repaint() 
aufgerufen aber das ist mit meinem oben aufgeführten Code doch recht 
langsam.

MainPanel:

...
button1.addActionListener(new ActionListener(){

public void actionPerformed(ActionEvent e){
   daten = berechneNeueDaten();
   canvasPanel.setDaten(daten)
   canvasPanel.repaint();
}
});

button2.addActionListener(new ActionListener(){
   
   Timer timer;

   public void actionPerformed(ActionEvent e){
      timer = new Timer(stepTime, new ActionListener(){
       
         public void actionPerformed(ActionEvent e){
            daten = berechneNeueDaten();
            canvasPanel.setDaten(daten);
            canvasPanel.repaint();
         }
      });

      timer.start();

      // blockiere AWT-Thread bis OK gedrückt wird
      JOptionPane.showMessageDialog(parentFrame, "Timer mit OK stoppen", "",0);
   
      timer.stop();
   }
});
...

Wie könnte ich das ganze in einem SwingWorker realisieren, um deinen 
obigen Code für CanvasPanel nutzen zu können?

[Läubi .. (laeubi)]

Ich hab dir doch schon oben geschrieben wie man das (asyncron) machen 
könnte.
Du must einfach mal deine Gedanken/den Code etwas Strukturieren.
Es gibt hier doch mehrere Einzelaufgaben, die man sehr schön trennen:

- Das Zeichnen an sich
- Das Darstellen eines bereits gezeichneten Bildes
- Eine Regelmäßige Änderung der Daten (Timerbasiert)
- Die Steuerung von Parametern über das UI

Aktuell mantscht du einfach alles zusammen und das gibt Probleme. Der 
Swingworker ist z.B. eher für einmalige Tasks, nicht für wiederkehrende. 
Für Wiederkehrende gibt es den Timer, der ist aber eben nicht dafür um 
wiederkehrend lange Berechnungen zu machen. Dafür gibt es die 
SceduledExecutors und zusammen mit meinem obigen Beispiel sollte das 
dann auch gut zusammen spielen.

[tom (Gast)]

Danke für deine Hilfe.
Habe für mein Projekt leider nicht mehr die Zeit, um mich in 
ScheduledExecutor einzuarbeiten. Werd daher wohl bei meiner 
Timer-Variante bleiben müssen, auch wenn sie langsam ist.

Also nochmal vielen Dank.