Abgeleitete JComponent wird nicht gezeichnet

[AllBetterThanS7]

Hallo,

Ich habe ein Problem mit einer Klasse, die ich von JComponent abgeleitet habe. Diese enthält eigentlich nur den Konstruktor, der den Konstruktor von JComponent Aufruft und die Überschriebene Paint-Methode die (eigentlich) einen String auf dem Übergebenen Graphics Objekt zeichnen soll.

Aufgerufen wird diese Klasse von einer Von JPanel Abgeleiteten Klasse, da diese Später einiges beinhalten sollte. Diese hat wiederum nur den Konstruktor (Aufruf von

super()

) und die Methode paint. die mit

paintComponents()

die Komponenten Zeichnen soll.

Aufgerufen wird alles in einer Klasse Main mit einem dynamischen JFrame Objekt:

Mypanel p = new UMLPanel();
p.add(new ClassD(100,100));
JFrame frame = new JFrame("Test");
//frame.add(new JPanel().add(new ClassD(100,100)));
frame.add(p);
frame.setSize(300, 300);
frame.setVisible(true);
frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
frame.repaint();

Nun zum Problem: Die Zeichenmethode von ClassD wird aufgerufen (Ausgabe eines Strings auf der Konsole), jedoch wird nichts gezeichnet. Der String aus der Zeichenmethode von Mypanel allerdings wird gezeichnet. Wenn ich jedoch die auskommentierte Zeile anstatt der Darunter Ausführe, also über das Standard JPanel gehe, funktioniert alles wunderbar. Der Fehler muss also in Mypanel liegen:

public class Mypanel extends JPanel
{
	
	public Mypanel()
	{
		super();
	}

	public void paint(Graphics g)
	{
		System.out.println("Panel wird gezeichnet");
		this.paintComponents(g); //ruft entsprechende Methoden auf, zeichnet jedoch nichts
		g.drawString("Von UMLPanel", 0, 10); //wird gezeichnet
	}
}

Ist wahrscheinlich eine ziemlich einfache Anfängerfrage, jedoch hab ich keine Ahnung woran das liegen könnte. Danke schonmal für eure Hilfe

lg,
AllBetterThanS7

 

[bERt0r]

Du solltest nicht paint, sondern immer nur paintComponent überschreiben.

 

[Michael...]

Einen Konstruktor mit nur

super()

Aufruf, kann man sich auch sparen. Der Konstruktor der Super Klasse wird immer aufgerufen.
Bei Swing Komponenten überschreibt man die

paintComponent(Graphic)

nicht die paint, erster Aufruf darin ist üblicherweise

super.paintComponent(Graphic)

- dabei nicht mit paintComponents verwechseln.

Bei "selbstgezeichneten" Komponenten muss man u.U. die PreferredSize setzen, da einige LayoutManager sich daran bei der Darstellung orientieren.

 

[SlaterB]

man könnte sich auch schlicht an wohl jedes Tutorial dafür halten, etwa
Galileo Computing :: Java ist auch eine Insel - 14 Einführung in grafische Oberflächen

ab 14.9.3 für Swing (DrawPanel)

 

[AllBetterThanS7]

Danke erstmal für die schnellen Antworten
Also: habe nun die funktionalität meiner paint() Methode einfach in die paintComponent() Methode pgeschrieben, ändert nichts an der Funktionalität (PaintComponent() der JComponent wird aufgerufen, allerdings nichts gezeichnet)

Das Angegebene Tutorial zeigt nur, wie ich direkt auf das Panel Zeichne, nicht jedoch eine JComponent.

Leider bin ich im Moment noch nicht weitergekommen, werde das nachher nochmals angehen.

Über weitere Tipps wäre ich sehr dankbar.

 

[SlaterB]

wieso willst du denn von JComponent erben?
im ersten Posting ging es doch auch um Mypanel, UMLPanel usw.,

JComponent sollte wohl genauso gehen,
poste ein vollständiges Codebeispiel

 

[AllBetterThanS7]

Also nochmal, habe mich vielleicht undeutlich ausgedrückt:

