ein Gesicht zeichnen

[kulturfenster]

Hallo,

ich würde gerne ein Gesicht zeichnen, bekomme aber leider keine Zeichnung hin, dh es erscheint nur ein leeres Frame.

Sieht jemand den Fehler? Vielen Dank für Tipps!

import javax.swing.JComponent;
import java.awt.Graphics;
import java.awt.Graphics2D;
import java.awt.geom.Ellipse2D;
import java.awt.geom.Line2D;
import javax.swing.JFrame;

public class Face extends JComponent {

	private Ellipse2D.Double kopf;
	private Ellipse2D.Double linkesAuge;
	private Ellipse2D.Double rechtesAuge;
	private Line2D.Double fresse;
	
	public Face()
	{
		kopf = new Ellipse2D.Double(50,50,100,100);
		linkesAuge = new Ellipse2D.Double(60,75,10,10);
		rechtesAuge = new Ellipse2D.Double(80,75,10,10);
		fresse = new Line2D.Double(60,90,85,90);
	}
	
	public void draw(Graphics g)
	{
		Graphics2D g2 = (Graphics2D) g;
		
		g2.draw(kopf);
		g2.draw(rechtesAuge);
		g2.draw(linkesAuge);
		g2.draw(fresse);
	}
	
	public static void main(String[] args)
	{
		JFrame frame = new JFrame("Face");
		
		frame.setSize(500,500);
		frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		
		Face component = new Face();
		frame.add(component);
		frame.setVisible(true);
	}
}

 

[André Uhres]

...
public class Face extends JComponent {
...
    protected void paintComponent(Graphics g) {
        super.paintComponent(g);
        draw(g);
    }
}

 

[Marco13]

... aber bitte sowas NICHT von JComponent erben lassen! (Sofern das wirklich nur gezeichnet werden soll)

 

[kulturfenster]

alles klar, vielen Dank!

hmm, aber wieso so kompliziert?

was ist schlecht an folgendem?:

public void paintComponent(Graphics g)
	{
		Graphics2D g2 = (Graphics2D) g;
		
		g2.draw(kopf);
		g2.draw(rechtesAuge);
		g2.draw(linkesAuge);
		g2.draw(fresse);
	}

 

[kulturfenster]

aber bitte sowas NICHT von JComponent erben lassen!

wieso nicht?

 

[Marco13]

