Gezeichnetes Objekt verschwindet nachdem erneut gezeichnet wird

[DonBronson]

Hallo,

für ein aktuelles Projekt, will ich mehrere RoundRectangles zeichnen.
Ich habe bereits ein JFrame und eine Menüleiste, die super funktionieren. Wenn ich im Menü auf "Create Sphere" klicke, erscheint ein Dialog, wo man mit dem ColorChooser die Farbe auswählen kann. Dann muss man Namen, width&height und x-/y-Koordinaten angeben. Nachdem ich die Methode aufrufe, erhalte ich auch korrekterweise mein RoundedReactangle:



Wenn ich ein zweites erstellen will, klappt es auch, aber das erste verschwindet nun:



Wenn ich das Fenster mir der Maus resize, dann taucht das erste wieder auf und das zweite verschwindet.

[CODE lang="java" title="JFrame:"] public void createWindow() {
/** Frame */
this.setTitle("Diagram");
this.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
this.setSize(700,500);
this.setLocationRelativeTo(null);
this.setVisible(true);
}[/CODE]

[CODE lang="java" title="Die createSphere-Methode, welche durch das MenuItem aufgerufen wird:"]public void createSphere(String name, int width, int height, int x, int y, Color color) {
this.add(new Sphere(name, width, height, x, y, color));
this.validate();
}[/CODE]

