Anzeige erneuern, wie geht das?

[java starter]

Ich hab eine Uhr als GUI programmiert, durch zeichnen eines kreises und anbringen von Sekunden-, Minuten- und Stundenzeigern;
Dies hat geklappt;
Dann habe ich ein Button gemacht, durch dessen Betätigen die digitale Anzeige der Uhr erscheint, hat auch geklappt Nur mein Problem ist... Wenn ich den jetzt betätige, zeigt es von mir aus 11:58:47 an, aber es aktualisiert sich nicht, wie bekomme ich das mit wenig Aufwand hin?

Ich weiß mein Programmierstil ist hier sehr rustikal, ich hoffe ihr könnt darüber hinwegsehen

import java.awt.*; 
import java.awt.event.*; 
import javax.swing.*; 
 
 
// Ein Panel zum Zeichnen 
class ZeichenPanel extends JPanel { 
    public void paintComponent(Graphics g) { 
        // aktuellen Sekundenwert der Systemuhr berechnen 
        long sekunden = (System.currentTimeMillis() / 1000) % 60; 
        long minuten = (System.currentTimeMillis() / 1000) % 3600; 
        long stunden = (System.currentTimeMillis()/1000) %(43200); 
 
        // Zeigerposition berechnen 
        double zeiger_x = Math.sin(sekunden/60.0 * 2*Math.PI); 
        double zeiger_y = Math.cos(sekunden/60.0 * 2*Math.PI); 
         
        double zeiger_x1 = Math.sin(minuten/3600.0 * 2*Math.PI); 
        double zeiger_y1 = Math.cos(minuten/3600.0 * 2*Math.PI); 
         
        double zeiger_x2 = Math.sin(stunden/43200.0 * 2*Math.PI); 
        double zeiger_y2 = Math.cos(stunden/43200.0 * 2*Math.PI); 
         
         
 
        // alte Zeiger löschen 
        g.clearRect(0, 0, getWidth(), getHeight()); 
 
        // Uhrumrandung zeichnen 
        g.drawOval(10, 10, 180, 180); 
 
        // Sekundenzeiger zeichnen 
        g.drawLine(100, 100, 100+(int)(zeiger_x*80), 100-(int)(zeiger_y*80)); 
         
        //Minutenzeiger zeichnen 
        g.drawLine(100, 100, 100+(int)(zeiger_x1*70), 100-(int)(zeiger_y1*70)); 
         
        // Stundenzeiger zeichnen 
        g.drawLine(100, 100, 100+(int)(zeiger_x2*50), 100-(int)(zeiger_y2*50)); 
         
        g.drawString("12",95,28); 
        g.drawString("6",97,181); 
        g.drawString("9",16,105); 
        g.drawString("3",178,105); 
 
    } 
} 
 
// Die Hauptklasse Uhr 
public class Uhr { 
 
    // Variable für das Zeichenpanel 
    static ZeichenPanel panel; 
    static JButton analogdigital; 
    static JLabel aendern; 
 
 
    public static void main(String[] args) { 
        // Fenster einrichten 
        JFrame frame = new JFrame("Uhr"); 
        frame.setSize(610, 250); 
        frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE); 
 
        // Zeichenpanel anlegen 
        panel = new ZeichenPanel(); 
        frame.add(panel); 
     
        frame.setLayout(new GridLayout()); 
         
        analogdigital = new JButton("Wechsel Analog/Digital"); 
        frame.getContentPane().add(analogdigital); 
         
        aendern = new JLabel("",10); 
        frame.getContentPane().add(aendern); 
         
        analogdigital.addActionListener( 
                new ActionListener() { 
                    public void actionPerformed(ActionEvent e) { 
                    long sekunden = (System.currentTimeMillis() / 1000) % 60; 
                    long minuten = (System.currentTimeMillis() / 1000) % 3600; 
                    long stunden = (System.currentTimeMillis()/1000) %(43200); 
     
                    aendern.setText("                          "+stunden/3600+":"+minuten/60+":"+sekunden); 
                    } 
                } 
                ); 
     
 
        // Timer, löst alle 1000 Millisekunden eine neue Anzeige des Panels aus 
        Timer t = new Timer(1000, 
            new ActionListener() { 
                public void actionPerformed(ActionEvent e) { 
                    panel.repaint(); 
                } 
            } 
        ); 
        t.start();  // Timer starten 
 