Ich habe eine Klasse Main, die ein Standard JFrame erzeugt, auf dem ich MEIN Panel zeichne. Dieses Panel beinhaltet mehrere Drawable Objekte, abgeleitet von JComponent, damit ich sie einfach mittels Panle.add( ... ) hinzufügen kann.

Nun meine 3 Klassen:

Klasse Main:
[Java]
package main;

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

public class Main
{
public static void main (String args[])
{
MyPanel p = new MyPanel();
p.add(new Drawable(100,100));
JFrame frame = new JFrame("Test");
//frame.add(new JPanel().add(new Drawable(100,100)));
frame.add(p);
frame.setSize(300, 300);
frame.setVisible(true);
frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
frame.repaint();
}
}
[/Java]
Klasse MyPanel:
[Java]
package main;

import java.awt.Graphics;
import javax.swing.JPanel;

@SuppressWarnings("serial")
public class MyPanel extends JPanel
{
public MyPanel()
{
//super();
}

public void paintComponent(Graphics g)
{
super.paintComponent(g);
System.out.println("UMLPanel wird gezeichnet");
paintComponents(g);
g.drawString("Von UMLPanel", 0, 10);
}

}
[/Java]
Klasse Drawable:
[Java]
package main;

import java.awt.Graphics;
import javax.swing.JComponent;

@SuppressWarnings("serial")
public class Drawable extends JComponent
{

protected double startX;
protected double startY;
protected double endX;
protected double endY;

public Drawable(double startX, double startY)
{
this.startX = startX;
this.startY = startY;
this.setSize(100, 100);
}

public void paintComponent(Graphics g)
{
System.out.println("Drawable wird gezeichnet");
g.drawString("Noch nicht implementiert", (int)startX, (int)startY);
}

}
[/Java]

Wenn ich nun [Java]frame.add(p);[/code] auskommentiere und dafür [Java]frame.add(new JPanel().add(new Drawable(100,100)));[/Java] "ausführbar" mache (Klasse main) funktioniert alles tadellos. Also muss es an meiner Klasse MyPanel liegen.

Hoffe dadurch wurden alle Unklarheiten beseitig.

lg

 

[bERt0r]

Lass mal den Aufruf von paintComponents() weg.

 

[AllBetterThanS7]

Ändert nichts. Zeichenfunktion wird aufgerufen, zeichnet aber nichts.
Wenn ich auf der Konsole die Größe der Komponente Ausgeben lasse mit getHeight() und getWidth() werden mir unterschiedliche Werte ausgegeben. Mit Standardpanel bei beidem "262", und mit eigenem Panel bei beiden "1". Klar dass der String dann nicht sichtbar ist. Nur wo liegt der Fehler?

 

[SlaterB]

> frame.add(new JPanel().add(new Drawable(100,100)));

der Rückgabewert von add() ist die zu addende Komponente selber, das entspricht also
> frame.add(new Drawable(100,100));

das ContentPane des JFrames hat BorderLayout, die einzige eingefügte Komponente wird maximiert dargestellt,
dort ist alles zu sehen

-------

wenn du erst in ein JPanel einfügst, dann gilt das Standard-FlowLayout dort,
wie die meisten Layouts ist dort setPreferredSize() wichtig, das Layout liest die gewüschte Größe und erstellt daraus die tatsächliche Größe,

setSize() selber ist leider meist wirkungslos, bei BorderLayout eh egal, nur bei null-Layout relevant,
verwirrende Verwechslung in Swing, aber es gibt nunmal die Layouts, die die wirkliche Größe kontrollieren,

-------

wenn du erst ab Position 100,100 malst, ist es ungünstig, nur eine Breite/ Höhe von 100,100 vorzugeben,
da wird dann genau außerhalb des sichtbaren Bereichs begonnen

fange mit

		setPreferredSize(new Dimension(200, 200));
		setBorder(BorderFactory.createLineBorder(Color.BLUE));

in Drawable an, bzw. zeichne sicherlich lieber ab 0,0

 

[AllBetterThanS7]

Vielen Dank für die Ausführliche Antwort! Fehler behoben.
Auch das mit dem Rand farbig machen ist ne gute Idee um Fehler zu lösen, danke.