ActionListener Abhängigkeitenproblem

[vanBraQ]

Hi Leute,
habe mal wieder ein wohl etwas größeres Problem :/

Folgendes:
Habe eine MainFrame Klasse, in der das Frame gebuildet wird. Weiters habe ich eine ChangeMainFramePanel Klasse, in der ich auf das MainFrame die jeweilig richtigen Komponenten anzeige.
Dazu habe ich 4 verschiedene Panel Klassen: Header, Default, Kalender, Timetable.
In der Header Klasse werden die Toolbar usw auf das Frame getan, unteranderem gibt es dort 2 Buttons, die mir das umschalten auf die 2 Panels Kalender und Timetable ermöglichen sollen.
Wo genau muss ich jetzt den ActionListener implementieren ?
In der MainFrame Klasse oder in der HeaderPanel Klasse oder gehört der Code überhaupt ganz wo anders hin ?

Hier noch die Codes:

import javax.swing.*;

@SuppressWarnings("serial")
public class MainFrame extends JFrame{
	
	public MainFrame(){
		MainFramePanelChange.getParent(this);
		MainFramePanelChange.setDefaultPanel();
	}
	
	public static void main(String[] args) {	
		BuildMainFrameMainParts buildparts = new BuildMainFrameMainParts();
		buildparts.buildLookAndFeel();
		MainFrame frame = new MainFrame();
		buildparts.buildProperties(frame);
		LoginDialog dialog = new LoginDialog(frame);
	}

}

import java.awt.BorderLayout;

import javax.swing.JFrame;
import javax.swing.JPanel;


public class MainFramePanelChange {

	private  static CalendarPanel calendarPanel = new CalendarPanel();
	private  static DefaultPanel defaultPanel = new DefaultPanel();
	private  static TimetablePanel timetablePanel = new TimetablePanel();
	private  static HeaderPanel headerpanel = new HeaderPanel();
	private  static JFrame parent;
	
	public static void getParent(JFrame frame){
		parent = frame;
	}
	
	public static void setCalendarPanel() {
		JPanel defaultheader = new JPanel(new BorderLayout());
		parent.getContentPane().removeAll();
		defaultheader.add(headerpanel, BorderLayout.NORTH);
		defaultheader.add(calendarPanel, BorderLayout.CENTER);
		parent.getContentPane().add(defaultheader);
	}
	
	public static void setDefaultPanel() {
		JPanel defaultheader = new JPanel(new BorderLayout());
		parent.getContentPane().removeAll();
		defaultheader.add(headerpanel, BorderLayout.NORTH);
		defaultheader.add(defaultPanel, BorderLayout.CENTER);
		parent.getContentPane().add(defaultheader);
	}
	
	public static void setTimetablePanel() {
		JPanel defaultheader = new JPanel(new BorderLayout());
		parent.getContentPane().removeAll();
		defaultheader.add(headerpanel, BorderLayout.NORTH);
		defaultheader.add(timetablePanel, BorderLayout.CENTER);
		parent.getContentPane().add(defaultheader);
	}
}

import javax.swing.*;

import java.awt.*;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;


@SuppressWarnings("serial")
public class HeaderPanel extends JPanel implements ActionListener {
	
	private BuildMainFrameActions action = new BuildMainFrameActions();
	
	private JMenuBar menubar;
	private JToolBar toolbar1;
	private JMenu dateinmenu, hilfemenu;
	private JButton timetableButton, calendarButton;
	final Icon calendaricon = new ImageIcon(MainFrame.class.getResource("/images/calendar_20x20.jpg"));
	
	public HeaderPanel(){
		this.setLayout(new GridLayout(2,1));
		
		menubar = new JMenuBar();
		menubar.setBackground(new Color(0xA5D3DF));
		toolbar1 = new JToolBar();
		toolbar1.setRollover(true);
		toolbar1.setFloatable(false); 
		toolbar1.setBackground(new Color(0xA5D3DF));
		
		timetableButton = new JButton("Stundenplan");
	
		timetableButton.addActionListener(this);
		timetableButton.setIcon(calendaricon);
		timetableButton.setBackground(new Color(0xA5D3DF));
		calendarButton = new JButton("Kalender");
		calendarButton.addActionListener(this);
		calendarButton.setIcon(calendaricon);
		calendarButton.setBackground(new Color(0xA5D3DF));
		dateinmenu = new JMenu("Datei");
		hilfemenu = new JMenu("Hilfe");
		
		dateinmenu.add(action.openAction()); //OpenAction aus BuildMainFrameActions wird geadded
		dateinmenu.addSeparator();
		dateinmenu.add(action.exitAction()); //ExitAction aus BuildMainFrameActions wird geadded
		
		menubar.add(dateinmenu);
		menubar.add(hilfemenu);
		toolbar1.add(calendarButton);
		toolbar1.addSeparator();
		toolbar1.add(timetableButton);
		toolbar1.addSeparator();
		toolbar1.setBorder(BorderFactory.createRaisedBevelBorder());
		
		this.add(menubar);
		this.add(toolbar1);
		this.setVisible(true);
	}


