Reihenfolge von Befehlen - hier repaint() zu spät

[virtualAudio]

Hi Leute,

es ist mir diesmal ein wenig zu spät für ein KSKP...und ich hab die Hoffnung, dass vielleicht auch so jemand Rat weiß und das Problem gelöst sein kann bevor ich dazu komme ein KSKP zu schreiben.

(eclipse)

					if(e.getActionCommand().equals("Screenshot")){
						new Say("popup weg");
						popUpM.setVisible(false);		//popupmenü wieder schliessen
						new Say("davor");
						pane.repaint();
						new Say("danach");

						mutter.getScreenshot(pane, new Rectangle(getLocation().x, getLocation().y,getWidth(),getHeight()));
						
					}

dieses codeschnipsel aus dem Listener eine popupmenüs sollte eigentlich ein JPanel neuzeichnen, da das popup deaktiviert wird und an dessen Stelle sonst ein Fleck "nichts" übrigbleibt, und dann vom frisch gezeichneten JPanel einen screenshot machen.

Probelm: am screenshot sieht man den Fleck der nicht neu gezeichnet wurde. Ich hab dann diese ganzen Strings dazwischen geschrieben um zu sehen in welcher Reihenfolge abgearbeitet wird.

zur Erklärung:

public void Say(String s) {System.out.println("----> " + s)};

in der paintComponent steht zusätzlich noch "bin in paint" und in der getScreenshot steht "cheeeese".

Eigentlich sollte die Ausgabe also lauten: popup weg, davor, bin in paint, danach, cheeeese

sie lautet aber: popup weg, davor, danach, cheeeese, bin in paint -> was dann auch die nicht aktualisierte Grafik in meinem Screenshot(jpg) zeigt.


Vielleicht bringt das jemanden ja schon auf eine Idee woran es liegen könnte. Würd mich freuen.

Beste Grüße
Andi

 

[virtualAudio]

gerade hab ich noch ausprobiert herauszufinden ob repaint() vielleicht garnicht an dieser Stelle sondern erst wieder ausgeführt wird, wenn der FileSaveDialog(zum screenshot abspeichern) geschlossen wird.
Allerdings hab ich jetzt eine getScreenshot version die ohne diesen Dialog auskommt und immer an die selbe Stelle speichert...da steht auch nach dem ganzen 3x"bin in paint"...kann also doch sein, dass da mein Befehl die repaint auszuführen noch ausgeführt wird (zu spät wie gesagt)
hmmm

lg Andi

 

[Beni]

Repaint hat keinen direkten Einfluss, es wird nur eine Notiz gemacht "beim nächsten Durchlauf bitte zeichnen." Der EventDispatcherThread wird diese Notiz sehen, und dann was zeichnen. Solange du aber den EDT in deiner "actionPerformed"-Methode blockierst, hat er überhaupt keine Möglichkeit etwas neu zu zeichnen.

In diesem Fall weiss ich nicht, für was der Screenshot gut sein sollte. Wenn du das Panel in einem Bild haben willst, lass es lieber direkt auf ein Bild zeichnen. Dann hast du auch keine Probleme, falls es verdeckt ist.

 

[André Uhres]

       SwingUtilities.invokeLater(new Runnable(){
            public void run() {
                mutter.getScreenshot(pane, new Rectangle(getLocation().x, getLocation().y,getWidth(),getHeight())); 
            }
        });

 

[virtualAudio]

Hi Leute, danke für die Antwort.

@Beni:
müsste ich dann bei jedem paint ein neues BufferedImage erzeugen? um auf diesem zu zeichnen?
Bei jeder Grössenveränderung des JPanels müsste ja eine andere grösse des Bildes erzeugt werden auf dem gezeichnet wird richtig?

Frisst das nicht sehr viel Rechenleistung?
Vor allem wegen diesen Bedenken hab ich das bis jetzt nicht gemacht. Jetzt da Du mir den Grund dafür genannt hast warum repaint() nicht gleich ausgeführt wird, werd ich es mal versuchen.

@André: leider hab ich von threads noch nicht viel Ahnung. invoke later würde zu sowas führen wie führe zuerst repaint() aus und erst dann erzeuge den Screenshot?

Beste Grüße
Andi

 

[Wildcard]

