Swing JDialog liefert immer false, 2 JListen verknüpfen

[raffi]

Hallo, bin gerade dabei ein Programm zu schreiben, bei dem über Dialoge hinzugefügt bzw geändert werden kann.
Für die Dialoge habe ich mir eine BaseDialog-Klasse als Grundgerüst geschrieben, jene liefert mir aber immer nur true respektive false zurück, je nachdem wie ich die boolean Variable im Konsruktor des BaseDialog vereinbart habe.

public class BaseDialog extends JDialog {

	JFrame frame;
	JPanel buttonContainer;
	JPanel mainContainer;
	JButton okButton;
	JButton cancelButton;
	GridBagHelper gbh;
	AbstractAction closeAction;
	boolean ok;
	
	public BaseDialog (Window owner, String title) {
		super(owner, title);
		frame = new JFrame();
		frame.setDefaultCloseOperation(JFrame.DO_NOTHING_ON_CLOSE);
		frame.setDefaultLookAndFeelDecorated(true);
		frame.setLayout(new BorderLayout());
		
		ok = false;
		okButton = new JButton("OK");
		okButton.addActionListener(new CloseAction(true));
		cancelButton = new JButton("Cancel");
		cancelButton.addActionListener(new CloseAction(false));
		gbh = new GridBagHelper();
		gbh.setInsets(new Insets(3,3,3,3));
		buttonContainer = new JPanel();
		buttonContainer.setLayout(new GridBagLayout());
		mainContainer = new  JPanel();
		
		gbh.add(buttonContainer, okButton, 0, 0);
		gbh.add(buttonContainer, cancelButton, 0, 1);
		frame.add(buttonContainer, BorderLayout.SOUTH);
		frame.add(mainContainer, BorderLayout.NORTH);
		frame.pack();
		frame.setVisible(true);
	}
	
	class CloseAction extends AbstractAction {

		private boolean actionState;
		
		public CloseAction (boolean state) {
			this.actionState = state;
		}
		
		@Override
		public void actionPerformed(ActionEvent arg0) {
			ok = actionState;
			frame.setVisible(false);
		}
	}
	
	public JPanel getMainContainer () {
		return mainContainer;
	}
	
	public boolean exec () {
		frame.pack();
		frame.setVisible(true);
		return ok;
	}
}

Aufgerufen wird der BaseDialog dann in der Main-Klasse wie gehabt und die Methode exec ausgewertet.

Bei false ist das ersteinmal nicht so schlimm, da eben einfach nix passiert. Wenn ich jedoch ok testweise auf treu setze bringt er mir einen Fehler da ich in der Main-Klasse Felder auslese, diese aber beim Starten des Dialog logischerweise noch nicht gefüllt sind.

Das zweite Problem besteh darin, das es in meinem Programm um Pofessuren und Lehrveranstaltungen geht. Nun möchte ich die Professuren Liste mit der Veranstaltungsliste so verknüpfen, das bei Auswahl einer Profesur nur die zugehörigen Veranstaltungen in der Liste angezeigt werden. Ich hatte mir das so gedacht, das ich in der ProfModel-Klasse ein Veranstaltungsmodel erstelle und darüber dann agiere. Kann das so funktionieren wenn in der Main-Klasse jeweils eine JList für Professuren und Veranstaltungen vorhanden ist?

MfG

 

[Verjigorm]

Also erstens:
Der Code ist absolut grausam :autsch: Du hast eine Klasse die von JDialog erbt und baust darin einen JFrame :rtfm: Ich geh mal nicht weiter auf weitere 3-4 Grausamkeinten ein

und zweitens:
Wann genau rufst du exec() auf?

Generelle Lösung:
Mach einen RICHTIGEN JDialog und mach ihn modal, dann muss dein Hauptprogramm nach dem Öffnen des Dialogs warten, bis dieser geschlossen wurde.

edit:
hab dir mal was zusammenkopiert:

import java.awt.GridLayout;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import javax.swing.JButton;
import javax.swing.JDialog;

public class MyDialog extends JDialog 
{
	private boolean ok;
	private JButton cancelButton;
	private JButton okButton;
	
	public MyDialog()
	{
		cancelButton = new JButton("Cancel");
		cancelButton.addActionListener(new ActionListener() 
		{
			
			@Override
			public void actionPerformed(ActionEvent arg0) 
			{
				ok = false;
				dispose();
			}
		});
		
		okButton = new JButton("OK");
		okButton.addActionListener(new ActionListener() 
		{
			
			@Override
			public void actionPerformed(ActionEvent arg0) 
			{
				ok = true;
				dispose();
			}
		});
		
		setTitle("Modaler Dialog");
		setDefaultCloseOperation(DO_NOTHING_ON_CLOSE);
		setLocationRelativeTo(null);
		setLayout(new GridLayout());
		
		add(okButton);
		add(cancelButton);
		
		pack();
		setModal(true);
	}
	
	public boolean isOk()
	{
		return ok;
	}
}

public class ModalDialogTest {

	/**
	 * @param args
	 */
	public static void main(String[] args) 
	{
		MyDialog dia = new MyDialog();
		dia.setVisible(true); //Hier wird gewartet bis der Dialog "fertig" ist
		
		if(dia.isOk())
		{
			System.out.println("OK gedrückt");
                        //Hier könnte ich jetzt Werte vom Dialog abfragen und auswerten
		}	
		else
		{
			System.out.println("Cancel gedrückt");
		}
	}

}

 

[raffi]

Hallo und danke, das mit dem Frame war natürlich quatsch.
Ich habe das Problem aber nun gelöst, sieht zwar bestimmt immernoch bisschen schlimm aus aber gut.

public class BaseDialog extends JDialog {

	JPanel buttonContainer;
	JPanel mainContainer;
	GridBagHelper gbh;
	boolean ok;
	
	public BaseDialog (Window owner, String title) {
		super(owner, title);
		
		ok = false;
		setModal(true);
		setDefaultCloseOperation(DO_NOTHING_ON_CLOSE);
		gbh = new GridBagHelper();
		buttonContainer = new JPanel();
		buttonContainer.setLayout(new GridBagLayout());
		mainContainer = new  JPanel();
		mainContainer.setLayout(new GridBagLayout());
		gbh.setInsets(new Insets(3,3,3,3));
		JButton okButton = new JButton("OK");
		okButton.addActionListener(new CloseAction(true));
		JButton cancelButton = new JButton("Abbrechen");
		cancelButton.addActionListener(new CloseAction(false));
		
		gbh.add(buttonContainer, okButton, 0, 0);
		gbh.add(buttonContainer, cancelButton, 0, 1);
		add(buttonContainer, BorderLayout.SOUTH);
		add(mainContainer, BorderLayout.CENTER);
	}
	
	class CloseAction extends AbstractAction {

		private boolean actionState;
		
		public CloseAction (boolean state) {
			this.actionState = state;
		}
		
		@Override
		public void actionPerformed(ActionEvent arg0) {
			ok = actionState;
			setVisible(false);
		}
	}
	
	public JPanel getMainContainer () {
		return mainContainer;
	}
	
	public boolean exec () {
		pack();
		setVisible(true);
		return ok;
	}
}

Jetzt steht nur noch das zweite Problem im Raum. Und zwar wie ich die JListen verknüpfe das beim Selektieren einer Professur genau die zugehörigen Veranstaltungen in deren JList angezeigt werden. Ich hatte mir gedacht, das ich im ProfModel einfach ein VeranstModel anlege.

public class ProfModel extends AbstractListModel {

	ArrayList<Professur> pf;
	VeranstModel vModel;
	
	
	public ProfModel () {
		pf = new ArrayList<Professur>();
		vModel = new VeranstModel();
	}
	
	@Override
	public Professur getElementAt (int index) {
		return pf.get(index);
	}
	
	public void addElement (Professur prof) {
		pf.add(prof);
		fireIntervalAdded(pf, pf.size() - 1, pf.size() - 1);
	}
	
	public void deleteElement (int index) {
		pf.remove(index);
		fireIntervalRemoved(pf, index, index);
	}

	@Override
	public int getSize () {
		return pf.size();
	}
}

Kann das so funktionieren?

MfG

 

[raffi]

OK, ich probiers nochmal, hoffentlich antwortet jetzt wenigstens mal jemand

Also ich bin jetzt schon soweit, das ich die JListen hoffentlich richtig verknüpft habe. Ich habe das Model der Veranstaltungsliste in der Professurklasse erstellt. Das klappt auch soweit und die Daten werden auch vollständig und ordnungsgemäß abgespeichert. Die Listen sind also gefüllt, aber nur in der JList der Professur werden mir die Namen der Professuren angezeigt. Die Veranstatungsliste bleibt leer.

public class Professur {

	private String kuerzel;
	private String name;
	private String professor;
	private String mail;
	private int mAnz;
	VeranstModel vModel;

	public Professur (String kuerzel, String name, String professor, String mail, int mAnz) {
		this.kuerzel = kuerzel;
		this.name = name;
		this.professor = professor;
		this.mail = mail;
		this.mAnz = mAnz;
		vModel = new VeranstModel();
	}

	public String getKuerzel() {
		return kuerzel;
	}

	public void setKuerzel(String kuerzel) {
		this.kuerzel = kuerzel;
	}

	public String getName() {
		return name;
	}

	public void setName(String name) {
		this.name = name;
	}

	public String getProfessor() {
		return professor;
	}

	public void setProfessor(String professor) {
		this.professor = professor;
	}

	public String getMail() {
		return mail;
	}

	public void setMail(String mail) {
		this.mail = mail;
	}

	public int getmAnz() {
		return mAnz;
	}

	public void setmAnz(int mAnz) {
		this.mAnz = mAnz;
	}
	
	public Veranstaltung getVeranstatung (int index) {
		return vModel.getElementAt(index);
	}
	
	public void setVeranstaltung (Veranstaltung v) {
		vModel.addElement(v);
	}
	
	public int getVeranstSize () {
		return vModel.vs.size();
	}

	@Override
	public String toString() {
		return name;
	}
}

In der Main-Klasse sieht das ganze dann folgendermaßen aus:

JList veranstList;
...
public Main () {
...
		veranstList = new JList();
		veranstList.setPreferredSize(new Dimension(200, 250));
...
	}
...
class ProfSelectionListener implements ListSelectionListener {

		@Override
		public void valueChanged(ListSelectionEvent arg0) {
			int index = profList.getSelectedIndex();
			veranstList = new JList(prof.pf.get(index).vModel);
		}
	}

Sobald ich eine Professur anwähle müsste doch auch die veranstList gefüllt werden oder?