Swing Cardlayout aufräumen

[fetcher]

Hallo liebes Forum,
ich habe ein kleines Problem. Es geht darum alle "add"- Anweisungen was das Cardlayout bzw. Panel betrifft in eine extra Methode auszulagern, damit ich nicht alles(ganz unterschiedlicher Inhalt) in einer Methode stehen habe.

Der Code funktioniert so wie er ist anhand eines Beispiels bei Youtube.
Hier mal der Code:

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


public class MainGUI extends JFrame{
	private JButton button1, button2, button3;
	private JPanel panel1 = new JPanel(), panel2 = new JPanel(), contentPanel = new JPanel();
	private CardLayout cardLayout = new CardLayout();
	
	public MainGUI(){
		super("TEST");
		button1 = new JButton("Go to second");
		button2 = new JButton("Go to first card");
		button3 = new JButton("test");


		panel1.setBackground(Color.GREEN);
		panel2.setBackground(Color.blue);
		
		actionListener al = new actionListener();
		
		button1.addActionListener(al);
		button2.addActionListener(al);
		
		panel1.add(button1);
		panel2.add(button2);
		
		contentPanel.setLayout(cardLayout);
		
		contentPanel.add(panel1, "panel1");
		contentPanel.add(panel2, "panel2");
		
		this.setContentPane(contentPanel);
		
		//cardLayout.show(contentPanel, "panel1");
		contentPanel1();
	}
	
	
	public void contentPanel1()
	{
		// add 
		cardLayout.show(contentPanel, "panel2");
	}
	
	
	public void contentPanel2()
	{
		// add
		cardLayout.show(contentPanel, "panel1");
	}

	public class actionListener implements ActionListener{

		@Override
		public void actionPerformed(ActionEvent e) {
			JButton src = (JButton) e.getSource();
			if(src.equals(button1))
			{
				contentPanel1();
			}
			
			if(src.equals(button2))
			{
				contentPanel2();
			}
			
			
		}
		
	}
	
}

Ich möchte lediglich alle "add"-Anweisungen aus der MainGUI in verschiedene Themen splitten. Dazu habe ich schon 2 Methoden vorbereitet(contentPanel1(); und contentPanel2().

Wenn ich jetzt den Code aus der MainGUI einfach in contentPanel1 oder 2 kopiere läuft mein Programm nicht mehr. Kann mir vielleicht jemand dabei helfen?

Vielen Dank
Gruß fetcher

 

[kaoZ]

Öhm ich verstehe deine Frage Zwar nicht wirklich aber du möchtest die einzelnen "Karten" auslagern ?

falls dem so sein sollte kannst du einfach eine Abstrakte Basisklasse erstellen , nennen wir Sie Card

public abstract class Card extends JPanel{

 private String name;

 public Card(String name){
    this.name = name;
    setPreferredSize(new Dimension(300,300);
 }
}

und von dieser kannst du dann deine Spezialisierungen ableiten

z.B eine Bunte Karte

public class ColorCard extends Card{

  public ColorCard(){
   super("ColorCard");
   setBackground(Color.RED);   
 }
}

in deiner main instanziierst du diese dann lediglich

cardLayout.add(new ColorCard());

du kannst natürlich auch in deiner Gui eine Methode anbieten mit der Du Karten hinzufügen kannst

public void addCard(Card card){
  CardLayout layout = getContentPane().getLayout();
  layout.add(card);
}

 

[fetcher]

Hallo, vielen Dank für deine Antwort.
Es geht zB um das hier:

	button1.addActionListener(al);
button2.addActionListener(al);
 
panel1.add(button1);
panel2.add(button2);
 
contentPanel.setLayout(cardLayout);
 
contentPanel.add(panel1, "panel1");
contentPanel.add(panel2, "panel2");

Vielleicht ist auslagern falsch ausgedrückt. Ich meine einfach Inhaltlich der Methode zuordnen und nicht einfach alle Buttons und Textfelder in eine Methode packen, da es verdammt viele werden. contentPanel1 baut dann den Inhalt für Panel1 und nicht auch gleich für Panel2.

 

[kaoZ]

wenn du es in eine methode auslagern willst muss diese einen Rückgabewert ( nicht void ) haben , in deinem Fall JPanel

sprich

public JPanel createComponentOne(){
  JPanel panel = new JPanel();

   // your code here...

  return panel;

}

diese methode kannst du dann deinem CardLayout hinzufügen da der Rückgabewert ein JPanel ist

contentPanel.add(createComponentOne());

wenn du dann durch die einzelnen "Karten" schalten willst musst du deinem Button , welchem eine Listener hinzugefügt sein muss, dazu bringen das er die next(), oder previous() Methode des CardLayouts aufruft.

sprich sowas hier

@Override
public void actionPerformed(ActionEvent e){
 CardLayout layout = (CardLayout) getContentPane().getLayout();
 layout.next();
}

Hier mal ein Beispiel mit MouseListener

import java.awt.CardLayout;
import java.awt.Color;
import java.awt.event.MouseEvent;
import java.awt.event.MouseListener;
import javax.swing.JFrame;
import javax.swing.JPanel;



public class CardExample extends JFrame{

	JPanel contentPane;
	
	
	public CardExample(){
		contentPane = new JPanel();
		contentPane.setLayout(new CardLayout());
	
		contentPane.add(createComponentOne());
		contentPane.add(createComponentTwo());
		
		setSize(300,300);
		setDefaultCloseOperation(JFrame.DISPOSE_ON_CLOSE);
		setContentPane(contentPane);
		pack();
		setLocationRelativeTo(null);
		addMouseListener(new MouseListener(){
			
			@Override
			public void mouseReleased(MouseEvent e){}
			
			@Override
			public void mousePressed(MouseEvent e){
				CardLayout layout = (CardLayout) getContentPane().getLayout();
				layout.next(getContentPane());
			}
			
			@Override
			public void mouseExited(MouseEvent e){}
			
			@Override
			public void mouseEntered(MouseEvent e){}
			
			@Override
			public void mouseClicked(MouseEvent e){}
		});
		
	}
	
	private JPanel createComponentOne(){
		JPanel panel = new JPanel();
		
		panel.setBackground(Color.RED);
		
		return panel;
	}
	
	private JPanel createComponentTwo(){
		JPanel panel = new JPanel();
		
		panel.setBackground(Color.YELLOW);
		
		return panel;
	}
	
	
	public static void main(String[] args){
		new CardExample().setVisible(true);
	}
}

 

[fetcher]

Vielen Dank für deine Hilfe. Werde ich gleich mal ausprobieren.