Warum wird meine Graphik zwei Mal gezeichnet ?

[Kanitrino]

Hallo Experten,

In dem unten gelisteten Programm (das ich hier auf's Wesentliche abgemagert habe) geht es darum, drei Graphiken in einem Frame zu vereinigen :

1. Eine Simulation "Brett" (Hasen, die Gras fressen)
2. Eine Graphik "Population", die die Anzahl der Hasen im Laufe der Zeit darstellt
3. und eine Benutzeroberfläche "Benutzer"

Alle drei Graphiken sind in je einer Klasse JPanel-Klasse enthalten.

Außerdem gibt es eine main-Klasse "Hasen" und einen "Manager".

Problem : Das "Brett" und die "Population" werden je zwei Mal gezeichnet, und zwar in die beiden Hälften des "Panels" mit GridLayout (2,1).

Kann mir jemand sagen, warum ?

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

public class Hasen extends JFrame
{
	Manager manager;
	Brett brett;
	Population pop;
	Benutzer gui;
	int msec	= 100;
	Panel panel;
	Container container;

	public Hasen()
	{
		this.setSize(1000, 650);
		brett	= new Brett();
		container = getContentPane();
		pop		= new Population();
		gui		= new Benutzer();
		manager = new Manager(brett, pop, msec);
		setBackground(Color.yellow);
		panel	 = new Panel();
		panel.setLayout(new GridLayout(2, 1));
		panel.add(brett);
		panel.add(pop);
		container.add("Center", panel);
		container.add("East", gui);

		manager.start();
	}

	public static void main(String[] args)
	{
			Hasen hasen 	= new Hasen();
			hasen.setVisible(true);
	}
}

class Manager implements Runnable
{
	Brett brett;
	Population pop;
	Thread running;
	int msec;

	public Manager (Brett brett, Population pop, int msec)
	{
		this.brett 	= brett;
		this.pop		= pop;
		this.msec	= msec;
	}

	public void start()
	{	if (running == null)
		{	running = new Thread (this);
			running.start();
	}	}

	public void run()
	{
		int i=0;
		while(i<4)
		{
			i++;
			brett.malen();
			pop.malen();
			verzoegerung (msec);
		}
	}

	public void verzoegerung (int msec)
	{
		try {Thread.sleep (msec);}
		catch(Exception e){}
	}
}

class Brett extends JPanel
{
	public void malen()
	{
		repaint();
	}

	public void paint(Graphics g)
	{
		g.setColor(Color.pink);
		g.fillRect(100, 100, 300, 200);
	}
}

class Population extends JPanel
{
	public void malen()
	{
			repaint();
	}

	public void paint(Graphics g)
	{
		g.setColor(Color.green);
		g.fillRect(0, 20, 500, 100);
	}
}

class Benutzer extends JPanel
{
	JSlider slider1;
	JButton bStart;
	JLabel label;

	public Benutzer()
	{
		setLayout(new GridLayout(0, 1));
		label = new JLabel("  Benutzeroberfläche");
		bStart 	= new JButton("S T A R T");
		slider1 = new JSlider(JSlider.HORIZONTAL, 0, 4, 1);
		add(label);
		add(slider1);
		add(bStart);
	}
}

 

[Joker]

mh, weil du sie zweimal zeichnen lässt? Das aufrufen der von Population#malen und Brett#malen kannst du einsparen.

 

[Kanitrino]

Ja schon, aber es ist ja eigentlich eine Animation (bzw. zwei). Ich wollte es euch nur nicht zumuten, den gesamten Klumpatsch lesen zu müssen und habe daher alles zu Rechtecken "abgemagert". Animationen lässt man eben mehrfach zeichnen.

Aber was meinst Du genau ? irgendwie wird natürlich alles doppelt gezeichnet, aber warum ? hat es was mit dem "Graphics g"-Objekt zu tun ?

 

[André Uhres]

Was du "doppelt" siehst, sind einfach nur Unreinheiten weil der Hintergrund nicht gelöscht wird.
Statt paint musst du paintComponent benutzen, mit dem super Aufruf jeweils am Anfang:

   public void paintComponent(Graphics g) {
        super.paintComponent(g);
...
    }

 

[Kanitrino]

Hallo André,

Ich hab's ausprobiert & es funktioniert. Herzlichen Dank !

 

[Quaxli]

Ich hab' mal dran rumgefrickelt

Folgende Änderungen habe ich eingebaut:

- den super-Aufruf in paint-Component. Wie André ja schon angemerkt hat, war das die grundsätzliche Ursache
- Manger erbt jetzt von Thread und implementiert nicht mehr Runnable, spart etwas Code
- die malen-Methode habe ich aus Brett und Population rausgeworfen, da unnötig. Manager erhält jetzt ein Referenz
auf Hasen und macht einen repaint auf das komplette Fenster.


Was mir noch nicht ganz klar ist, ist der Sinn und Zweck des Codes, der in der run-Methode hinterlegt ist. Dieser Code ist nämlich sehr schnell abgearbeitet und dann wird keine repaint() mehr durchführt, so daß Veränderungen nicht mehr dargestellt werden, weil repaint() nicht mehr aufgerufen wird.

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

public class Hasen extends JFrame {

	private static final long serialVersionUID = 1L;

	Manager manager;
	Brett brett;
	Population pop;
	Benutzer gui;
	int msec = 100;
	Panel panel;

	Container container;

	public Hasen() {
		this.setSize(1000, 650);
		brett = new Brett();
		container = getContentPane();
		pop = new Population();
		gui = new Benutzer();
		manager = new Manager(brett, pop, msec, this);
		setBackground(Color.yellow);
		panel = new Panel();
		panel.setLayout(new GridLayout(2, 1));
		panel.add(brett);
		panel.add(pop);
		container.add("Center", panel);
		container.add("East", gui);

		manager.start();
	}

	public static void main(String[] args) {
		Hasen hasen = new Hasen();
		hasen.setVisible(true);
	}
}

class Manager extends Thread {
	Brett brett;
	Population pop;
	int msec;
	Hasen parent;

	public Manager(Brett brett, Population pop, int msec, Hasen parent) {
		this.brett = brett;
		this.pop = pop;
		this.msec = msec;
		this.parent = parent;
	}

	public void run() {
		int i = 0;
		while (i < 4) {
			//i++;       <<< 4 repaints und dann ist Schluß????
                       parent.repaint();
			verzoegerung(msec);
		}
	}

	public void verzoegerung(int msec) {
		try {
			Thread.sleep(msec);
		} catch (Exception e) {
		}
	}
}

class Brett extends JPanel {
	
	public void paintComponent(Graphics g) {
		super.paintComponent(g);
		g.setColor(Color.pink);
		g.fillRect(100, 100, 300, 200);
	}
}

class Population extends JPanel {

	public void paintComponent(Graphics g) {
		super.paintComponent(g);
		g.setColor(Color.green);
		g.fillRect(0, 20, 500, 100);
	}
}

class Benutzer extends JPanel {
	JSlider slider1;
	JButton bStart;
	JLabel label;

	public Benutzer() {
		setLayout(new GridLayout(0, 1));
		label = new JLabel("  Benutzeroberfläche");
		bStart = new JButton("S T A R T");
		slider1 = new JSlider(JSlider.HORIZONTAL, 0, 4, 1);
		add(label);
		add(slider1);
		add(bStart);
	}
}

 

[Kanitrino]

Hallo Quaxli,

Herzlichen Dank für Deine Antwort, ich werde es mir heute Abend genau angucken.

Der Grund für die kurze Schleife, die ja in der aktuellen Form eigentlich keinen Sinn hat, weil die Rechtecke nicht animiert sind : Ganz ohne Schleife trat das Problem nicht auf. Die Wiederholung an sich musste also Teil des Problems sein - was sich nun auch bestätigt hat.

Ich hätte da noch eine Zusatzfrage :
Eigentlich wäre die angemessene Darstellung für die zwei Animationen (= Rechtecke) ein BorderLayout, so dass die größere - pinke - Graphik ("Brett") den meisten Platz im "Center" einnimmt und die kleinere, grüne ("Population") darunter im "South". Wenn ich das aber so mache, drängt die größere die kleinere ganz aus dem Bild und lässt ihr nur noch einen viel zu schmalen Streifen übrig, obwohl sie garnicht den ganzen Platz bräuchte.

Wäre es nicht eigentlich die Aufgabe von BorderLayout, das besser zu organisieren ?

 

[André Uhres]

Bei BorderLayout.SOUTH muss man in der preferredSize die Höhe angeben. Beispiel:

class Population extends JPanel {
    private Dimension size = new Dimension(0, 120);
    public Dimension getPreferredSize() {
        return size;
    }

 

[Kanitrino]

André,

Danke für den Tipp. Layout-Manager können ganz schön anstrengend sein !

Quaxli,

Deine Modifikationen sind insbesondere deshalb für mich unerwartet, da die paint-Methode nicht in den Klassen angesprochen wird. in denen sie steht, sondern in der main-Klasse, wo sie nicht steht, nämlich in (Hasen) parent. repaint();

Ist das den eleganter als umgekehrt ? Hat man denn so immer noch alles im Griff, wenn man mit mehreren Klassen mit je einer paint-Anweisung arbeitet ?

 

[Quaxli]

Du sparst Dir jede Menge repaint-Aufrufe. Was soll daran also unelegant sein ?

 

[André Uhres]

Du sparst Dir jede Menge repaint-Aufrufe. Was soll daran also unelegant sein ?

Die Erklärung scheint plausibel zu sein.
Allerdings, wenn man "intelligent malen" wollte, d.h. nur den Teilbereich, der sich verändert hat,
dann könnte es sich wieder anders verhalten. Denn mit "parent.repaint();" wird immer
der ganze JFrame mit all seinen Kindern neu gemalt, was u.U. nicht performant genug sein könnte.

 

[Quaxli]

Was heißt "scheint plausibel"?

Ich behaupte, daß die meisten Anfänger kein Bedarf nach "intelligentem Malen" haben dürften. Zumal im ursprünglichen Programm auch nur die malen()-Methode hintereinander aufgerufen werden. Java ist durchaus mächtig genug, komplexere Zeichenmethoden ohne Probleme zu bewältigen. Und solange das ursprügliche Problem nicht darauf hinweist, daß der entsprechende Bedarf besteht (z. B. durch komplexe Berechnungen im Hintergrund), sollte man doch den Code so einfach wie möglich halten.

 

[André Uhres]

Was heißt "scheint plausibel"?

Ich meine damit einfach, daß deine Lösung die beste ist, wenn nix Komplexeres daraus entsteht.
Ich wollte nur darauf hinweisen, daß sie nicht unbedingt allgemein gültig ist, mehr nicht :wink: