Methoden Objekt auf JFrame zeichnen

[koni1996]

Erstmal ein hallo in die Runde!
Vielleicht könnt ihr mit bei meinem kleinen Problem weiterhelfen:

Ich habe zwei Klassen:

import javax.swing.*;

public class Hauptklasse 
{
	private JFrame fenster = new JFrame("Fenster");
	private Rechteck rechteck = new Rechteck(10, 10, 200, 500);
	
	public Hauptklasse()
	{
		fenster.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        fenster.setSize(800, 500);

        // Um diese Stelle geht es!
		fenster.add(rechteck);

		fenster.setVisible(true);
	}
	
	public static void main(String[] args)
	{
		new Hauptklasse();
	}
}

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

public class Rechteck extends JPanel
{
	private Rectangle rechteck = new Rectangle();
	
	public Rechteck(int x, int y, int breite, int hoehe)
	{
		rechteck.setLocation(x, y);
		rechteck.setSize(breite, hoehe);
	}
}

Wie kann ich nun mein Rechteck-Objekt auf das Fenster zeichnen?
Wenn es dafür keine Methode gibt, wie könnte ich das Problem alternativ mit den beiden Klassen lösen?

 

[MrClave]

In Rechteck paintComponent(Graphics g) überschreiben und dort das gewünschte Rechteck zeichen.

g.drawRect(..);

 

[koni1996]

Danke für deine Hilfe!
Ich weiß zwar nicht, ob ich dir meinen "Geistesblitz" zu verdanken habe, aber jedenfalls ist mir nun die Herangehensweise klar geworden (erst die Attribute in Rechteck festlegen).

 

[koni1996]

Hm, ich schiebe gleich mal noch eine Frage hinterher:


Klasse "(Gefülltes) Rechteck":

...

public class Rechteck extends JLabel
{
	...
	private Color farbe;
	
	public Rechteck(..., Color farbeNeu)
	{
		...
		farbe = farbeNeu;
	}
	
	@Override
	protected void paintComponent(...) 
	{
		...
		grafik.setColor(farbe);
	}	
}

Klasse "Hauptklasse":

...
private Rechteck automat = new Rechteck(10, 10, 200, 500, Color.red);
...

Warum funktioniert diese Lösung nicht?
Welchen Datentyp muss die Farbe jeweils haben?

 

[MrClave]

Gibt meines Wissens nach nur eine Color Klasse.
Aber müsste schon funktionieren so.

@Override
    protected void paintComponent(...)
    {
        ...
        grafik.setColor(farbe);
    }

Rufst du grafik.setColor(farbe); erst nach dem anderen kram auf ?
Wenn ja, ändern! setColor muss immer nach vorne bzw. oben.
Default wird die Farbe immer auf schwarz gesetzt.

Oder compiliert dein Programm erst gar nicht ?

 

[koni1996]

setColor muss immer nach vorne bzw. oben.

Danke, das war mir nicht klar!

 

[koni1996]

Nun habe ich außerdem eine Klasse Kreis geschrieben.
Wenn ich diese in der Hauptklasse hinzufüge, wird jedoch das Rechteck nicht mehr gezeichnet. Wie kann ich das vermeiden?

 

[MrClave]

Hm, Code?

Du musst eigentlich einfach nur die paintComponent Methode in beiden Klassen überschreiben, und halt noch dem Frame hinzufügen und ggf. repaint aufrufen.
Ein LayoutManager sollte zudem nicht fehlen in deinem JFrame.
Sonst spielen die Panels gerne mal Ninja und machen sich ganz klein und unauffällig.

Mehr sollte das eigentlich nicht sein.

 

[koni1996]

...
	private RechteckGefuellt automat = new RechteckGefuellt(10, 10, 300, 500, Color.red);
	private KreisGefuellt walze1 = new KreisGefuellt(60, 60, 50, 50, Color.white);
	
	public Hauptklasse()
	{
		fenster.setTitle("Spielautomat");
		fenster.setSize(325, 545);
		fenster.setResizable(false);
		fenster.setLocationRelativeTo(null);
		fenster.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		fenster.setVisible(true);
		fenster.add(automat);
		fenster.add(walze1);
	}
...
}

 

[MrClave]

Aha, der klassische Fehler

.setVisible(true);

muss immer der letzte Aufruf sein!

Oder besser gesagt, immer nach dem hinzufügen von Komponenten.

 

[koni1996]

fenster.add(automat);
fenster.add(walze1);
fenster.setVisible(true);

Ändert leider nichts...

 

[MrClave]

Dann gib' deinem Frame ein Layout. GridLayout z.B.
Die beiden Panel werden sich überschneiden ohne LayoutManager, wodurch das unterste, bzw. erste Panel nicht sichtbar wird.

 

[koni1996]

Die sollen sich ja auch überschneiden?

 

[MrClave]

Dann musst du es aber auch auf das selbe Panel zeichen.

Ein Panel kannst du dir wie ein Blatt Papir vorstellen.
Wenn du auf eins zeichnest, und ein neues drüber legst, sieht man das untere nicht mehr.
Und wenn du 2 Blätter hast, kann sich da auch nichts überschneiden.

