Effizientes zeichnen mit Swing

[Fabian23]

Hallo zusammen,

ich stehe im Moment vor dem Problem, viele grafische Objekte auf einem JPanel zeichnen zu müssen. Diese Objekte sind editierbar (Größe, Position etc.).

Das Problem dabei ist, dass dieses zeichnen nicht wirklich performant ist. Beim Bewegen einer Figur beispielsweise steigt die Prozessorlast auf 100%.

Folgendermaßen zeichne ich:

Ich habe die einzelnen Objekte gekapselt und denen eine paint-Methode verpasst. Die Objekte, die sich logisch auf dem JPanel befinden, liegen alle in einem Vector. Wird das JPanel neu gezeichnet, zeichne ich erst einen Hintergrund und gehe dann den Vector durch, und rufe für jedes Objekt die Paint-Methode auf (ich übergebe das Graphics-Objekt). Dadurch zeichnen sich die ganzen Figuren auf dem JPanel.

Was ich schon rausgefunden habe, ist, dass jedes Objekt, dass neu gezeichnet werden will, ein repaint des JPanels aufruft und dadurch die gesamte Collection der Objekte durchgegangen wird. Das ist auf jeden Fall nicht performant.

Habt ihr Ideen, wie man effizient auf ein JPanel (oder einem anderen Control) zeichnen kann? Ich weiß leider nicht, wie ich das Zeichnen verbessern kann.

Bin auf Ideen jeglicher Art, Links, Tipps und Tricks oder was auch immer gespannt .


Gruß,
Fabian

 

[m@nu]

1. bau dir clipping in deine paint-methoden ein. somit sagst du java, welcher bereich gezeichnet werden soll (nicht vergessen wieder zurückzusetzen!)

g.setClip(x,y,w,h);
//...
g.setClip(0,0, getWidth(), getHeight());

2. schau zu, dass wenn repaint() aufgerufen wird, du dort ebenfalls den bereich mitgeben kannst, welcher neu gezeichnet werden soll (hat mir kürzlich extrem geholfen :arrow: http://www.java-forum.org/de/viewtopic.php?p=238612#238612)

repaint(x,y,w,h);

und zu guter letzt 3.
nicht die paint-methode vom JPanel überschreiben (macht man einfach nicht ) ... schreib dir den eigenen code in der paintComponent() methode.

das mit der prozessorlast läst irgendwie darauf schliessen, dass du ein problem mit den repaintings bekommst. kann sein das sich dein code da irgendwie in eine endlosschleife verläuft? (hatte ich 'mal... bekam direkt StackOverflow-Exceptions, welche ich mir erst nach 1-tägigem debuggen erklären konnte )

hoffe das hilft dir irgendwie weiter ansonsten wieder melden und evtl. code posten


info am rande: du zeichnest nicht mit swing, du zeichnest mit AWT resp Java2D (wenn du Graphics2D verwendest ) ... swing is nur die GUI bibliothek. (ja ich weis, gibts bei AWT auch, aber ist dort halt der selbe name)

 

[Wildcard]

Was ich schon rausgefunden habe, ist, dass jedes Objekt, dass neu gezeichnet werden will, ein repaint des JPanels aufruft und dadurch die gesamte Collection der Objekte durchgegangen wird. Das ist auf jeden Fall nicht performant.

Das muss ja einen Grund haben. Warum ist das so? Zeig mal Beispielcode?
Bringt nicht übermäßig viel, aber schonmal ein Anfang:
Mach aus deinem Vector eine LinkedList. Bei sequentiellem Zugriff ist die schneller.

 

[kaie]

An dem grundsätzlichen Vorgehen dürfte es nicht liegen, das sollte von der Performanz keine Probleme bereiten. Ich arbeite hier an einer Verkehrssimulation, die nach dem gleichen Prinzip mehrere Tausend Objekte (Autos, Straßen, Fußgänger, Hilfslinien) gleichzeitig darstellt, und dabei sogar noch einigermaßen performant bleibt.

Dein Grundschema sollte wie folgt aussehen:

public class Bild extends JPanel
{
  public Vector objekte = new Vector();
  public void paintComponent( Grapics g )
  {
    super.paintComponent(g);
    for( int i=0; i<objekte.size(); j++ )
    {
      ((Component)objekte.elementAt(i)).paint(g);
    }
  }
}

Damit solltest Du eigentlich alles ohne Probleme zeichnen können. Und das repaint() musst Du nur aufrufen, wenn sich irgendwo was ändert.

 

[Fabian23]

Hallo zusammen,

danke für die vielen Antworten und zahlreichen Tipps.

@m@nu: Das mit dem clipping und dem repaint(Rectangle) ist eine gute Idee. Im Moment habe ich noch Probleme, weil nicht richtig neu gezeichnet wird. Da probiere ich aber noch mal rum.

Die paint-Methode habe ich nicht überschrieben. Habe ich mich wohl falsch ausgedrückt. Ich habe meine Zeichenoperationen in die paintComponent-Methode eingebaut.

Eine Endlosschleife habe ich wohl nicht drin. Zumindest hört die Prozessorlast ja irgendwann auf und eine StackOverflowException erhalte ich auch nicht .

@Wildcard: Das hat den Grund, weil ich es so programmiert habe . Immer, wenn sich etwas in einem Shape ändert, wird das repaint() des JPanels aufgerufen und eben alle Shapes neu gezeichnet. Sollte man das evtl. auch ändern?

Den Vector werde ich mal in eine LinkedList ändern.

@kaie: Danke für die Beruhigung, dass ich anscheinend nicht alles grundlegend falsch gemacht habe. So wie Dein Code-Beispiel sieht der Part meiner paintComponent-Methode aus, die alle Shapes zeichnet.

 

[André Uhres]

Wenn man die ganze Vectorzeichnerei auf ein Offscreenbild machen würde und einfach nur das fertige Bild in paintComponent zeichnet, geht repaint dann nicht schneller?

 

[Fabian23]

@André Uhres: Das habe ich mich auch schon gefragt. Allerdings habe ich dann doch einen Schritt mehr beim Zeichnen oder? Erst alles auf ein Bild und dann aufs JPanel, statt direkt alles aufs JPanel. Ist das nicht langsamer?

 

[byte]

Wenn Du nur die Objekte neu ins Bild zeichnen lässt, die sich auch wirklich geändert haben und dann das Bild in repaint() darstellst, sollte das schon deutlich schneller sein, als wenn Du immer alles direkt neu zeichnest.

 

[Fabian23]

Gibt es eine bestimmte Vorgehensweise, wenn ich zuerst auf das Bild zeichne? Welchen Typ Bild sollte man nehmen (BufferedImage) oder ist das egal?

Hat jemand vielleicht ein Beispiel, wie man es machen sollte?

 

[Wildcard]

Am besten nimmst du ein VolatileImage