por que sempre dá o mesmo tempo?

Question

Usei o seguinte programa para obter o relógio. O que acabou por ser um relógio estático ?Por que é assim?

import java.util.*;
import java.awt.*;
import javax.swing.*;

class tester {

    JFrame fr = new JFrame();
    JPanel p = new JPanel();
    JLabel l = new JLabel();
    Date d = new Date();
    GregorianCalendar gc = new GregorianCalendar();

    tester() {
        p.setBackground(Color.red);
        l.setVisible(true);
        p.add(l);
        fr.add(p);
        fr.setSize(200,200);
        fr.setVisible(true);
        startClockThread();
    }

    public void startClockThread() {
        Runnable r = new Runnable() {
            @Override
            public void run() {
                startClock();
            }
        };
        new Thread(r).start();
    }

    public void startClock() {
        l.setVisible(true);  
        while(true) {
            l.setText(gc.get(gc.HOUR) + ":" + gc.get(gc.MINUTE) + ":" + gc.get(gc.SECOND));
            System.out.println(gc.get(gc.SECOND));
        }
    }

    public static void main(String args[]) {
        new tester();
    } 
} 

Answer 1

GregorianCalendar() Constrói um GregorianCalendar padrão usando a hora atual no fuso horário padrão com o local padrão. Java Doc

Você pode fazer assim.

while(true) {
       GregorianCalendar gc = new GregorianCalendar();
   l.setText(gc.get(gc.HOUR) + ":" + gc.get(gc.MINUTE) + ":" + gc.get(gc.SECOND));
}

Agora você deve entender por que está obtendo um relógio estático!

Answer 2

Você cria o GregorianCalendar apenas uma vez e ele nunca é atualizado.Portanto, a data é sempre a mesma.

Answer 3

existem grandes problemas além do que você identificou:

• não deixe os tópicos correrem soltos, eles congelarão a interface do usuário eventualmente
• todo e qualquer acesso a um componente Swing deve acontecer no EDT

Você pode resolver ambos da maneira mais fácil usando um javax.swing.Timer

ActionListener nextSecond = new ActionListener() {
     @Override
     public void actionPerformed(ActionEvent e) {
         // get time ...
         timeLabel.setText(...);
     }
}
new Timer(1000, nextSecond).start();