Swing Panel wird nicht neu gezeichnet

[st8ch1]

Hallo Community,

hoffentlich könnt ihr mir helfen. Ich bin nämlich inzwischen echt verzweifelt 😳
Für ein Programm empfang ich drei Werte und diese sollen grafisch als Balken mit der entsprechenden relativen Häufigkeit dargestellt werden.

public void zeichnen_m(int h1, int h2, int h3){
        
        double ges = h1 + h2 + h3;
        double rh1, rh2, rh3;
        
        rh1 = h1/ges;
        rh2 = h2/ges;
        rh3 = h3/ges;
                
        int x =  p_dark_green_m.getX();
        int y =  p_dark_green_m.getY();
        
        int h = p_m_Bewertung.getHeight();
        
        l_dark_green_m.setText("" + h1);
        l_yellow_m.setText("" + h2);
        l_dark_red_m.setText("" + h3);
        
        int w1 = 0; p_dark_green_m.setBounds(x+w1, y, (int)(p_m_Bewertung.getWidth()*rh1), h);
        int w2 = p_dark_green_m.getWidth(); p_yellow_m.setBounds(x+w2, y, (int)(p_m_Bewertung.getWidth()*rh2), h);
        int w3 = w2 + p_yellow_m.getWidth(); p_dark_red_m.setBounds(x+w3, y, (int)(p_m_Bewertung.getWidth()*rh3), h);
    }

Der Balken ist ein Panel im BoxLayout, welcher drei Kind-Panels enthält. Die Methode soll die Verhältnisse live anpassen (was sie im Grunde auch tut )
Das ganze wird von einem Timer gesteuert

        Timer schedule = new Timer(1000, taskPerformer);
        schedule.start();
 
    
    ActionListener taskPerformer = new ActionListener() {
            @Override
            public void actionPerformed(ActionEvent e) {
                
                int[] bewertung_m = controller.getBewertung_M();
                int[] kritik_m = controller.getKritik_M();
                zeichnen_m(bewertung_m[0], bewertung_m[1], bewertung_m[2]);
                setkritik_m(kritik_m[0], kritik_m[1], kritik_m[2], kritik_m[3]);
                
                int[] bewertung_f = controller.getBewertung_F();
                int[] kritik_f = controller.getKritik_F();
                zeichnen_f(bewertung_f[0], bewertung_f[1], bewertung_f[2]);
                setkritik_f(kritik_f[0], kritik_f[1], kritik_f[2], kritik_f[3]);
                  
                l_Status.setText("Connected with [" + controller.getConnections() + "] Client(s)"); 
            }
    };

Allerdings ist mein Problem, dass zwar alle Berechnungen richtig sind, es jedoch nur bei einer erneuten iteration tatsächlich neu gezeichnet wird.
Ich habe schon versucht das Elternpanel mittels revalidate und repaint neu zeichnen zu lassen, jedoch ohne erfolg.

Dankeschön für eure Hilfe
🙂

 

[mihe7]

es jedoch nur bei einer erneuten iteration tatsächlich neu gezeichnet wird.

Was heißt bei einer erneuten Iteration?

 

[st8ch1]

Naja, wenn man die Methode z.B. testweise über einen Button auslöst, dann erst bei erneutem betätigen des Buttons

 

[mihe7]

Mit den Schnipseln kann man wenig anfangen. Erstell mal ein kompilierbares, ausführbares Beispiel, das gerade so viel enthält, dass das Problem noch auftritt. Oft findet man dabei den Fehler und wenn nicht, postest Du bitte das Beispiel.

Spoiler: Lauffähiges Beispiel

import java.awt.*;
import java.awt.event.WindowAdapter;
import java.awt.event.WindowEvent;
import java.util.*;
import javax.swing.*;
import javax.swing.Timer;

public class Test {
    int h1, h2, h3;

    JLabel green;
    JLabel yellow;
    JLabel red;
    JPanel bar;

    Timer timer = new Timer(1000, e -> {
        Random rand = new Random();
        h1 = rand.nextInt(100);
        h2 = rand.nextInt(100);
        h3 = rand.nextInt(100);        
        zeichne();
    });

    private void zeichne() {
        double ges = Math.max(h1 + h2 + h3,1);
        double rh1, rh2, rh3;
        
        rh1 = h1/ges;
        rh2 = h2/ges;
        rh3 = h3/ges;
                
        int x =  green.getX();
        int y =  green.getY();
        
        int h = bar.getHeight();
        
        green.setText("" + h1);
        yellow.setText("" + h2);
        red.setText("" + h3);
        
        int w1 = 0; green.setBounds(x+w1, y, (int)(bar.getWidth()*rh1), h);
        int w2 = green.getWidth(); yellow.setBounds(x+w2, y, (int)(bar.getWidth()*rh2), h);
        int w3 = w2 + yellow.getWidth(); red.setBounds(x+w3, y, (int)(bar.getWidth()*rh3), h);

        System.out.printf("%d %d %d, %d %d %d\n", h1, h2, h3, w1, w2, w3);
    }

    private JLabel createBox(Color c) {
        JLabel lbl = new JLabel();
        lbl.setOpaque(true);
        lbl.setBackground(c);
        return lbl;
    }

    public void run() {
        green = createBox(Color.GREEN);
        yellow = createBox(Color.YELLOW);
        red = createBox(Color.RED);

        bar = new JPanel(null);
        bar.setPreferredSize(new Dimension(300,40));
        bar.add(green);
        bar.add(yellow);
        bar.add(red);

        Box box = Box.createHorizontalBox();
        box.add(new JLabel("<html><b>Bewertung</b><br>Heute</html>"));
        box.add(bar);
        
        JFrame frame = new JFrame();
        frame.setDefaultCloseOperation(JFrame.DISPOSE_ON_CLOSE);
        frame.addWindowListener(new WindowAdapter() {
            public void windowClosing(WindowEvent e) {
                timer.stop();
            }
        });

        frame.add(box);
        frame.setSize(400, 40);
        frame.setVisible(true);

        timer.start();
    }

    
    public static void main(String[] args) {
        SwingUtilities.invokeLater(() -> new Test().run());
    }
}