[CODE lang="java" title="Meine Sphere-Klasse"]public class Sphere extends JPanel {

private String name;
private int width;
private int height;
private int xPos;
private int yPos;
private Color color;

public Sphere(String name, int width, int height, int xPos, int yPos, Color color) {
this.name = name;
this.width = width;
this.height = height;
this.xPos = xPos;
this.yPos = yPos;
this.color = color;
}

public void paintComponent (Graphics g) {
super.paintComponent(g);
g.drawRoundRect(this.xPos,this.yPos,this.width,this.height, 15, 15);
g.setColor(this.color);
g.fillRoundRect(this.xPos,this.yPos,this.width,this.height, 15, 15);
}[/CODE]

Ich will mit dem Programm mehrere Rechtecke zeichnen können. Ich bin seit gestern Nachmittag auf Fehlersuche, aber leider klappt für mich nichts.

 

[Robert Zenz]

Ich glaube, dass die Groesze von deinen Spheres nicht gleich der Panel Groesze ist, also die Panels welche du da erzeugst fuellen wahrscheinlich das gesamte oder zumindest teilweise das Eltern-Panel. Das kannst du verifizieren in dem du in der paint-Methode *alles* mit einer Farbe befuellst, dann siehst du wie grosz deine Sphere-Panels in Wirklichkeit sind.

Besser waere es wenn die Sphere-Panels direkt die richtige Groesze und Position in Eltern-Panel haben, und dann einfach nur in sich selbst zeichnen.

 

[DonBronson]

Ich glaube, dass die Groesze von deinen Spheres nicht gleich der Panel Groesze ist, also die Panels welche du da erzeugst fuellen wahrscheinlich das gesamte oder zumindest teilweise das Eltern-Panel. Das kannst du verifizieren in dem du in der paint-Methode *alles* mit einer Farbe befuellst, dann siehst du wie grosz deine Sphere-Panels in Wirklichkeit sind.

Besser waere es wenn die Sphere-Panels direkt die richtige Groesze und Position in Eltern-Panel haben, und dann einfach nur in sich selbst zeichnen.

Habe es grade ausprobiert und wenn ich dem JPanel eine BackgroundColor gebe, dann wird das ganze JFrame eingenommen.
edit: Danke für die schnelle Antwort, vllt. hilft das schon.. :/

 

[DonBronson]

Ich glaube, dass die Groesze von deinen Spheres nicht gleich der Panel Groesze ist, also die Panels welche du da erzeugst fuellen wahrscheinlich das gesamte oder zumindest teilweise das Eltern-Panel. Das kannst du verifizieren in dem du in der paint-Methode *alles* mit einer Farbe befuellst, dann siehst du wie grosz deine Sphere-Panels in Wirklichkeit sind.

Besser waere es wenn die Sphere-Panels direkt die richtige Groesze und Position in Eltern-Panel haben, und dann einfach nur in sich selbst zeichnen.

Ich habe es mal ausprobiert und im Konstruktor der Sphere wie folgt die BackgroundColor gesetzt:

this.setBackground(Color.black);

Nun wollte ich direkt die Größe korrekt setzen mit:

this.setBounds(xPos, yPos, width, height);

Das funktioniert allerdings nicht. Wenn ich eine Sphere erstelle, nimmt das JPanel der Sphere trotzdem das ganze JFrame ein.

 

[Robert Zenz]

Das sollte daran liegen dass du dem Eltern-Panel erst den layoutManager wegnehmen musst, damit du dann die Positionen selbst bestimmen kannst.

 

[DonBronson]

Das sollte daran liegen dass du dem Eltern-Panel erst den layoutManager wegnehmen musst, damit du dann die Positionen selbst bestimmen kannst.

Habe ich, jetzt wird allerdings gar nichts mehr gezeichnet 😵

edit: es wird doch etwas gezeichnet, aber erst, wenn ich das fenster resize + es wird allerdings nur so halb dargestellt 😀:

 

[DonBronson]

So habe nun nochmal rumprobiert. Also es wird immer noch erst nach resize mit der Maus sichtbar (warum auch immer?)
Aber ich habe der setBounds()-Methode nicht wie vorher einfach die height und width übergeben, sondern manuell eingegeben.
[CODE lang="java" highlight="9"]public Sphere(String name, int width, int height, int xPos, int yPos, Color color) {
this.name = name;
this.width = width;
this.height = height;
this.xPos = xPos;
this.yPos = yPos;
this.color = color;
this.setBackground(Color.black);
this.setBounds(xPos,yPos,200,200);
}[/CODE]

Nun wird es komplett dargestellt, aber natürlich immer noch zu groß:



Ich verstehe allerdings nicht, warum sich die Größen unterscheiden. Warum ist die width&height, die ich übergeben habe kleiner als beim Zeichnen des Rechtecks. Ich mein 100x100 müsste doch gleich sein, egal, ob JPanel oder Rectangle?!

 

[DonBronson]

Okay, das habe ich nun auch gelöst. Das JPanel bekommt xPos, yPos und width&height übergeben. Die Position des Rectangles hängt nun aber vom JPanel ab und nicht vom JFrame, weswegen ich hier nun statt xPos und yPos einfach zweimal 0 übergebe, damit das Rectangle "oben-links" vom JPanel gezeichnet wird.

[CODE lang="java" highlight="17, 22, 24"]public class Sphere extends JPanel {

private String name;
private int width;
private int height;
private int xPos;
private int yPos;
private Color color;

public Sphere(String name, int width, int height, int xPos, int yPos, Color color) {
this.name = name;
this.width = width;
this.height = height;
this.xPos = xPos;
this.yPos = yPos;
this.color = color;
this.setBounds(xPos,yPos,width,height);
}

public void paintComponent (Graphics g) {
super.paintComponent(g);
g.drawRoundRect(0,0,this.width,this.height, 15, 15);
g.setColor(this.color);
g.fillRoundRect(0,0,this.width,this.height, 15, 15);
}
}[/CODE]

Ich kann jetzt sogar mehrere Spheres zeichnen:



Allerdings besteht immer noch das Problem, dass alles, was gezeichnet wird, erst angezeigt wird, sobald ich das Fenster mit der Maus resize.


Edit: Das Problem mit dem resize wurde durch die Methode repaint() behoben:

[CODE lang="java" highlight="4"] public void createSphere(String name, int width, int height, int x, int y, Color color) {
this.add(new Sphere(name, width, height, x, y, color));
validate();
repaint();
}[/CODE]

 

[Blender3D]

Das JPanel bekommt xPos, yPos und width&height übergeben.

Du könntest statt eines JPanels für die Grafiken eigene Klassen machen und dies in einem JPanel zeichnen lassen.
[CODE lang="java" title="StartGraphicUtil" highlight="7"]import javax.swing.JFrame;

public class StartGraphicUtil {
public static void main(String[] args) {
JFrame frame = new JFrame("Grafik Objekte ");
frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
frame.add(new GraphicObjectPanel(500, 500));
frame.setResizable(false);
frame.pack();
frame.setVisible(true);
}
}[/CODE]


[CODE lang="java" title="GraphicObjectPanel" highlight="25-30,9"]import java.awt.Color;
import java.awt.Dimension;
import java.awt.Graphics;
import java.util.Vector;
import javax.swing.JPanel;

@SuppressWarnings("serial")
public class GraphicObjectPanel extends JPanel {
private Vector<Object2D> shapes = new Vector<Object2D>();

public GraphicObjectPanel(int width, int height) {
setPreferredSize(new Dimension(width, height));
Oval oval = new Oval("Das ist rund!", 100, 100);
oval.setPosition(100, 50);
oval.setBackColor(Color.CYAN);
RoundRect rect = new RoundRect("Das ist beinahe eckig", 80, 100);
rect.setBackColor(Color.RED);
rect.setPosition(300, 150);
shapes.add(oval);
shapes.add(rect);
setBackground(Color.DARK_GRAY);
}

@Override
protected void paintComponent(Graphics g) {
super.paintComponent(g);
for (Object2D s : shapes) {
s.draw(g);
}
}
}[/CODE]
[CODE lang="java" title="RoundRect"]import java.awt.Graphics;

public class RoundRect extends Object2D {

public RoundRect(String name, int width, int height) {
super(name, width, height);
}

@Override
protected void drawObject(Graphics g) {
g.setColor(backColor);
g.fillRoundRect(x, y, width, height, width / 5, width / 5);
}
}[/CODE]
[CODE lang="java" title="Oval"]import java.awt.Graphics;

public class Oval extends Object2D {
public Oval(String name, int width, int height) {
super(name, width, height);
}

@Override
protected void drawObject(Graphics g) {
g.setColor(backColor);
g.fillOval(x, y, width, height);
}
}[/CODE]



[CODE lang="java" title="Object2D"]import java.awt.Color;
import java.awt.Graphics;
import java.awt.Point;
import java.awt.Rectangle;

public abstract class Object2D {
protected String name;
protected int x;
protected int y;
protected int width;
protected int height;
protected Color backColor = Color.WHITE;
protected Color frameColor = Color.LIGHT_GRAY;
private boolean visible = true;

public Object2D(String name, int width, int height) {
this.name = name;
this.width = width;
this.height = height;
}

public final void draw(Graphics g) {
if (!visible)
return;
drawObject(g);
};

protected abstract void drawObject(Graphics g);

public boolean isVisible() {
return visible;
}

public Rectangle getBoundingBox() {
return new Rectangle(x, y, width, height);
}

public Point getCenter() {
return new Point(x + width / 2, y + height / 2);
}

public void setBackColor(Color backColor) {
this.backColor = backColor;
}

public void setFrameColor(Color frameColor) {
this.frameColor = frameColor;
}

public void setPosition(int x, int y) {
this.x = x;
this.y = y;
}

public void setVisible(boolean ok) {
this.visible = ok;
}

@Override
public String toString() {
return "[" + name + "| " + x + "," + y + "]";
}
}[/CODE]