Swing repaint()

[Blagg Dogg]

hi,

kurze frage (wahrscheinlich irgend nen sau blöder fehler^^). ich dachte immer repaint ruft paintComponent automatisch auf. ist dem nicht so?

mein code:

package de.sack.main;

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

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

public class ForestPanel extends JPanel implements Runnable {

	private static final long serialVersionUID = 1L;
	private int width, height;
	Drop d;
	JFrame f;
	Thread t;
	
	public ForestPanel(int w, int h){
		f = new JFrame();
		setPreferredSize(new Dimension(w,h));
		d = new Drop(this);
		f.add(this);
		f.pack();
		f.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		f.setVisible(true);
		t = new Thread(this);
		t.start();
	}
	
	@Override
	public void paintComponent(Graphics g){
		System.out.println("Repainted");
		super.paintComponent(g);
		d.draw(g);
	
	}
	
	public int getWidth() {
		return width;
	}


	public int getHeight() {
		return height;
	}


	@Override
	public void run() {
		// TODO Auto-generated method stub
		while(true){
			System.out.println("Thread runs");
			repaint();
			
			try {
				Thread.sleep(10);
			} catch (InterruptedException e) {
				// TODO Auto-generated catch block
				e.printStackTrace();
			}
		}
		
	}
	
	public static void main(String[] args){
		new ForestPanel(800, 200);
	}
	
	

}

warum wird paintComponent nicht aufgerufen? hab ich was übersehen?

danke schonmal

 

[Marco13]

repaint() ruft paintComponent() nicht sofort direkt auf. Es sagt nur, dass "so bald wie möglich" mal paintComponent aufgerufen werden soll. Und das würde bei dir auch aufgerufen werden. Im Moment gibt es für Swing aber keinen Grund, paintComponent() auf deinem Panel aufzurufen. Dein Panel hat nämlich eine Größe von 0x0 Pixeln.

Du hast getWidth und getHeight von 'Component' überschrieben. Benenne die Methoden (und am besten die dazugehörigen Variablen) anders. (BTW: Bei modernen IDEs kann man einstellen, dass er eine Warnung anzeigen soll, wenn man eine Methode überschreibt, aber NICHT "@Override" davorschreibt. Dann würde sowas nicht mehr so leicht passieren...)

 

[Blagg Dogg]

danke

also benutzt der intern getHeight und getWidth und hat statt den normalen meine verwendet, die noch null waren. verstehe ich das richtig?

 

[Marco13]

Ja. Allgemein aufpassen, es gibt auch nocht getX(), getY() .... und viele andere. Sieht man mit den entsprechenden Warning-Einstellungen.