        //Hier habe ich aus eigener Überlegung mal die Actions gemacht aber bin mir nicht so sicher ob das stimmt ...
	@Override
	public void actionPerformed(ActionEvent e) {
		if(e.getSource() == calendarButton){
			MainFramePanelChange.setCalendarPanel();
		}
		if(e.getSource() == timetableButton){
			MainFramePanelChange.setTimetablePanel();
		}
		
	}

}

 

[bERt0r]

Wo genau muss ich jetzt den ActionListener implementieren ?

Das bleibt dir überlassen. Im wesentlichen gibt es 3 empfehlenswerte Möglichkeiten:
Den Actionlistener in eine separate Datei implementieren.

import java.awt.event.ActionListener;
import java.awt.event.ActionEvent;
public class MyActionListener implements ActionListener
{
@Override
public void actionPerformed(ActionEvent e)
{
}
}
MyActionListener al=new MyActionListener();

Den Actionlistener als annonyme Klasse implementieren.

JButton button=new JButton();
ActionListener al=new ActionListener()
{
@Overrides
public void actionPerformed(ActionEvent e)
{
}
};
button.addActionListener(al);

Den Actionlistener als innere Klasse implementieren.

class MyActionListener implements ActionListener
{
@Override
public void actionPerformed(ActionEvent e)
{
}
}
//Methode in der du deine GUI zusammenbaust
JButton button=new JButton();
MyActionListener al=new MyActionListener();
button.addActionListener(al);

 

[vanBraQ]

Das bleibt dir überlassen. Im wesentlichen gibt es 3 empfehlenswerte Möglichkeiten:
Den Actionlistener in eine separate Datei implementieren.

Den Actionlistener als annonyme Klasse implementieren.

Den Actionlistener als innere Klasse implementieren.

Diese 3 Möglichkeiten sind wusste ich ja schon das Problem ist nun:

Um das Panel zu verändern rufe ich im ChangeMainFramePanel folgende Methode auf:

public static void setCalendarPanel() {
		JPanel defaultheader = new JPanel(new BorderLayout());
		parent.getContentPane().removeAll();
		defaultheader.add(headerpanel, BorderLayout.NORTH);
		defaultheader.add(calendarPanel, BorderLayout.CENTER);
		parent.getContentPane().add(defaultheader);
	}

parent ist in diesem Fall das MainFrame welches ich mir mit foglender Methode im ChangeMainFramePanel hole:

public static void getParent(JFrame frame){
		parent = frame;
	}

und im MainFrame aufrufe:

MainFramePanelChange.getParent(this);

Den ActionListener habe ich nun im HeaderPanel implementiert:

public void actionPerformed(ActionEvent e) {
		if(e.getSource() == calendarButton){
			MainFramePanelChange.setCalendarPanel();
		}
		if(e.getSource() == timetableButton){
			MainFramePanelChange.setTimetablePanel();
		}
		
	}

Das Problem derzeit ist, wenn ich auf den Button klicke hängt sich das gesamte Programm auf.
Ich habe schon versucht mittels Debug das Programm durchzugehen aber bin auch auf keinen ersichtlichen Fehler gestoßen.
Wo könnte der Fehler nun liegen ?

 

[SlaterB]

bei jeder GUI-Änderung musst du validate(); + repaint(); am JFrame aufrufen, damit das Layout neuberechnet wird
(oder dich genauer dazu schlau machen, gibt sicher auch etwas andere Möglichkeiten)

kopiere das aber besser nicht auch noch in drei Methoden hinein,
erstelle lieber eine Methode mit Parameter und rufe diese von den drei Methoden auf,
nur defaultPanel usw. ändert sich als Parameter für die neue 4. Methode,


das Programm hängt sich übrigens nicht auf, aber es ist noch das alte Bild zu sehen,
was keine Funktion mehr hat, Buttons reagieren nicht mehr, weil entfernt, neue noch nicht an ihrem Platz,
ein Screenshot quasi

 

[vanny]

Ich würde konzeptionell eher zum CardLayout tendieren, das ist für diesen Zweck da.

Gruß Vanny

 

[vanBraQ]

Funktioniert Tatsächlich
Danke vielmals

 

[bERt0r]

validate(); + repaint();

oder revalidate();