Fehler beim Aufrufen von Dialog aus ActionListener

[spacegaier]

Hallo zusammen,

ich habe wieder ein Problem. Ich habe ein Frame mit einem Menü. Dieses Menü soll jetzt auf Klicks reagieren. Dazu habe ActionListener eingebaut. Nun it aber das Problem, dass das Programm immer abstürzt (oder Eclipswe will zumindest in die Debug-Ansicht) wenn ich im ActionListener einen
Dialog als Reaktion aufrufen will. Das gleiche Proiblem hatte ich auch, wenn ich anstatt dem Dialog z.B. direkt einen neuen Eintrag in meine Taelle (welche in dem Frame auch liegt) machen möchte.

Ich poste jetzt mal den Code, den ich für relevant halte. Wenn ihr mehr braucht melden.

ActionListener (der prizipiell funktioniert, da er z.B. dsa System.exit(0); ausführt):
Das frame bei NewEntryDialog ist am Anfang der Klasse definiert: private MainFrame frame;

public void actionPerformed(ActionEvent evt)
	{
		String command = evt.getActionCommand();
		
		if(command.equals("NEW"))
			new NewEntryDialog(frame);

		else if(command.equals("EXIT"))
			System.exit(0);	
	}

Die Dialog-Klasse:

public class NewEntryDialog extends JDialog 
{
	private static final long serialVersionUID = 1L;
	

	public NewEntryDialog(MainFrame frame)
	{
		super(frame, "Neuen Eitnrag anlegen", false);
		this.setDefaultCloseOperation(EXIT_ON_CLOSE);
		
		JLabel titel = new JLabel("Titel");
		
		this.add(titel);
		this.pack();
		this.setVisible(true);
	}
}

 

[SlaterB]

Fehlermeldung?
vollständiges Programm (fehlt nur noch einfaches JFrame mit Button)?

 

[spacegaier]

Falsch ausgedrückt: Das Programm läuft weiter, aber in Eclipse kommt die Warnung/Fehler Source not found und der Reiter heißt EventDispatchThread.run() line: not available

 

[spacegaier]

Was meinst du mit vollständiges Programm? Den ganzen Code?

 

[SlaterB]

etwas, was man kopieren und starten kann,

möglichst nur diesen Code den du schon gepostet hast,
aber eben noch das notwende JFrame drumherum + Button + main-Operation usw.,
ein kleines Testprogramm

 

[spacegaier]

Achso...okay...no problem.

NewEntryDialog.java:

import javax.swing.JDialog;
import javax.swing.JLabel;

public class NewEntryDialog extends JDialog 
{
	private static final long serialVersionUID = 1L;
	

	public NewEntryDialog(MainFrame frame)
	{
		super(frame, "Neuen Eitnrag anlegen", false);
		this.setDefaultCloseOperation(EXIT_ON_CLOSE);
		
		JLabel titel = new JLabel("Titel");
		
		this.add(titel);
		this.pack();
		this.setVisible(true);
	}
}

MainMenu.java:

import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.awt.event.KeyEvent;
import javax.swing.JMenu;
import javax.swing.JMenuBar;
import javax.swing.JMenuItem;
import javax.swing.KeyStroke;

import smdb.gui.NewEntryDialog;

public class MainMenu extends JMenuBar implements ActionListener
{
	private static final long serialVersionUID = 1L;
	
	private MainFrame frame;
	