Daher bau dir eine Klasse, die von JPanel erbt.
Erstelle verschiedene Methoden:

public void zeichneKreis( Kreis kreis, Graphics g ) {
g.drawOval( ... );
}

 

[koni1996]

Ich verstehe nicht ganz wie das funktionieren soll, sorry.

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

public class Grafik extends JPanel
{
	public void rechteckGefuelltZeichnen(RechteckGefuellt rechteckgefuellt, Graphics grafik)
	{
		grafik.fillRect();
	}
}

 

[MrClave]

Hab' mal in 15 min. einen Beispiel Code gebastelt.
Man könnte natürlich noch mehr mit GoF Patterns arbeiten, aber ich denke, darauf kommt es hier gerade nicht an.
(Normal arbeitet man bei Grafiken mit public-fields (fast access), aber das ist hier noch irrelevant)

Die wesentlichen Elemente sind:
- Die Liste in PaintPanel
- Die paintComponent() Methode in PaintPanel
- Figure-Klasse
- Figure-Unterklassen + deren drawMethode

Falls du das Beispiel nicht verstehst, sag' einfach bescheid.

Spoiler: Code

Spoiler: MyFrame

import java.awt.BorderLayout;
import java.awt.Color;
import java.awt.GridLayout;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;

import javax.swing.BorderFactory;
import javax.swing.ButtonGroup;
import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JPanel;
import javax.swing.JRadioButton;
import javax.swing.JTextField;
import javax.swing.SwingConstants;



public class MyFrame implements ActionListener {
	
	private JFrame 				frame;
	private JPanel 				figurePanel, sizePanel, selectionPanel,
								widthPanel, heightPanel, posXPanel, posYPanel;
	private PaintPanel			paintPanel;
	private JButton 			paint;
	private JRadioButton 		circle, circleFilled, rect, rectFilled;
	private ButtonGroup 		figures;
	private JTextField			width, height, posX, posY;
	private JLabel				lb_width, lb_height, lb_posX, lb_posY;

	
	public MyFrame( ) {
			
		frame 				= new JFrame( "Painter" );
		
		paintPanel			= new PaintPanel( );
		figurePanel			= new JPanel( );
		sizePanel			= new JPanel( );
		selectionPanel		= new JPanel( );
		widthPanel			= new JPanel( );
		heightPanel			= new JPanel( );
		posXPanel			= new JPanel( );
		posYPanel			= new JPanel( );
		
		paint 				= new JButton( "Paint" );
		
		circle 				= new JRadioButton( "Circle" );
		circleFilled 		= new JRadioButton( "Circle Filled" );
		rect 				= new JRadioButton( "Rectangle" );
		rectFilled 			= new JRadioButton( "Rectangle Filled" );
		
		width 				= new JTextField( );
		height 				= new JTextField( );
		posX				= new JTextField( );
		posY				= new JTextField( );
		
		lb_width			= new JLabel( "Width: ", SwingConstants.RIGHT );
		lb_height			= new JLabel( "Height: ", SwingConstants.RIGHT );
		lb_posX				= new JLabel( "Pos. X: ", SwingConstants.RIGHT );
		lb_posY				= new JLabel( "Pos. Y: ", SwingConstants.RIGHT );
		
		figures				= new ButtonGroup( );
		
//		----------------------------------------------
		
		frame.setDefaultCloseOperation( JFrame.DISPOSE_ON_CLOSE );
		frame.setSize( 800, 600 );
		frame.setLocationRelativeTo( null );
		frame.setLayout( new BorderLayout( ) );
		
		figurePanel.setLayout( new GridLayout( 3, 1, 0, 20 ) );
		widthPanel.setLayout( new GridLayout( 1, 2 ) );
		heightPanel.setLayout( new GridLayout( 1, 2 ) );
		posXPanel.setLayout( new GridLayout( 1, 2 ) );
		posYPanel.setLayout( new GridLayout( 1, 2 ) );
		sizePanel.setLayout( new GridLayout( 4, 1, 0, 10 ) );
		selectionPanel.setLayout( new GridLayout( 4, 1 ) );
		
		sizePanel.setBorder( BorderFactory.createTitledBorder( "Size" ) );
		selectionPanel.setBorder( BorderFactory.createTitledBorder( "Figures" ) );
		
		paint.setFocusable( false );
		paint.addActionListener( this );
		
		circle.setFocusable( false );
		circleFilled.setFocusable( false );
		rect.setFocusable( false );
		rectFilled.setFocusable( false );
		
//		-----------------------------------------------
		
		figures.add( circle );
		figures.add( circleFilled );
		figures.add( rect );
		figures.add( rectFilled );
		
		widthPanel.add( lb_width );
		widthPanel.add( width );
		
		heightPanel.add( lb_height );
		heightPanel.add( height );
		
		posXPanel.add( lb_posX );
		posXPanel.add( posX );
		
		posYPanel.add( lb_posY );
		posYPanel.add( posY );
		
		sizePanel.add( widthPanel );
		sizePanel.add( heightPanel );
		sizePanel.add( posXPanel );
		sizePanel.add( posYPanel );
		
		selectionPanel.add( circle );
		selectionPanel.add( circleFilled );
		selectionPanel.add( rect );
		selectionPanel.add( rectFilled );
		
		figurePanel.add( sizePanel );
		figurePanel.add( selectionPanel );
		figurePanel.add( paint );
		
		frame.add( paintPanel, BorderLayout.CENTER );
		frame.add( figurePanel, BorderLayout.EAST );
		
		frame.setVisible( true );
		
	}

