Swing JFrame und JPanel Problematik

[htz]

Hallo,

ich habe eine Aufgabe, in welcher ein einfaches Programm geschrieben werden soll, welches aus einem JFrame und einem JPanel besteht. Auf dem JPanel soll eine Ente gezeichnet werden. Die Position der Ente wird 50 mal zufällig ermittelt und im Abstand von einer Sekunde soll diese dann neu gezeichnet werden.

Das Programm ist denkbar einfach und auch kein Problem. Allerdings habe ich eine Lösung zu dieser Aufgabe erhalten, welche einen Fehler enthält, den ich nicht nachvollziehen kann. Hier mal der Quellcode des JFrames:

public class Fenster {
	public static void main(String[] args) {
		JFrame f =new JFrame ("Beste Ente ");
		f.setSize(800,600);
		f.setVisible(true);
		f.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		PaintIcon p = new PaintIcon();
		try {
			BufferedImage image = ImageIO.read(new File("ente.gif"));
			f.add(p);
			p.setBild(image);
			p.setBorder(new LineBorder(Color.black));
		} catch (IOException e) {
			e.printStackTrace();
		}
		Random a = new Random();
		for (int i = 0; i < 50; i++) {
			System.out.println("Breite"+p.getWidth());
			System.out.println("Hoehe"+p.getHeight());
//			int x = a.nextInt(p.getWidth()-80);
//			int y = a.nextInt(p.getHeight()-60);
			int x = a.nextInt(800-80);
			int y = a.nextInt(600-60);
			System.out.println("X: "+x+", Y: "+y);
			p.setX(x);
			p.setY(y);
			f.repaint();
			try{
				Thread.sleep(1000);
			}catch(InterruptedException e){
			}
		}
	}
}

und hier das JPanel:

public class PaintIcon extends JPanel {
	int x = 0, y = 0;

	public int getX() {
		return x;
	}

	public void setX(int a) {
		this.x = a;
	}

	public int getY() {
		return y;
	}

	public void setY(int b) {
		this.y = b;
	}

	private BufferedImage bild;

	public void setBild(BufferedImage bild) {
		this.bild = bild;
		//repaint();

	}

	@Override
	protected void paintComponent(Graphics g) {
		super.paintComponents(g);
		g.drawImage(bild, x, y, 80, 60, this);

	}
}

Anstatt das die Ente nun 50 mal innerhalb der Fenstergröße neu gezeichnet wird, wird das komplette JPanel an anderen Stellen gezeichnet, was man anhand der Border sehen kann. Ich hatte gedacht, dass das an der setX() und der setY() Methode liegt, da diese evtl. bestehende Methoden überschreiben. Allerdings sind diese Methoden in der API zum JPanel nicht dokumentiert. Warum verschiebt sich also nun das JPanel im JFrame?

p.s.: die ente.gif ist im Anhang

Gruß
htz

 

[KrokoDiehl]

Ja dein Problem ist, dass

getX()

und

getY()

am PaintIcon-Panel die JComponent-Methoden überschreiben. Lass sie doch einfach weg, die werden eh nicht benutzt. Ansonsten musst du sie umbennen, z.B. in getBildX().

 

[njans]

Mach doch einfach eine Methode: setIconPosition(int x, int y) und das könntest du dann in einem Point objekt speichern

Noch etwas: So, wie du es momentan machst, wird die Gui auf nichts reagieren, wenn du deine 50 mal zeichnest. Solche Thread.sleep() aufrufe sollten nicht auf dem EDT passieren sondern in einem eigenen Thread. Ist in deinem Fall vielleicht etwas unwichtig, aber trotzdem solltest du das wissen.

 

[SlaterB]

> So, wie du es momentan machst, wird die Gui auf nichts reagieren, wenn du deine 50 mal zeichnest.

wenn, dann wird ja auch nicht 50x gezeichnet, sondern nur das letzte Bild am Ende

allerdings befindet sich dieser Code in der main-Methode, nicht in einem Listener,
da funktioniert das ausnahmsweise noch, main ist ein separater Thread,

allgemein natürlich wichtiger Hinweis, wie hier in gefühlt 50% aller Themen zu GUIs das Problem

 

[njans]

wenn, dann wird ja auch nicht 50x gezeichnet, sondern nur das letzte Bild am Ende

Hmm das sehe ich nicht. Erleuchte mich bitte.

allerdings befindet sich dieser Code in der main-Methode, nicht in einem Listener, da funktioniert das ausnahmsweise noch, main ist ein separater Thread

Da hätte ich mal genauer hinschauen sollen. Oh blind me!

 

[SlaterB]

wenn der AWT-Thread blockiert ist, dann reagiert er die ganze Zeit nicht auf Ereignisse, und genausowenig auf repaint(),
das ist doch alllgemein bekannt dass man dann nur den letzten Stand zu sehen bekommt, die Wartezeit passiert nichts

 

[htz]

Ja dein Problem ist, dass

getX()

und

getY()

am PaintIcon-Panel die JComponent-Methoden überschreiben. Lass sie doch einfach weg, die werden eh nicht benutzt. Ansonsten musst du sie umbennen, z.B. in getBildX().

Die Lösung ist von einem Studenten, nicht von mir. Ich habe das Problem auch genau so gelöst, was mich jedoch irritiert ist, dass laut Dokumentation JComponent keine setX bzw setY Methoden besitzt. Und wenn diese private wären und nicht in der Doku stehen, dann müsste es den Compiler ja stören, dass ich die überschriebenen Methoden public mache, oder nicht?

Das Thread.sleep() wird hier von den Studierenden einfach wie beschrieben verwendet, da sie zu dem Zeitpunkt noch keine Threads kennen gelernt haben.

 

[htz]

Soeben ist der Groschen gefallen, als ich deine Antwort nochmal gelesen habe. Es geht gar nicht um setX und setY, sondern um die get-Methoden. Und die werden in beim neuzeichnen des Panels aufgerufen, oder? Danke für den Hinweis

 

[njans]

wenn der AWT-Thread blockiert ist, dann reagiert er die ganze Zeit nicht auf Ereignisse, und genausowenig auf repaint(),
das ist doch alllgemein bekannt dass man dann nur den letzten Stand zu sehen bekommt, die Wartezeit passiert nichts

Absolut, anscheinend reden wir aneinenader vorbei. Was ich, aus deiner ersten Aussage verstanden habe, war, dass du meinst, dass die Ente nur einmal gezeichnet wird und nicht 50 mal. Das ist allerdings nicht wahr, sie wird 50 mal gezeichnet, da der repaint vor dem sleep aufgerufen wird.

Mir ist schon klar, dass der AWT-Thread nicht reagiert, wenn er blockiert wird

public static class PaintIcon extends JPanel {

            private BufferedImage bild;
	    private int x = 0, y = 0;

	    public void setImageLocation(int x, int y) {
	        this.x = x; this.y = y;
	    }
	 
	    public int[] getImageLocation() {
	        return new int[] {x,y}; 
	    }
	 
	    public void setBild(BufferedImage bild) {
	        this.bild = bild;
	    }
	 
	    @Override
	    protected void paintComponent(Graphics g) {
	        super.paintComponents(g);
	        g.drawImage(bild, x, y, 80, 60, this);
	 
	    }
	}

 

[SlaterB]

und ich sage weiter dass solange Blockade besteht vielleicht unsauberer getGraphics()-Zugriff funktioniert,
aber repaints NICHT ausgeführt werden,

siehe z.B.
http://www.java-forum.org/awt-swing-swt/99978-actionlistener-verhindert-repaint.html

DAS PROBLEM:
Der Thread des WaitIcon wird zwar fortgesetzt, jedoch ist, solange der Main-Thread schläft, keine Animation zu sehen. Das JLabel bzw das JFrame wird erst nach Abarbeitung des sleep() Befehls aktualisiert. Es ist also keine Animation zusehen wie in dem Beispiel der Main-Klasse.

und ähnliche Threads, wobei ich doch leichte Probleme hatte einen zu finden und dieser hier ist auch extra lang,
ein Testprogramm mit paintComponent mag ich nicht zusammenstellen, kannst du ja selber zusammenstellen,
falls du nicht von deiner Position ebenso felsenfest überzeugt bist (oder ich sie weiterhin falsch verstehe)

wir können es dabei auch belassen, nocheinmal mit demselben Inhalt werde ich kaum antworten, die Rekursion muss abgebrochen werden

> da der repaint vor dem sleep aufgerufen wird.

vor oder nach ist in einer Schleife von 50 sleeps recht egal,
die ganzen 50 sec würde nichts passieren, falls in einem Listener ausgeführt


> Mir ist schon klar, dass der AWT-Thread nicht reagiert, wenn er blockiert wird

ich habe natürlich
> wenn, dann wird ja auch nicht 50x gezeichnet, sondern nur das letzte Bild am Ende
am Anfang geschrieben, nur wenn eine Blockade vorliegt,
dass in diesem Programm alles bestens läuft wegen main-Thread, das ist klar


edit:
ah, aktuell doch schon wieder ein Thread
http://www.java-forum.org/awt-swing...tor-funktioniert-nur-beim-initialisieren.html