	public MainMenu(MainFrame frame)
	{
		JMenu file 	= new JMenu("Datei");
		JMenu edit 	= new JMenu("Edit");
		JMenu help	= new JMenu("Hilfe");
		
		JMenuItem open 	= new JMenuItem("Datenbank öffnen");
		open.setActionCommand(MainFrame.EActions.OPEN.name());
		open.addActionListener(this);
		JMenuItem close	= new JMenuItem("Datenbank schließen");
		close.setActionCommand(MainFrame.EActions.CLOSE.name());
		close.addActionListener(this);
		JMenuItem exit 	= new JMenuItem("Programm verlassen");
		exit.setActionCommand(MainFrame.EActions.EXIT.name());
		exit.addActionListener(this);
		exit.setAccelerator(KeyStroke.getKeyStroke(KeyEvent.VK_Q, ActionEvent.CTRL_MASK));
		
		JMenuItem new_entry 	= new JMenuItem("Neuer Eintrag");
		new_entry.setActionCommand(MainFrame.EActions.NEW.name());
		new_entry.addActionListener(this);
		new_entry.setAccelerator(KeyStroke.getKeyStroke(KeyEvent.VK_N, ActionEvent.CTRL_MASK));
		JMenuItem delete_entry 	= new JMenuItem("Eintrag löschen");
		delete_entry.setActionCommand(MainFrame.EActions.DELETE.name());
		delete_entry.addActionListener(this);
		delete_entry.setAccelerator(KeyStroke.getKeyStroke(KeyEvent.VK_L, ActionEvent.CTRL_MASK));
		JMenuItem edit_entry 	= new JMenuItem("Eintrag berarbeiten");
		edit_entry.setActionCommand(MainFrame.EActions.EDIT.name());
		edit_entry.addActionListener(this);
		edit_entry.setAccelerator(KeyStroke.getKeyStroke(KeyEvent.VK_B, ActionEvent.CTRL_MASK));
		
		JMenuItem info = new JMenuItem("Info");
		info.setActionCommand(MainFrame.EActions.INFO.name());
		info.addActionListener(this);
		
		file.add(open);
		file.add(close);
		file.addSeparator();
		file.add(exit);
		
		edit.add(new_entry);
		edit.add(delete_entry);
		edit.add(edit_entry);
		
		help.add(info);
		
		add(file);
		add(edit);
		add(help);
		
		
	}
	
	public void actionPerformed(ActionEvent evt)
	{
		String command = evt.getActionCommand();
		
		if(command.equals("NEW"))
			new NewEntryDialog(frame);

		else if(command.equals("EXIT"))
			System.exit(0);	
	}

}

MainFrame.java:

import java.awt.Insets;
import java.awt.Toolkit;
import javax.swing.JFrame;

import smdb.gui.MainMenu;

public class MainFrame extends JFrame
{
	private static final long serialVersionUID = 1L;
	private MainMenu mainMenu;
	public enum EActions
	{
		OPEN,
		CLOSE,
		EXIT,
		NEW,
		DELETE,
		EDIT,
		INFO;
	}	
	
	public MainFrame()
	{
		this.setTitle("SheetMusicDataBase 1.0");
		this.setDefaultCloseOperation(EXIT_ON_CLOSE);
			
		Insets inset = Toolkit.getDefaultToolkit().getScreenInsets(this.getGraphicsConfiguration());
				
		int screenWidth  = Toolkit.getDefaultToolkit().getScreenSize().width;
		int screenHeight = Toolkit.getDefaultToolkit().getScreenSize().height;
		
		int sizeW = screenWidth  - (inset.left + inset.right);
		int sizeH = screenHeight - (inset.top  + inset.bottom);
		
		this.setSize(sizeW, sizeH);
		this.setLocation(0,0);
				
		mainMenu = new MainMenu(this);
		this.setJMenuBar(mainMenu);
				
		this.setVisible(true);
	}
}

Main.java:

import smdb.gui.MainFrame;

public class Main 
{
	public static void main(String[] args) 
	{
		new MainFrame();
	}

}

Alles mit import smdb.* sind meine eigenen packages.

Hoffe das hilft weiter.

 

[SlaterB]

das nennst du ein kurzes Programm?
ist es dir nicht möglich, z.B. alle anderen MenuItems zu entfernen?
verstehe ich nie..

naja, ich hab zu Hause nur Java 1.4, da musste ich eh mindestens die Enums entfernen, habe es dann gleich gekürzt auf

public class MainFrame extends JFrame {
	private MainMenu mainMenu;

	public MainFrame() {

		setDefaultCloseOperation(EXIT_ON_CLOSE);
		mainMenu = new MainMenu(this);
		this.setJMenuBar(mainMenu);
		setSize(400, 400);
		setVisible(true);
	}
	public static void main(String[] args) {
		new MainFrame();
	}
}

class MainMenu extends JMenuBar implements ActionListener {
	private static final long serialVersionUID = 1L;