Ich bin mir nicht ganz sicher was du eigentlich vorhast, aber wenn dir das hilft, es gibt auch eine Möglichkeit direkt neu zu zeichnen:

paintImmediately(Rectangle r);

 

[André Uhres]

Jup, Wildcard's Vorschlag müsste so funktionieren:
pane.paintImmediately(pane.getBounds());

 

[virtualAudio]

wow..ich glaube das wäre perfekt...schauma mal


warum ich auf die Leistung achte...

die zu zeichnenden Daten sind von der cursorposition abhängig, sprich paint wird immer ausgeführt wenn sich die Maus gerade über dem Panel bewegt...

 

[Wildcard]

warum ich auf die Leistung achte...
die zu zeichnenden Daten sind von der cursorposition abhängig, sprich paint wird immer ausgeführt wenn sich die Maus gerade über dem Panel bewegt...

Aber vorsicht:
bei paintImmediatly werden redundante Aufrufe nicht abgefangen weshalb es schnell unmengen an Rechenzeit verschlingen kann. Achte darauf den so gezeichneten Bereich möglichst klein zu halten.

 

[virtualAudio]

Hi wildcard,

erstmal danke für den Tipp....erstmal funktioniert der Spaß wie gewünscht.

Leider funktioniert es nicht wenn ich als neu zu zeichnende Fläche PupUpM.getBounds() verwende...mit pane.getbounds() funktioniert es...vielleicht spiel ich mich da noch.

meinst Du mit redundant...mehrere Aufrufe wo es eine auch tun würde?
Sollte hier kein Problem darstellen da ich paintImmediately nur einmal vor dem Screenshot aufrufe...ansonsten verwende ich weiterhin überall rapaint().
Oder meinst Du etwas anderes?

Dankbarste Grüße

 

[Wildcard]

Leider funktioniert es nicht wenn ich als neu zu zeichnende Fläche PupUpM.getBounds() verwende...mit pane.getbounds() funktioniert es...vielleicht spiel ich mich da noch.

Einfach mal Debuggen und schauen was da ankommt...

meinst Du mit redundant...mehrere Aufrufe wo es eine auch tun würde?
Sollte hier kein Problem darstellen da ich paintImmediately nur einmal vor dem Screenshot aufrufe...ansonsten verwende ich weiterhin überall rapaint().
Oder meinst Du etwas anderes?

Ja, so ziemlich genau das hab ich gemeint. Wenn du es allerdings nur einmal aufrufst sollte das bei dir wirklich kein Problem sein.

 

[virtualAudio]

hmmm, laut Debugger ist width=181, height =89;
dann gibts zwei Variablen x und y die sind 0 ....dachte das erklärts

es gibt aber noch zwei variablen desiredlocationx und y die haben einen Absolutwert (Bildschirm) der passen würde und aus dem man sich ja leicht ausrechnen könnte wo man neuzeichnen muss. Allerdings: versteh ich nicht warum der wert x und der wert y die ja über getX und getY angesprochen werden 0 ist und bleibt (hab an verschiedenen stellen breakpoints um das zu überprüfen... zB auch bei PopUpM.setVisible(true)...und zumindest dort wären x und y wenn die dafür zuständig wären > = 0... sind sie nicht...wenn sie aber nicht zuständig sind für die Position des popups versteh ich nicht warum getX() und getY() diese zurückliefern.

über desiredLocation hab ich in der API nix gefunden... hmm

naja, es funktioniert ja wenn ich das ganze panel neu zeichnen lasse... interessant wärs trotzdem...

gruuhs Andi

 

[André Uhres]

..@André: leider hab ich von threads noch nicht viel Ahnung.
invoke later würde zu sowas führen wie führe zuerst repaint() aus und erst dann erzeuge den Screenshot?..

Ja, so funktioniert das.
Btw, mit invokeLater wird kein neuer Thread gestartet.
Der Screenshot wird lediglich verzögert: erst repaint, dann Screenshot (alles auf dem Eventdispatching Thread).

 

[virtualAudio]

Danke nochmal allen Helfern!!

Hat mir nicht nur bei diesem Problem, dass dank Euch ja schnell gelöst wurde, geholfen, sondern hat auch mein Verständnis gegenüber Java bereichert. Super!

beste Grüße
Andi