Deswegen:

    private boolean isAlignmentXSet;
    private float alignmentX;
    private boolean isAlignmentYSet;
    private float alignmentY;
    protected transient ComponentUI ui;
    protected EventListenerList listenerList = new EventListenerList();
    private transient ArrayTable clientProperties;
    private VetoableChangeSupport vetoableChangeSupport;
    private boolean autoscrolls;
    private Border border;
    private int flags;
    private InputVerifier inputVerifier = null;
    private boolean verifyInputWhenFocusTarget = true;
    transient Component         paintingChild;
    private JPopupMenu popupMenu;
    private InputMap focusInputMap;
    private InputMap ancestorInputMap;
    private ComponentInputMap windowInputMap;
    private ActionMap actionMap;
    private boolean aaText;
    int ncomponents;
    Component component[] = new Component[0];
    LayoutManager layoutMgr;
    private LightweightDispatcher dispatcher;
    private transient FocusTraversalPolicy focusTraversalPolicy;
    private boolean focusCycleRoot = false;
    private boolean focusTraversalPolicyProvider;
    private transient Set printingThreads;
    private transient boolean printing = false;
    transient ContainerListener containerListener;
    transient int listeningChildren;
    transient int listeningBoundsChildren;
    transient int descendantsCount;
    transient ComponentPeer peer;
    transient Container parent;
    transient AppContext appContext;
    int x;
    int y;
    int width;
    int height;
    Color       foreground;
    Color       background;
    Font        font;
    Font        peerFont;
    Cursor      cursor;
    Locale      locale;
    transient GraphicsConfiguration graphicsConfig = null;
    transient BufferStrategy bufferStrategy = null;
    boolean ignoreRepaint = false;
    boolean visible = true;
    boolean enabled = true;
    boolean valid = false;
    DropTarget dropTarget;
    Vector popups;
    private String name;
    private boolean nameExplicitlySet = false;
    private boolean focusable = true;
    private int isFocusTraversableOverridden = FOCUS_TRAVERSABLE_UNKNOWN;
    Set[] focusTraversalKeys;
    private boolean focusTraversalKeysEnabled = true;
    Dimension minSize;
    boolean minSizeSet;
    Dimension prefSize;
    boolean prefSizeSet;
    Dimension maxSize;
    boolean maxSizeSet;
    transient ComponentOrientation componentOrientation= ComponentOrientation.UNKNOWN;
    boolean newEventsOnly = false;
    transient ComponentListener componentListener;
    transient FocusListener focusListener;
    transient HierarchyListener hierarchyListener;
    transient HierarchyBoundsListener hierarchyBoundsListener;
    transient KeyListener keyListener;
    transient MouseListener mouseListener;
    transient MouseMotionListener mouseMotionListener;
    transient MouseWheelListener mouseWheelListener;
    transient InputMethodListener inputMethodListener;
    transient RuntimeException windowClosingException = null;
    long eventMask = AWTEvent.INPUT_METHODS_ENABLED_MASK;
    private static final DebugHelper dbg = DebugHelper.create(Component.class);
    private PropertyChangeSupport changeSupport;
    boolean isPacked = false;

Das sind die Fields, die man erbt, wenn man von JComponent ableitet. Von den 100ten Methoden mal ganz abgsehen (die stören ja nicht). Dafür, dass du nur 3 Ellipsen und eine Linie zeichnen willst, ist das ziemlich viel überflüssiger Mist.

Rufe lieber von der paintComponent-Methode der Component, die das Face entahlten soll, deine (ursprüngliche!) draw-Methode auf.

 

[kulturfenster]

verstehe, vielen Dank!

public void paintComponent(Graphics g)
	{
		Graphics2D g2 = (Graphics2D) g;
		
		draw(g2);
	}
	
	public void draw(Graphics2D g2)
	{	
		g2.draw(kopf);
		g2.draw(rechtesAuge);
		g2.draw(linkesAuge);
		g2.draw(fresse);
	}

 

[Marco13]

Sicher? Das gepostete sieht aus, als stünde es in einer JComponent. Muss es aber nicht. *schulternzuck*

 

[kulturfenster]

hmm, was heisst "es steht in einem JComponent"?

 

[Xams]

E als ob es eine eigene JCOmponent wär

 

[André Uhres]

... aber bitte sowas NICHT von JComponent erben lassen! (Sofern das wirklich nur gezeichnet werden soll)

Bitte bring hier die Leute nicht durcheinander.
Natürlich muss es eine JComponent sein, denn das Gesicht kommt ja direkt in den JFrame:

     Face component = new Face();
      frame.add(component);

@kulturfenster: man kann selbstverständlich den Code, der in der draw Methode ist,
auch direkt in die paintComponent tun. Ich wollte einfach nur die Methoden nutzen, die schon da sind.

 

[Marco13]

Bitte bring hier die Leute nicht durcheinander.

Wenn "durcheinander" so viel heißt wie "auf den richtigen Weg": DOCH :wink:

Dir ist hoffentlich klar, dass sowas dann dazu führt, dass (z.B. für ein "Zeichenprogramm") sowas geschrieben wird wie

class Linie extends JComponent { ... g.drawLine ... }
class Kreis extends JComponent { ... g.drawOval ... }
class Rechteck extends JComponent { ... g.drawRect... }
class Face extends JComponent { ... g.drawOval g.drawOval g.drawOval g.drawLine ... }

somePanel.add(new Linie(...));
somePanel.add(new Kreis(...));
somePanel.add(new Rechteck(...));
somePanel.add(new Face(...));

Derartige Ansätze habe ich schon gesehen, und das ist (sehr zurückhaltend formuliert) absoluter Bockmist. Ein "Face" besteht in diesem Fall aus 3 Ellipsen ud einer Linie, und braucht keinen VetoableChangeSupport oder HierarchyBoundsListener. Ich gehe mal stark davon aus, dass das Face wirklich nur gezeichnet werden soll, und es KEINEN Grund gibt, das von JComponent abzuleiten. Aber wenn, dann bitte konsequent

class Pixel extends JComponent { ... g.drawLine(x,y,x,y); ... }

// A line of length 4:
panel.add(new Pixel(10,2));
panel.add(new Pixel(10,3));
panel.add(new Pixel(10,4));
panel.add(new Pixel(10,5));

:wink:

 

[André Uhres]

Das sind doch nur Unterstellungen die hier wirklich fehl am Platz sind. Es geht doch hier einfach nur um eine simple GUI mit zwei Teilen: ein JFrame mit einer JComponent. Mach doch bitte nicht gleich ein Monster oder was weiss ich daraus. Das ist imho im vorliegenden Fall nur irreführend.
Was übrigens die Reaktion von kulturfenster auch schon gezeigt hat.

 

[Marco13]

Ich weiß nicht genau, was ich "unterstellt" habe. Worum es geht, hat der Threadersteller gesagt: Er will ein Gesicht zeichnen. Man könnte auch das Face direkt auf den Frame zeichnen, was aber auch schlecht wäre, darum hast du daraus etwas ("richtigeres") extrapoliert, wobei ich dir zustimme: Es geht nur um eine GUI mit zwei Teilen. Einen JFrame und eine JComponent. Und auf die JComponent soll ein Gesicht gezeichnet werden. Allerdings ist ein Gesicht keine JComponent, sondern einfach ein Objekt, das aus drei Ellipsen und einer Linie besteht. Naja. Wir wissen beide worum es geht. Nur in der Frage, wie die Hilfe für den Threadersteller auszusehen hat, haben wir unterschiedliche Ansichen. Aber ich kann andere Ansichten akzeptieren....

:bae:

 

[André Uhres]

Ich meinte lediglich, du hast unterstellt, daß er vielleicht viele JComponents zum Zeichnen haben will.
Deine Warnung ist sicher gut gemeint, vielleicht an anderer Stelle auch angebracht, passt aber hier nicht :wink:

 

[Marco13]

Das wollte ich zwar nicht unterstellen, aber man muß ja davon ausgehen, dass dort noch mehr gezeichnet werden soll, als ein Gesicht. Und wenn er für das Gesicht eine Klasse anlegt (die von JComponent erbt) dann wird er für voraussichtlich auch für jedes andere "Objekt" eine derartige Klasse anlegen. Ich finde, sowas sollte man vermeiden. Wenn das eine Face aber das einzige ist, was in dem ganzen Programm ("von Hand") gezeichnet werden soll, wäre es OK, dafür eine solche Klasse anzulegen, die dann aber nicht "Face" sondern eher "FacePaintingComponent" heißen sollte :wink:

 

[André Uhres]

..dann wird er für voraussichtlich auch für jedes andere "Objekt" eine derartige Klasse anlegen..

Naja, dann sagen wir eben "prophezeien" statt "unterstellen", wenn das etwas ändert :wink:
(kulturfenster könnte klären, ob du ein wahrer Prophet bist, aber der lacht sich wahrscheinlich nur krumm über unser Spielchen)
Wie dem auch sei, das Thema ist ja jetzt abgehakt
Ich würde es dann einfach dabei belassen,
du kannst aber gern das letzte Wort haben (das Wort zum Sonntag ) :lol: