Swing Vorgehen beim Aufruf der Klassen/Methoden

[martin1989]

Hallo!

Ich habe schlicht gesagt ein sehr "einfaches" Problem, dass mir aber leider ein wenig den Kopf zerbricht.

Ich habe eine Klasse Gui, in dem Konstruktor dieser Klasse baue ich mein GUI auf.
Dann habe ich in einem extra package eine Klasse CreateTable wo ich eine JTable erzeugen und befüllen will. Dann habe ich in diesem Package noch eine Klasse "model" dies ist TableModel für meine JTable.

Ich wollte fragen, ob mein Aufruf der JTable in dem Konstruktor von "gui" so richtig ist, und vor allem ob man das so macht?

        ...
        CreateTable table1 = new CreateTable(); 
        JTable table = table1.createTable(model);
         
         
         panelOne.add( new JScrollPane( table ), BorderLayout.CENTER );

....


Folgend ein vereinfachter Überblick der relevanten Klassen:

public class Gui {

	
    final Model model = new Model();
    JFrame frame = new JFrame("tool");
    PanelMain panelMain = new PanelMain();
    //USW
	
    public  Gui() {
        
         frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
         
         panelMain.setLayout(cl);
     	
         panelOne.setName("1");
         panelOne.setLayout(bl);
         panelSecond.setName("2");
         //USW 

        
        CreateTable table1 = new CreateTable(); 
        JTable table = table1.createTable(model);
         
         
         panelOne.add( new JScrollPane( table ), BorderLayout.CENTER );
         
             	 
    	 frame.add(panelMain, BorderLayout.CENTER);
    	//USW

public class CreateTable extends JTable{
		

		
		public JTable createTable(Model model){
			

			// Das JTable initialisieren
			 JTable table = new JTable( model );

			 table.setAutoResizeMode(JTable.AUTO_RESIZE_OFF);
			

	     
	        TableRowSorter<TableModel> sorter = new TableRowSorter<TableModel>();
	 
                      // MIT DATEN BEFÜLLEN USW 

			
	
			return table;

			
		}

Vielen Dank!

 

[Gucky]

Du benutzt eine sehr unsinnige Variante des FactoryPatterns. Deine Klasse CreateTable erweitert JTable, von dieser Erweiterung wird aber an keiner Stelle profitiert. Stattdessen gibt es eine Methode, die genauso heißt, wie die Klasse und sich um die Erzeugung der Table kümmert. Die Erweiterung kann weg. Allgemein solltest du nur Klassen erweitern, wenn du auch Funktionalitäten hinzufügst.
Klassen werden nicht nach ihrem Befehl benannt sondern nach ihrer Aufgabe. Deine Klasse müsste demzufolge TableFactory oder TableCreator heißen. (eher Ersteres)
Du solltest eine Variable auf alle Komponenten des GUIs halten.

Ansonsten sehe ich keine Mängel.

 

[martin1989]

Besten Dank für deine Antwort!

Habe die Erweiterung weggemacht.

Was meinst du mit

Du solltest eine Variable auf alle Komponenten des GUIs halten.

?

Ok, mir kam es nur irgendwie komisch vor, dass diese Methode in der Klasse "TableFactory" dann quasi einen return type hat.


Ähnliche Frage:

Ich habe eine Klasse gemacht "PanelLeftMenue", diese soll das Panel sein das links am rand angezeigt wird.
In dieser Klasse füge ich im Konstruktor einen Button auf dieses Panel hinzu.
Daher gibt es auch eine anonyme Klasse wo die methode "action performed" drinnen ist.
Nun will ich, wenn man auf diesen Button des PanelLeftMenue klickt, das auf dem Panel daneben (PanelMain), welches ein Cardlayout hat, ein anderes panel aufgerufen wird.


Ich habe es so gelöst:

Klasse PanelLeftMenue:

public class PanelLeftMenue extends JPanel{

	JButton buttonOne = new JButton("Open",createImageIcon("/resources/Open.gif"));
	

    public PanelLeftMenue(final PanelMain panelMain, final CardLayout cl) {


      	 this.add(buttonOne);
    	 buttonOne.setPreferredSize(new Dimension(100,25));	


     	 buttonOne.addActionListener(new ActionListener(){
			public void actionPerformed(ActionEvent arg0){
				 cl.show(panelMain, "3");
				
			  }
		});

Klasse GUI:

public class Gui {
	
....
    
    CardLayout cl = new CardLayout();
    PanelLeftMenue panelLeftMenue = new PanelLeftMenue(panelMain, cl);

macht man dies so?


also konkret bezieht sich meine frage auf das übergeben des "panelMain" bzw. "cl" und ob man dies so macht wenn man das panel quasi als externe klasse auslagert?


Plane das auch so für die anderen Panels zu machen.


VIelen Dank!

 

[Gucky]

Da du keine Funktionalitäten hinzufügst, wäre entweder eine Factory oder einfach das direkte Benutzen des Panels sinnvoller.

Zu dem Bild für den Button: du solltest ImageIO nehmen. Wenn du das Programm fertig kompilierst, dann wird das so nicht funktionieren.

 

[martin1989]

was hast du den mit diesem satz in deinem ersten post gemeint?

"Du solltest eine Variable auf alle Komponenten des GUIs halten."



Naja ich füge dann dem Button schon noch einen FileOpener hinzu usw...
Wenn ich das alles in der GUI Klasse mache, wird sie halt sehr unübersichtlich finde ich ....


Was meinst du genau mit dem direkten benutzen des panels?

danke!

 

[Gucky]

Das Panel erfüllt keine Aufgabe, außer ein Panel sein.
Zum Beispiel habe ich einmal ein kleines Programm zur Zeitmessung in Verbindung mit Events geschrieben. Man konnte eingeben, nach wie vielen Millisekunden ein Event ausgelöst werden sollte und wenn es ausgelöst wurde, wird es in einer JList angezeigt. In diese JList habe ich bestimmte Objekte eingefügt. Da die Standard add-Methode aber alle Components akzeptiert, anstatt nur meiner, habe ich sie überschrieben. Das ist ein Grund eine Klasse zu erweitern.
Auslagern kannst du aber für deine Zwecke wäre eine Factory sinnvoller.

Was macht der FileOpener im GUI? Dafür solltest du eine Handler Klasse erstellen. Das GUI ist ausschließlich dazu da, um Dinge anzuzeigen.


Mit dem Satz meine ich, dass du nicht

panel.add(new Component());

schreiben solltest, sondern

class Exempel{
  private Component comp;

...

  public Exempel(){
    this.comp = new Component();
  }
...
  private void init(){
    this.add(comp);
  }
}

Das sieht auf den ersten Blick nach viel mehr Arbeit aus aber es kostet (fast) keinen Speicher (je nach JRE 32 oder 64 Bit) und es kann dir viel Ärger und Mühen ersparen, falls du doch noch einmal auf die Komponente zugreifen willst. In der Regel ist das der Fall.

 

[martin1989]

Der FileOpener ist auch in der Klasse JPanelLeftMenue deklariert:

public class PanelLeftMenue extends JPanel{

	JButton buttonOne = new JButton("Open",createImageIcon("/resources/Open.gif"));
	

    
    public PanelLeftMenue(final PanelMain panelMain, final CardLayout cl) {


      	 this.add(buttonOne);
    	 buttonOne.setPreferredSize(new Dimension(100,25));	

    	 
     	 buttonOne.addActionListener(new ActionListener(){
			public void actionPerformed(ActionEvent arg0){
				 cl.show(panelMain, "3");
				 System.out.println("ZZ");
				 

			        JFileChooser chooser = new JFileChooser();
			      		  
			        chooser.setDialogType(JFileChooser.OPEN_DIALOG);
			        
			        JFrame meinJFrame = new JFrame("Dateiauswahl");
			     
			        meinJFrame.setSize(450,300);
			        
			        meinJFrame.getContentPane().add(chooser);
			       
			        meinJFrame.setVisible(true);
				 
				 
				 
				 
			  }
		});

Bei deinem geposteten Code ist mir nicht ganz klar, woher die "add" methode sein soll, habs mal so getestet:

public class Test {

	private JButton button;

	
	public Test(){
		
		this.button = new JButton();
	
		
	}
	
	
	public void init(){
		
		
		 this.add(button); // ERROR add unbekannt 
		
	}
	
	
	public static void main(String[] args) {
		
	Test test = new Test();
        test.init();
		
		
	}
	
	
	
	
}

 

[Gucky]

Oh ja. Exempel sollte eigentlich JPanel erweiter. Mein Fehler

init() sollte vom Konstruktor aufgerufen werden.