	private void paintAction( ) {
		
		int x = Integer.parseInt( posX.getText( ) );
		int y = Integer.parseInt( posY.getText( ) );
		int width = Integer.parseInt( this.width.getText( ) );
		int height = Integer.parseInt( this.height.getText( ) );
		
		if ( circle.isSelected( ) ) {
			
			paintPanel.addFigure( new Circle( x, y, width, height ) );
			
		} else
			
			if ( circleFilled.isSelected( ) ) {
				
				paintPanel.addFigure( new FilledCircle( x, y, width, height, Color.RED ) );
				
			} else 
				
				if ( rect.isSelected( ) ) {
					
					paintPanel.addFigure( new Rect( x, y, width, height ) );
					
				} else
					
					if ( rectFilled.isSelected( ) ) {
						
						paintPanel.addFigure( new FilledRect( x, y, width, height, Color.BLUE ) );
						
					}
			
	}

	@Override
	public void actionPerformed( ActionEvent e ) {

		if ( e.getSource( ) == paint ) {
			
			paintAction( );
			
		}
		
	}

}

Spoiler: PaintPanel

import java.awt.Color;
import java.awt.Graphics;
import java.util.ArrayList;
import java.util.List;

import javax.swing.JPanel;



public class PaintPanel extends JPanel {

	private static final long		serialVersionUID	= 1L;
	private List<Figure> 			figures;
	
	
	public PaintPanel( ) {
		
		figures = new ArrayList<>( );

	}
	
	@Override
	protected void paintComponent( Graphics g ) {
	
		super.paintComponent( g );
		
		for ( Figure fg : figures ) {
			
			fg.draw( g );
			g.setColor( Color.BLACK );
			
		}
		
	}
	
	public void addFigure( Figure figure ) { figures.add( figure ); repaint( ); }
	public boolean removeFigure( Figure figure ) { return figures.remove( figure ); }
	public List<Figure> getFigures( ) { return figures; }

}

Spoiler: Figure

import java.awt.Graphics;



public abstract class Figure {
	
//	Fields
	protected int 			x;
	protected int 			y;
	protected int 			width;
	protected int 			height;
	
//	Constructors
	public Figure( int x, int y, int width, int height ) {

		this.x 				= x;
		this.y 				= y;
		this.width 			= width;
		this.height 		= height;
		
	}
	
//	Abstract Methods
	public abstract void draw( Graphics g );

//	Set/Get
	public int getX( ) { return x; }
	public void setX( int x ) { this.x = x; }
	
	public int getY( ) { return y; }
	public void setY( int y ) { this.y = y; }
	
	public int getWidth( ) { return width; }
	public void setWidth( int width ) { this.width = width; }
	
	public int getHeight( ) { return height; }
	public void setHeight( int height ) { this.height = height; }
	
}

Spoiler: Circle

import java.awt.Graphics;



public class Circle extends Figure {


	public Circle( int x, int y, int width, int height ) {

		super( x, y, width, height );
		
	}

	@Override
	public void draw( Graphics g ) {

		g.drawOval( x, y, width, height );
		
	}

}

Spoiler: FilledCircle

import java.awt.Color;
import java.awt.Graphics;



public class FilledCircle extends Figure {
	
//	Fields
	private Color color;

	
//	Constructors
	public FilledCircle( int x, int y, int width, int height, Color color ) {

		super( x, y, width, height );
		this.color = color;
		
	}

//	Inherited Methods
	@Override
	public void draw( Graphics g ) {

		g.setColor( color );
		g.fillOval( x, y, width, height );
		
	}
	
	public void setColor( Color color ) { this.color = color; }
	public Color getColor( ) { return color; }
	
}

Spoiler: Rect

import java.awt.Graphics;



public class Rect extends Figure {

	
//	Constructors
	public Rect( int x, int y, int width, int height ) {

		super( x, y, width, height );
	}

//	Inherited Methods
	@Override
	public void draw( Graphics g ) {

		g.drawRect( x, y, width, height );
		
	}

}

Spoiler: FilledRect

import java.awt.Color;
import java.awt.Graphics;



public class FilledRect extends Figure {

//	Fields
	private Color color;
	
	
//	Constructors
	public FilledRect( int x, int y, int width, int height, Color color ) {

		super( x, y, width, height );
		this.color = color;
		
	}

//	Inherited Methods
	@Override
	public void draw( Graphics g ) {

		g.setColor( color );
		g.fillRect( x, y, width, height );
		
	}

	public void setColor( Color color ) { this.color = color; }
	public Color getColor( ) { return color; }


}

 

[koni1996]

Danke, muss ich mir morgen anschauen.
Schaut aufwendig aus...