        // Fenster sichtbar machen... 
        frame.setVisible(true); 
    } 
}

 

[Volvagia]

Du musst im Timer den Text des JLabels aktualisieren.
btw. nimm besser die Berechnungen aus dem Zeichencode raus und halte ihm so schlank es möglich ist.

 

[java starter]

Danke für die schnelle Antwort...

Ja, das habe ich mir schon gedacht, nur war der Timer schon vorgegeben und ich weiß nicht, wie ich den Text des JLabels erneue, weil der Befehlt repaint wird wohl eher nicht funktionieren

Ach und zu den Berechnungen im Zeichencode: Kannst du mir sagen, von welcher Zeile du da sprichst?

LG

 

[Volvagia]

Mit #setText(String).
repaint wird funktionieren, nur da Strings immuntable sind wirst du keinen anderen Text bekommen.

Ich meinte besonders die Berechnungen unter "// Zeigerposition berechnen". Da der EDT bei jeder Kleinigkeit mit Events bombardiert wird muss man ihm nicht auch noch mit Gleitkommaberechnungen aufhalten. ^^ Ich würde dem Timer es zuerst berechnen lassen (ebenso die Zeit und sie gleichzeitig für Analog- und Digitaluhr verwenden), setzen und dann erst repaint aufrufen.

 

[java starter]

Perfekt, jetzt funktionierst, danke!!

Zwei Fragen habe ich noch;
Wenn du in den Quelltext schaust, hast du die Anzeige des JLabel, das mit ganz vielen Leerzeichen beginnt, dass ich die digitale Anzeige in die Mitte rücke, geht das mit EINFACHEN Mitteln auch anders?

Und bei der Anzeige des JLabels benuze ich die Zahlen stunden, minuten und sekunden, die sind jedoch in der Klasse ZeichenPanel definiert, also habe ich sie nochmals definiert, geht das auch eleganter?
Habs schon mit ZeichenPanel.stunden probiert, aber das geht iwie nicht!

LG

 

[Volvagia]

aendern.setHorizontalAlignment(SwingConstants.CENTER);

Du könntest ja z. B. setter dafür schreiben:

Timer t = new Timer(1000, new ActionListener() {
	public void actionPerformed(ActionEvent e) {
		recalculateClocks();
	}
});

private static void recalculateClocks() {
	long sekunden = (System.currentTimeMillis() / 1000) % 60;
	long minuten = (System.currentTimeMillis() / 1000) % 3600;
	long stunden = (System.currentTimeMillis()/1000) %(43200);
	
	String digitalText = (stunden / 3600) + ":" + (minuten / 60) + ":" + sekunden;
	
	double zeiger_x = Math.sin(sekunden/60.0 * 2*Math.PI);
	double zeiger_y = Math.cos(sekunden/60.0 * 2*Math.PI);
    
	double zeiger_x1 = Math.sin(minuten/3600.0 * 2*Math.PI);
	double zeiger_y1 = Math.cos(minuten/3600.0 * 2*Math.PI);
    
	double zeiger_x2 = Math.sin(stunden/43200.0 * 2*Math.PI);
	double zeiger_y2 = Math.cos(stunden/43200.0 * 2*Math.PI);
	
	panel.setSekundenZeigerSinus(zeiger_x);
	panel.setSekundenZeigerCoSinus(zeiger_y);
	
	panel.setMinutenZeigerSinus(zeiger_x1);
	panel.setMinutenZeigerCoSinus(zeiger_y1);
    
	panel.setStundenZeigerSinus(zeiger_x2);
	panel.setStundenZeigerCoSinus(zeiger_y2);
	
	panel.repaint();
	aendern.setText(digitalText);
}

 

[java starter]

Sieht für mich schon schwer aus, schau ich mir morgen auf jeden Fall in Ruhe an!

Vielen Dank!