	private MainFrame frame;

	public MainMenu(MainFrame frame) {
		JMenu edit = new JMenu("Edit");

		JMenuItem new_entry = new JMenuItem("Neuer Eintrag");
		new_entry.setActionCommand("NEW");
		new_entry.addActionListener(this);
		new_entry.setAccelerator(
			KeyStroke.getKeyStroke(KeyEvent.VK_N, ActionEvent.CTRL_MASK));

		edit.add(new_entry);
		add(edit);
	}

	public void actionPerformed(ActionEvent evt) {
		String command = evt.getActionCommand();

		if (command.equals("NEW")) {
			new NewEntryDialog(frame);
		}
	}
}

class NewEntryDialog extends JDialog {
	private static final long serialVersionUID = 1L;

	public NewEntryDialog(MainFrame frame) {
		super(frame, "Neuen Eitnrag anlegen", false);
		setDefaultCloseOperation(EXIT_ON_CLOSE);

		JLabel titel = new JLabel("Titel");

		getContentPane().add(titel);
		pack();
		setVisible(true);
	}
}

in dem Dialog musst ich
getContentPane().add(titel);
statt
add(titel)
setzen, was bei alter Java-Version typisch ist, sonst kein Fehler zu erkennen

funktioniert die gekürzte Version bei dir, deine aber immer noch nicht?
keine Fehlermeldung?
starte das Programm von der Konsole, dann kann kein Debug-Modus als Ausrede kommen

 

[spacegaier]

Das mit von der Konsole starten habe ich noch nie hinbekommen . Wäre net, wenn das mal jemand anderes hier machen könnte.

Wenn ich es aber in eine .exe umwandeln lasse und diese ausführe kann ich so oft ich will auf den Menüeintrag klicken, es passiert nix. Ebenso asl .jar.

 

[SlaterB]

tritt denn nun der Fehler auch in meiner Kurzversion auf?
vielleicht liegt er ja ganz woanders

 

[spacegaier]

Ja, ich habe gerade deine Version in .jar un .xe umgesetz und probiert (meine lange habe ich so noch gar nicht getestet).

Ich habe jetzt noch von Eclipse einen Screenshot gemacht, so wie es aussieht, wenn ich gerade vorher auf den besagten Menüpunkt geklickt habe und ich automatisch wieder in der IDE lande. Wie man sieht läuft mein Javaprogramm aber weiter (siehe Taskleiste).
Außerdem ist mir nun aufgefallen, dass er doch einen Fehler ausgibt und zwar unten im Debugfenster (ist mir vorher gar nicht aufgefallen; ich arbeite erst seit vier Tagen mit Java und somit auf mit Eclipse).
Dort steht folgendes (falls man es auf dem Screenshot nicht erkennen kann:

• Thread [AWT-EventQueue-0] (Suspended (exception IllegalArgumentException))
  EventDispatchThread.run() line: not available

 

[spacegaier]

Und noch ein Fehler.....Aus der Konsolenausgabe....

• Exception in thread "AWT-EventQueue-0" java.lang.IllegalArgumentException: defaultCloseOperation must be one of: DO_NOTHING_ON_CLOSE, HIDE_ON_CLOSE, or DISPOSE_ON_CLOSE
  at javax.swing.JDialog.setDefaultCloseOperation(Unknown Source)
  at NewEntryDialog.<init>(MainFrame.java:57)
  at MainMenu.actionPerformed(MainFrame.java:47)
  at javax.swing.AbstractButton.fireActionPerformed(Unknown Source)
  at javax.swing.AbstractButton$Handler.actionPerformed(Unknown Source)
  at javax.swing.DefaultButtonModel.fireActionPerformed(Unknown Source)
  at javax.swing.DefaultButtonModel.setPressed(Unknown Source)

 

[Wildcard]

Wie könnte eine Fehlermeldung noch eindeutiger sein?

 

[spacegaier]

Okay...es lag an folgender Zeile:

this.setDefaultCloseOperation(EXIT_ON_CLOSE);

Einfach auskommentieren (in der Dialogklasse natürlch, nicht in der Frameklasse).

DANKE, für all eure Mühen !!! :toll: