wie Jframe an ActionListener übergeben

[Lirave]

Hallo

ich habe in einer Klasse ein Frame erstellt mit Menü.
Je nachdem welcher Eintrag im Menu geklickt wird, soll ein anderes Panel dem Frame hinzugefügt werden.

Speziell soll es einmal ein 9x9 Sudokufeld sein, und ein anderes mal ein 16x16 Sudokufeld.

Der "Ordnung" halber habe ich alle Events in einer anderen Klasse ausgelagert.

Nur wie kann ich der der Klasse "Events" sagen was frame1 ist ?

Danke schon mal für jede Hilfestellung.

import javax.swing.*;


public class Frame {

	public static void main(String[] args) {
		JFrame frame1 = new JFrame("Sudoku Solver");
		frame1.setSize(500,500);
		frame1.setResizable(false);
		frame1.setLocationRelativeTo(null);  	      
		frame1.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
				
		JMenuBar menubar = new JMenuBar();
		frame1.setJMenuBar(menubar);
		
		JMenu programm = new JMenu("Programm");
		menubar.add(programm);
		
		JMenuItem exit = new JMenuItem("Beenden");
		programm.add(exit);
		exit.addActionListener(new Events.exitaction());
		
		JMenu varianten = new JMenu("Varianten");
		menubar.add(varianten);	
		
		JMenu sudokun = new JMenu("Sudoku - normal");
		JMenuItem sudokun9mal9 = new JMenuItem("9x9 Sudokufeld");
		JMenuItem sudokun16mal16 = new JMenuItem("16x16 Sudokufeld");		
		sudokun.add(sudokun9mal9);
		sudokun9mal9.addActionListener(new Events.sudokun9mal9());
		sudokun.add(sudokun16mal16);
		
		JMenuItem sudokuk = new JMenuItem("Killersudoku");		
		varianten.add(sudokun);
		varianten.add(sudokuk);

		frame1.setVisible(true);
	}
}

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

public class Events {
	
	static class exitaction implements ActionListener{
		public void actionPerformed (ActionEvent e){
			System.exit(0);
		}
	}
	
	static class sudokun9mal9 implements ActionListener{
		public void actionPerformed (ActionEvent e){
			JPanel panel = new JPanel();
			frame1.add(panel);                                          // hier ist das Problem
			JLabel label = new JLabel("test");
			panel.add(label);
		}
	}
}

 

[goeoe7]

du könntest zum Beispiel deiner Klasse sudokun9mal9 ein den JFrame als Attribut übergeben, sähe dann so aus:

static class sudokun9mal9 implements ActionListener{
  private JFrame frame1;

  public sudokun9mal9(JFrame frame) {
    this.frame1 = frame;
  }

Den Rest einfach lassen wies ist, dann müsstest du Problemlos auf frame1 zugreifen können
Hilft das?

 

[hdi]

Der "Ordnung" halber habe ich alle Events in einer anderen Klasse ausgelagert.

In deinem Fall finde ich das Auslagern Overkill. Deine Events sind ja nun nicht gerade komplex, ich finde mit anonymen Klassen ist das übersichtlicher und auch nicht so umständlich für den Frame-Zugriff.

// direkt in deiner main-Methode:
sudokun9mal9.addActionListener(new ActionListener(){
       @Override
       public void actionPerformed(ActionEvent e){
            frame1.add(...); // Achtung frame1 muss als final deklariert sein!
       }
});

Übrigens: Den kompletten Code der main-Methode solltest du in ein Runnable packen und dann per

EventQueue.invokeLater(deinRunnable);

ausführen.

 

[Lirave]

danke hat geklappt

nur noch eine Frage, in dem ActionListener erzeuge ich ein label und füge es dem panel und das dem frame hinzu.

Es wird aber nicht angezeigt, nur als ich ein erneutes "frame1.setVisible(true);" hinzugefügt habe.

Warum muss ich den Befehl erneut ausführen, der Frame wird doch schon angezeigt ?

 

[hdi]

Wenn du den Inhalt des Frames änderst während es angezeigt wird musst du nach Änderungen ein frame.validate() oder frame.revalidate() durchführen. Sollte dann klappen.