Frage zum einfügen von Componenten

[kaoZ]

Aloha , ich steh grad etwas auf dem schlauch , ich habe folgende Situation:

Ein JFrame, und ein Panel, in diesem Panel möchte ich bei Button Betätigung ein neues Panel erstellen, soweit so gut es sieht zzt. folgendermaßen aus

fügt das panel dem mainframe hinzu

	private void createPrintPanel(){
		printPanel = new JPanel(new BorderLayout());
		printPanel.setSize(frame.getHeight(), frame.getWidth());
		frame.add(BorderLayout.CENTER, printPanel);
	}

added auf Buttonklick dem panel eine neue "Komponente" und übergibt dieser die Referenz auf das elternelement

switch(e.getActionCommand()){
		case "new":
			printPanel.add(BorderLayout.CENTER, new Invoice(printPanel));
			break;

und zu guter letzt das panel was innen dargestellt werden soll, dieses erbt in diesem fall von JPanel, ich hab allerdings auch schon versucht es von JComponent erben zu lassen, das ergebnis bleibt allerdings das gleiche ....

public class Invoice extends JPanel{
	private static final long serialVersionUID = 1L;
	
	JPanel parent;
	Dimension d;
	
	public Invoice(JPanel parent) {
		super();
		this.parent = parent;
		d = parent.getSize();
		init();
	}

	private void init(){
		setSize(d);
		setBackground(Color.cyan);
		repaint(); // das hier war nur ein versuche ob es vielleicht nicht richtig gezeichnet wird
		setVisible(true);
	}
}

So sieht es dann aus:



wenn ich Debugge referenziert

d

folgende werte

 height = 1019 width = 954

, warum um alles in der Welt stellt er es nur so klein da ^^

vielleicht gibt es ja auch eine einfachere Lösung sowas zu realisieren, wenn ja nehmen ich Tipps gerne entgegen , ich möchte lediglich erreichen das ich diesen bereich später ausdrucken kann, mit samt seinem Inhalt

 

[kaoZ]

Wenn ich das Fenster minimiere und wieder aufrufe, also wenn sich der inhalt des Panel neu Zeichnet, füllt er wie gesollt die ganze Fläche, wenn ich wie im Beispiel aber repaint() aufrufe bleibt es unverändert...

[EDIT]
oder gibt es eine andere Möglichkeit einem Panel child elemente zur laufzeit zuzuweisen ?
oder MUSS ich mein Frame das ich darstellen möchte von JComponent anstelle von JPanel ableiten ?
[/EDIT]

jemand eine Idee oder eine Ahnung wie ich sowas realisieren kann ?

JInternalFrame ist nicht das was ich suche,

ich möchte über das JMenü dem PrintPanel eine neue "Rechnung" hinzugügen oder später einen "Kostenvoranschlag" , beides soll dadurch das es sich dann im elternelement printPanel befindet, über einen PrintJob ausdruckbar sein.

wäre klasse wenn jemand mal drüberschauen könnte.

[EDIT]
oder muss ich in dem Elternelement tatsächlich ein Null-Layout verwenden um die gewünschte größe zu erhalten ? denn egal welche größe ich dem Childelement gebe wird es immer klein oben links in die Ecke gezeichnet, mit scheint als müsste ich irgendwie die größe vorher festlegen ... aber wie ^^
[/EDIT]

 

[MrClave]

Du musst (re-)validate aufrufen.

public void validate()
Validates this container and all of its subcomponents.
Validating a container means laying out its subcomponents. Layout-related changes, such as setting the bounds of a component, or adding a component to the container, invalidate the container automatically. Note that the ancestors of the container may be invalidated also (see Component.invalidate() for details.) Therefore, to restore the validity of the hierarchy, the validate() method should be invoked on the top-most invalid container of the hierarchy.

Validating the container may be a quite time-consuming operation. For performance reasons a developer may postpone the validation of the hierarchy till a set of layout-related operations completes, e.g. after adding all the children to the container.

If this Container is not valid, this method invokes the validateTree method and marks this Container as valid. Otherwise, no action is performed.

 

[kaoZ]

Du musst (re-)validate aufrufen.

An welcher Stelle muss ich den validate() aufrufen ?, wenn ich nämlich repaint() durch validate() ersetze, komme ich zu genau dem gleichen ergebnis.

[EDIT]oder muss ich das printPanel (JPanel) validieren ? in das ja dann per klick die Komponente hinzugefüge [/EDIT]

 

[MrClave]

Nein, nicht unbedingt, denn die Komponente, auf der du validate aufrufst, validiert alle Kinderkomponenten.

Wie ich aber gerade festgestellt habe, ist das setSize( d ); zuviel.
Lass das in der Invoice Klasse weg, und benutz den valdidate Aufruf auf deinem JFrame.
(Das setVisible( true ); kannst du auch weglassen)

 

[kaoZ]

dann passiert genau garnichts mehr also zzt. ist es folgendermaßen

private void createPrintPanel(){
		printPanel = new JPanel(new BorderLayout());
		printPanel.setSize(frame.getHeight(), frame.getWidth());
		frame.add(BorderLayout.CENTER, printPanel);
	}

diese Methode erstellt den Panel in dem später ein zusätliches Panel mit inhalt hinzugefügt werden soll

public class Invoice extends JPanel{
	private static final long serialVersionUID = 1L;
	
	JPanel parent;
	
	public Invoice(JPanel parent) {
		super();
		this.parent = parent;
		init();
	}

	private void init(){
		setBackground(Color.cyan);
		System.out.println("Komponente Hinzugefügt");
	}
	
}

Dies soll der Inhalt des Panels sein welchen ich dem printPanel hinzufügen möchte, zu testzwecken soll hier erstmal nur der hintergrund umgefärbt werden und auf der Konsole der Text ausgegeben werden um zu sehen ob der Listener funktioniert, was er tut.

Das ist aber auch schon alles was passiert, der Hintergrund bleibt unverändert, egal an welcher stelle ich validate() aufrufe, ob im Frame, im printPanel oder in Invoice.

Jemand eine Idee warum sich hier diese "Komponente" nicht hinzufügen bzw. anzeigen lässt ?

[EDIT]
sobald ich in Invoice folgendes wieder hinzufüge

setSize(parent.getHeight(), parent.getWidth());

funktioniert es wie vorher, kleiner blauer kasten , kurz minimiert, und schon füllt er das ganze panel so wie er es soll, nur halt ohne voheriges minimieren ( bzw. neu zeichnen ).
[/EDIT]

 

[MrClave]

Denn zeig' mal den ganzen Code. Irgendwas machst du falsch, denn normalerweise muss es so funktionieren.
Das setSize kannst du dir übrigens überall sparen, da BorderLayout die Komponente automatisch auf die maximale Größe streckt.


Hier mal ein KSKB

Spoiler: Code

public class Test {
	
	static class MyPanel extends JPanel {

		private static final long	serialVersionUID	= 1L;
		private JPanel parent;
		
		public MyPanel( JPanel parent ) {
			
			super( );
			this.parent = parent;
			init( );
			
		}
		
		private void init( ) {
			
			setBackground( Color.RED );
			
		}
		
	}

	public static void main( String[] args ) {

		final JFrame frame = new JFrame( );
		frame.setSize( 800, 600 );
		frame.setLocationRelativeTo( null );
		frame.setDefaultCloseOperation( JFrame.EXIT_ON_CLOSE );
		frame.setLayout( new BorderLayout( ) );
		
		final JPanel mainPanel = new JPanel( new BorderLayout( ) );
		
		JButton btnAdd = new JButton( "Add" );
		btnAdd.addActionListener( new ActionListener( ) {
			
			@Override
			public void actionPerformed( ActionEvent e ) {
				
				MyPanel panel = new MyPanel( mainPanel );
				mainPanel.add( panel, BorderLayout.CENTER );
				frame.validate( );
				
			}
		} );
		
		frame.add( mainPanel, BorderLayout.CENTER );
		frame.add( btnAdd, BorderLayout.NORTH );
		frame.setVisible( true );

	}

}

 

[kaoZ]

Komisch das ganze , vielleicht siehst du ja warum es nur so merkwürdig funktioniert

Spoiler: MainView.java

package de.kaoZ.mainViews;

import java.awt.BorderLayout;
import java.awt.Dimension;
import java.awt.Toolkit;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.io.File;

import javax.swing.JFrame;
import javax.swing.JMenu;
import javax.swing.JMenuBar;
import javax.swing.JMenuItem;
import javax.swing.JPanel;

import de.kaoZ.handler.StartupHandler;
import de.kaoZ.handler.UserHandler;
import de.kaoZ.interfaces.IView;
import de.kaoZ.setup.DialogView;
import de.kaoZ.user.User;

public class MainView implements IView, ActionListener{

	private boolean setupFinished;
	private String workPath;
	
	User user;
	
	JFrame frame;
	JPanel printPanel;
	
	StartupHandler check;
	
	public MainView() {
		check = new StartupHandler();
		this.setupFinished = check.getSetupStatus(); 
		if (!(setupFinished || new File(check.getRootPath() + "/SetupStatus.dat").exists())) {
			new DialogView().show();
		}
		else if(setupFinished && new File(check.getRootPath() + "/SetupStatus.dat").exists()){
			this.workPath = check.getWorkPath();
			this.user = UserHandler.deserializeData(this.workPath + "/License.dat");
			createFrame();
			addMenue();
			show();
			createPrintPanel();
		}
	}
	
	private void createFrame() {
		frame = new JFrame(TITLE);
		
		Dimension x = Toolkit.getDefaultToolkit().getScreenSize();
		
		frame.setSize(x.width / 2 , x.height - 10);
		frame.setDefaultCloseOperation(JFrame.DISPOSE_ON_CLOSE);
		frame.setLocationRelativeTo(null);
		frame.setResizable(false);
	}
	
	private void addMenue() {
		JMenuBar bar = new JMenuBar();
		
		JMenu file = new JMenu("Datei");
		JMenu edit = new JMenu("Bearbeiten");
		JMenu lookFeel = new JMenu("Fenster");
		JMenu clients = new JMenu("Kundendaten");
		JMenu options = new JMenu("Optionen");
		JMenu help = new JMenu(" ? ");
		
		JMenuItem createNew = new JMenuItem("Neu...");
			createNew.addActionListener(this);
			createNew.setActionCommand("new");
		JMenuItem open = new JMenuItem("Öffnen...");
			open.addActionListener(this);
			open.setActionCommand("open");
		JMenuItem close = new JMenuItem("Beenden");
			close.addActionListener(this);
			close.setActionCommand("close");
			
		JMenuItem resolution = new JMenuItem("Auflösung");
		
		file.add(createNew);
		file.add(open);
		file.add(close);
		
		lookFeel.add(resolution);
		
		bar.add(file);
		bar.add(edit);
		bar.add(lookFeel);
		bar.add(clients);
		bar.add(options);
		bar.add(help);
		
		frame.add(BorderLayout.PAGE_START, bar);	
	}
	
	private void createPrintPanel(){
		printPanel = new JPanel(new BorderLayout());
		printPanel.setSize(frame.getHeight(), frame.getWidth());
		frame.add(BorderLayout.CENTER, printPanel);
	}
	
	@Override
	public void show() {
		frame.setVisible(true);
	}
	
	@Override
	public void actionPerformed(ActionEvent e) {
		
		switch(e.getActionCommand()){
		case "new":
			printPanel.add(BorderLayout.CENTER, new Invoice(printPanel));
			break;
		case "open":
		
			break;
		case "close":
			frame.dispose();
			System.exit(0);
			break;
		}	
	}
}

Spoiler: Invoice.java

public class Invoice extends JPanel{
	private static final long serialVersionUID = 1L;
	
	JPanel parent;
	
	public Invoice(JPanel parent) {
		super();
		this.parent = parent;
		init();
	}

	private void init(){
		setBackground(Color.cyan);
		setSize(parent.getHeight(), parent.getWidth());
		System.out.println("Komponente Hinzugefügt");
	}
	
}

 

[kaoZ]

Ich habs, steht

validate()

mit in dem Listener, wird also bei buttonklick mit ausgeführt funktioniert es:

switch(e.getActionCommand()){
		case "new":
			printPanel.add(BorderLayout.CENTER, new Invoice(printPanel));
			printPanel.validate(); // hier muss validate() mit rein 
			break;
		case "open":

 

[MrClave]

Häh?
Funktioniert doch mit validate...
Du hattest es praktisch schon richtig vorher, nur hättest du in der Invoice Klasse

parent.validate();

, statt einfach nur

validate();

aufrufen müssen.

case "new":
	printPanel.add( BorderLayout.CENTER, new Invoice( printPanel ) );
	printPanel.validate( );
	break;

Wenn nur Änderungen im printPanel passieren, würde ich auch nur dieses validieren.
Denn wenn du den frame validierst, wird ALLES validiert, was unter Umständen auf die Performance geht.
Wenn noch andere Dinge im Frame passieren, musst du halt

frame.validate();

benutzen.

Das

setSize( ... );

solltest du außerdem immer noch entfernen, da es ohnehin keine Wirkung hat beim BorderLayout.


[EDIT]Zu langsam...[/EDIT]

 

[kaoZ]

Häh?
Funktioniert doch mit validate...
Du hattest es praktisch schon richtig vorher, nur hättest du in der Invoice Klasse

parent.validate();

, statt einfach nur

validate();

aufrufen müssen.

Hab ich versucht, funktioniert aus welchem Grund auch immer nicht .....

Spoiler

public class Invoice extends JPanel{
	private static final long serialVersionUID = 1L;
	
	JPanel parent;
	
	public Invoice(JPanel parent) {
		super();
		this.parent = parent;
		init();
	}

	private void init(){
		parent.validate();
		setBackground(Color.cyan);
	}
	
}

[EDIT]

super.validate(); // funktioniert übrigens auch nicht ;)

[/EDIT]

Untenstehender Code funktioniert Reibungslos

ggf. kann ich ja aus einem Child auch einfach nicht das Elternelement validieren ?! auch wenn das Child eine Referenz darauf hält.

case "new":
	printPanel.add( BorderLayout.CENTER, new Invoice( printPanel ) );
	printPanel.validate( );
	break;

nun müsste es ja auch möglich sein Invoice einfach von JComponent anstellle von JPanel abzuleiten, da es ja ebend auch eine "Komponente" ist oder ?

 

[MrClave]

Mein Fehler. Das obere kann natürlich nicht funktionieren, weil du validate() indirekt durch init im Konstruktor aufrufst.
Und das Panel kann natürlich erst hinzugefügt werden, wenn es bereits fertig erzeugt ist, sprich validate() schon aufgerufen wurde.
Wenn du init von außen aufrufen würdest, würde es funktionieren.
(Falls das nicht stimmt, entschuldige)

Dass super.validate() auch nicht funktioniert, ist ebenfalls klar.
Super steht ja schließlich für die Vaterklasse, nicht die Vaterkomponente.

Könntest du theoretisch machen, ja.

 

[kaoZ]

Soweit , sogut.

wenn ich jetzt allerdings von JComponent ableite, wird nichts mehr angezeigt, JComponent erbt ja von Container und JPanel von JComponent, demnach sollte es doch möglich sein den "Container" in einem Panel anzeigen zu lassen , oder muss ich dann explizit noch setVisible(true) aufrufen ?

 

[MrClave]

Das liegt daran, dass JComponent seinen Hintergrund nicht zeichnet. Außerdem besitzt JComponent keinen LayoutManager, wodurch man mit setBounds arbeiten müsste, beim hinzufügen von Komponenten.

Entweder überschreibst du also die paintComponent Methode in Invoice

@Override
	protected void paintComponent( Graphics g ) {

		super.paintComponent( g );
		g.fillRect( 0, 0, getWidth( ), getHeight( ) );
		
	}

oder du belässt es bei JPanel, was dir wohl viel Arbeit ersparen wird.

 

[kaoZ]

sowas dachte ich mir irgendwie schon , war mir aber nicht sicher ....dann werd ich es wohl dabei belassen ;P