Probleme mit paintComponent

[Carcharoth]

Hallo

Da ich paintComponent nicht direkt aufrufen will, habe ich mir eine Methode zeichne gebaut die repaint aufruft.
Ich dachte bisher den Aufruf von repaint würde intern zur Folge haben, dass paintComponent aufgerufen wird.
Liege ich damit falsch ?

Jedenfalls muss ich feststellen, dass der Aufruf von zeichne, paintComponent nicht aufruft (habe ich getestet).
Hat jemand ein Tipp?

public class GrafischeDarstellung extends JLabel {

    public void zeichne() {
        repaint();
    }

    @Override
    protected void paintComponent(Graphics g) {
        super.paintComponent(g);

        ...
    }

}

Danke und Gruss

 

[mihe7]

Ich dachte bisher den Aufruf von repaint würde intern zur Folge haben, dass paintComponent aufgerufen wird.
Liege ich damit falsch ?

Nein, das stimmt.

Jedenfalls muss ich feststellen, dass der Aufruf von zeichne, paintComponent nicht aufruft (habe ich getestet).
Hat jemand ein Tipp?

Mach ein kleines Beispiel, anhand dessen man das Problem nachvollziehen kann.

 

[kneitzel]

Also repaint ruft nicht direkt paintComponent auf. Es wird lediglich ein Event generiert, das die Komponente anweist, bei nächster Gelegenheit neu gemalt zu werden.

Der EDT muss das Event also erst verarbeiten. Wenn Du aber den EDT blockierst, dann wird auch nichts gemalt. (Als kleine Vermutung, dass Du sowas machen könntest...)

 

[Carcharoth]

In die Richtung habe ich auch gedacht. denn wenn ich vor dem Aufruf ein JOptionPane.Message einbaue, dann geht es.

Ich hatte mal ein ähnliches Problem mit einem MouseListener der das Updaten eines JFrames blockierte. Damals wurde es mit einem separat laufender Thread gelöst. Ich versuche hier die Lösung umzusetzen, wobei mir noch nicht so ganz klar ist, was jetzt in welchem Thread laufen muss... Ich muss noch etwas drüber brüten.

 

[kneitzel]

wobei mir noch nicht so ganz klar ist, was jetzt in welchem Thread laufen muss

Alles, was etwas länger läuft, das kommt in einen eigen Thread, damit der EDT nicht blockiert wird.
Dabei aber alle Änderungen an der UI über SwingUtilities invokeLater durchführen, denn Änderungen an der UI sollten immer im EDT erfolgen.

Das ist unter dem Strich auch ähnlich dem repaint: Es wird ein Event in die EventQueue gepackt, das den EDT anweist, das im invokeLater aufzurufen. Es gibt also einen gewissen Zeitversatz,

 

[Carcharoth]

Meine stark entschlackte Klasse mit der Main-Methode sieht wie folgt aus.
- crossExchange.optimieren() ist eine Methode einer anderen Klasse die umfangreiche Berechnungen anstellt und ein Boolean zurückgibt falls eine Optimierung gemacht werden konnte.
- meineGrafischeDarstellung.updateWegSOE() ist eine Methode der Klasse GrafischeDarstellung in der die Ergebnisse der Berechnungen dargestellt werden und die überschriebene paintComponent Methode aufgerufen wird. Ich habe versucht dieses Update des UI innerhalb des Invokelaters zu platzieren aber ich bin mir nicht sicher mit dem new Thread. Müsste dort die gesamte Berechnungsschleife drin sitzen ?

import java.awt.Color;
import java.awt.EventQueue;

import javax.swing.JFrame;
import javax.swing.JPanel;
import javax.swing.SwingUtilities;
import javax.swing.border.EmptyBorder;
import javax.swing.border.LineBorder;

public class UF extends JFrame {

    public JPanel panel;
    private JPanel contentPane;
    private GrafischeDarstellung meineGrafischeDarstellung;
    
    public static void main(String[] args) {
        EventQueue.invokeLater(new Runnable() {
            public void run() {
                try {
                    UF frame = new UF();
                    frame.setVisible(true);
                } catch (Exception e) {
                    e.printStackTrace();
                }
            }
        });
    }
    
    public UF() {
        // UserForm einrichten
        createComponents();
        
        // GrafischeDarstellung erstellen
        meineGrafischeDarstellung  = new GrafischeDarstellung(800,800);
        panel.add(meineGrafischeDarstellung);

        // Berechnung durchführen und GrafischeDarstellung anpassen
        while (crossExchange.optimieren()) {
                new Thread(new Runnable() {
                    @Override
                    public void run() {
                        SwingUtilities.invokeLater(new Runnable() {
                            @Override
                            public void run() {
                                meineGrafischeDarstellung.updateWegSOE();
                            }
                        });
                    }
                }).start();       
        }
    }
    
    private void createComponents() {
        setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        setBounds(0, 0, 1300, 950);
        setLocationRelativeTo(null);
        setResizable(false);

        contentPane = new JPanel();
        contentPane.setLayout(null);
        contentPane.setBorder(new EmptyBorder(5, 5, 5, 5));
        setContentPane(contentPane);

        panel = new JPanel();
        panel.setBorder(new LineBorder(new Color(0, 0, 0)));
        panel.setBounds(0, 0, 800, 800);
        contentPane.add(panel);
        panel.setLayout(null);
    }

}

 

[mihe7]

Die Schleife ab Zeile 38 muss in einen eigenen Thread.

 

[kneitzel]

Also das sieht extrem dubios aus:

        while (crossExchange.optimieren()) {
                new Thread(new Runnable() {
                    @Override
                    public void run() {
                        SwingUtilities.invokeLater(new Runnable() {
                            @Override
                            public void run() {
                                meineGrafischeDarstellung.updateWegSOE();
                            }
                        });
                    }
                }).start();       
        }

Das crossExchange.optimieren() sind die Berechnungen? dann gehört das in einen Thread....

Der Aufruf von invokeLater muss in keinen eigenen Thread - das ist etwas, das ganz schnell geht. Das macht also keinen Sinn.

Also evtl. sollte es etwas sein wie:

new Thread(new Runnable() {
    @Override
    public void run() {
        while (crossExchange.optimieren()) {
            SwingUtilities.invokeLater(() -> meineGrafischeDarstellung.updateWegSOE());
        }
    }
}).start();

Also es wird ein Thread erzeugt, der so lange optimieren aufruft, bis es false zurück gibt.
Und immer, wenn es true zurück gibt, wird updateWegSOE im EDT ausgeführt.

 

[Carcharoth]

OK jetzt habe ich es umsetzen können. Es läuft super.

Was ich nicht ganz verstehe, ist warum meine Berechnungen den EDT blockiert haben (und dadurch die GrafischeDarstellung nicht angepasst wurde).

Bei folgendem Code, den ich vor dem Einsatz des separaten Threads hatte, wurde ja der Berechnungsschritt immer fertiggemacht, und dann erst der UI-Update verlangt. Ich verstehe nicht wieso der UI-Update sich durch die Berechnung blockiert fühlt ;-)
Ich habe wohl die Essenz der Threadgeschichte noch nicht ganz geschnallt.

while (crossExchange.optimieren()) {
            meineGrafischeDarstellung.updateWegSOE();   
}

 

[kneitzel]

Das ist ganz einfach. Stell Dir vor, du bist der Zuständige Thread ...

Du hast einen Stapel mit Aufgaben, die Du erledigen sollst. Und was Du nun machst ist:
Du berechnest etwas. Wenn Du mit der Berechnung fertig bist, dann schreibst Du Dir einen Zettel mit einer neuen Aufgabe: "Fenster neu zeichnen" und legst ihn auf den Stapel - aber dann machst Du weiter wo Du aufgehört hast: Bei der Berechnung ...

Somit berechnest du und berechnest und berechnest ... schreibst auch fleissig ganz viele Zettel mit "Fenster neu zeichnen", aber da Du ja non stop am berechnen bist, kommst Du nie zum zeichnen des Fensters ...

Mit dem Thread sagst Du einem anderen: mach die Berechnungen!
Der macht jetzt die Berechnungen und legt Dir dann hin und wieder einen Zettel hin ... Und da Du nichts zu tun hast, nimmst Du den Zettel und machst, was da drauf steht. (Ja, EDT muss man sein ... das ist ein ruhiger Job - man kann in der Regel ganz viel schlafen und auf Zettel warten!)

 

[Carcharoth]

OK, ich glaube ich habe es verstanden.
und, ja, EDT wäre schon ein cooler Job aber vielleicht auch nicht sooo spannend ;-)
Danke für die Hilfe.

 

[Carcharoth]

Ich habe eine kleine Nachtragsfrage:
Ich habe herausgefunden, wie ich eine Variable (hier den Zähler i) in das invokeLater reinschmuggeln kann.
Wie könnte ich aber eine Variable wieder herausholen (hier tourID)?

for(int i=0;i<10;i++) {
    final int iKopie = i;
    SwingUtilities.invokeLater(new Runnable() {
        @Override
        public void run() {
            tourID[iKopie]=meineGrafischeDarstellung.updateWegSOE(iKopie);
        }
    });
}

 

[Carcharoth]

...oder ist das eine unsinnige Frage, weil man nicht sicher sein kann, wann der Inhalt des InvokeLaters ausgeführt werden kann?

Dann muss ich mir was anderes überlegen, denn ich brauch die ID die während dem Updaten der GrafischenDarstellung generiert wird, um sie später gezielt einzeln ansprechen zu können.

 

[mihe7]

In der Regel sollte die View keine Logik enthalten. Für die Geschäftslogik gibt es einen eigenen Layer, auf den der Presentation Layer zugreift, um eben für den Benutzer eine entsprechende Darstellung anzubieten.