Java - Problem med Thread (klocka)

Du kör aldrig din start metod.

I en JFrame ska du överlagra paintComponent, inte paint

edit: korrigering: du bör överlagra paintComponent på den JComponent som används som contentPane i din JFrame. Dessutom bör du anropa super.paintComponent det första du gör. Det kan räcka för att komponenten ska rengöra sig mellan paints.

Annars kan du köra med.

g.clearRect(0, 0, 200, 200);

Jag menar såhär:

import java.awt.Color;
import java.awt.Dimension;
import java.awt.Graphics;

import javax.swing.JFrame;
import javax.swing.JPanel;

public class TimePanel extends JPanel implements Runnable {
	private static final int WIDTH = 200;
	private static final int HEIGHT = 40;
	
	private int h = 18;
	private int m = 46;
	private int s = 2;
		
	public TimePanel() {
		this.setPreferredSize(new Dimension(WIDTH, HEIGHT));
		this.setBackground(Color.BLACK);
		this.setForeground(Color.GREEN); // matrixy!
		new Thread(this).start();
	}
	
	@Override
	protected void paintComponent(Graphics g) {
		super.paintComponent(g); // viktig
		String time = h + ":" + m + ":" + s;
		g.drawString(time, 20, 20);
	}
	
	@Override
	public void run() {
		while(true) {
			repaint();

			try {
				Thread.sleep(1000);
			} catch (InterruptedException e) {
				e.printStackTrace();
			}

			s++;
			m = m + s / 60;
			h = h + m / 60;
			s = s % 60;
			m = m % 60;
			h = h % 24;			
		}
	}
	
	public static void main(String[] args) {
		JFrame window = new JFrame("Timer");
		window.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		window.setContentPane(new TimePanel());
		window.pack();
		window.setVisible(true);	
	}

}

edit: förresten, klockan kommer att gå fel. Detta beror på att Thread.sleep(1000) sover i ungefär 1000 millisekunder (minst). Det handlar om millisekunder till en början men efter tusen sådana har klockan halkat en sekund efter jämfört med systemtiden..

edit 2: glömde naturligtvis att argumentera för min sak
Faktum är att det inte är nödvändigt att överlagra paintComponent. Man kan, mycket riktigt, köra över paint rätt och slätt, men det brukar inte rekommenderas.

Varför?
En Swing-komponent målas i tre omgångar: Först målas komponenten (paintComponent()), sedan komponentens bård (paintBorder()), och till sist dess barn (paintChildren()). Du kan läsa mer om detta här: http://java.sun.com/products/jfc/tsc/articles/painting/#swing

Varför vill vi anropa super.paintComponent() från vår överlagrade paintComponent?
JComponent, som är JPanels superklass, standardiserar vissa attribut mellan de olika komponenterna i Swing-ramverket. Genom att anropa super.paintComponent() först av allt låter vi vår komponent måla sig enligt den fördefinierade standarden; detta innebär att komponenten fylls med sin satta bakgrundsfärg (om inte setOpaque(false) har anropats på den). Kort och gott låter man sin nya komponent uppföra sig (eller åtminstone måla om sig) som en vanlig JComponent.