Problem mit setVisible(true)

[Gayson]

Hallo!
Ich habe folgende Quellcode, doch leider kommt das Programm nicht an setVisible(true) vorbei, er spuckt zwar Hi1 aus, leider aber nicht Hi2. Wenn ich setVisible(true) ganz rausnehme zeigt er Hi1 und Hi2 an, aber natürlich nicht die GUI.
Keine Ahnung wo der Fehler liegen könnte...

import java.net.*;
import java.io.*;
import java.util.*;
import org.jdom.*;
import org.jdom.input.*;
import org.*;
import java.util.List;


public class MitarbeiterEigenschaften extends JDialog{
	JTextField nameTextfeld = new JTextField();
	JTextField stundenTextfeld = new JTextField();
	JTextField kuerzelTextfeld = new JTextField();
	JLabel label1 = new JLabel("Name:");
	JLabel label2 = new JLabel("Kürzel:");
	JLabel label3 = new JLabel("Stunden pro Woche:");
	JButton save = new JButton("Speichern");
	JButton cancel = new JButton("Abbrechen");


    public MitarbeiterEigenschaften(String selectedMitarbeiter) {
		setModal(true); //Hauptfenster wird nicht bearbeitbar
        setSize(300, 500);
        setDefaultCloseOperation(WindowConstants.HIDE_ON_CLOSE);
		setLocation(
		        (Toolkit.getDefaultToolkit().getScreenSize().width-this.getSize().width) / 2,
		        (Toolkit.getDefaultToolkit().getScreenSize().height-this.getSize().height) / 2
		);

		JPanel pane = new JPanel();
		GridLayout grid= new GridLayout(4,2, 10, 10);
		pane.setLayout(grid);

		pane.add(label1);
		pane.add(nameTextfeld);
		pane.add(label2);
		pane.add(stundenTextfeld);
		pane.add(label3);
		pane.add(kuerzelTextfeld);
		pane.add(save);
		pane.add(cancel);


		setContentPane(pane);
		pack();
		System.out.println("Hi1");
        setVisible(true);
		System.out.println("Hi2");

    }

}

 

[L-ectron-X]

Es fehlen die Import-Anweisungen für die Pakete javax.swing.* und java.awt.* , ohne die ich einen Kompilerfehler erhielt.
"Hi2" wird erst nach der Rückkehr aus der Methode setVisible(true) ausgegeben, die in Deinem Dialog so lange aktiv ist, bis das Dialogfenster unsichtbar gemacht bzw. beendet wird.

 

[Beni]

Das liegt daran, dass der Dialog modal ist (und das ist kein Fehler, sondern ist sogar extra so gemacht).

Das Programm arbeitet weiter, sobald der Dialog wieder geschlossen ist.

Du könntest den Dialog auch nicht-modal machen, dann würde das Progi sofort weiterarbeiten.
Oder du machst zusätzliche Threads, dann arbeitet das Progi "gleichzeitig" an mehreren Fronten.

Kommt ganz darauf an, was du damit machen willst.

 

[Gayson]

Es fehlen die Import-Anweisungen für die Pakete javax.swing.* und java.awt.* , ohne die ich einen Kompilerfehler erhielt.
"Hi2" wird erst nach der Rückkehr aus der Methode setVisible(true) ausgegeben, die in Deinem Dialog so lange aktiv ist, bis das Dialogfenster unsichtbar gemacht bzw. beendet wird.

Hast Recht, habe nur vergessen, die beiden Pakete mitzukopieren, in meinem Originalcode sind die natürlich drin.

@ Beni: Stimmt, wenn ich setModal ganz rausnehme, erscheinen Hi1 und Hi2, nur der JDialog schließt sich sofort wieder...
Könntest du mal so nett sein, das setModal in einen Thread zu packen, damit ich weiß, wie es geht?
Werd mir gleich dann noch das ganze zu Threads bei Java ist eine Insel durchlesen.
Und muss ich dann noch anderes als setModal in einen Thread packen, oder reicht das?

 

[Gayson]

Mh, ein einfaches einfügen von

		new Thread(){
			public void run(){
				setModal(true); //Hauptfenster wird nicht bearbeitbar
			}
		}.start();

bringt nichts.

 

[Gayson]

aber ein

		new Thread(){
			public void run(){
				setVisible(true);
			}
		}.start();

hilft!

Danke Jungs!!

 

[Beni]

Moment, was willst du machen?

mit "setModal" wird einfach ein boolean umgeschaltet, das ist egal, welcher Thread das macht.
und "setVisible" hält den Thread an, der die Methode aufruft.

Wenn du jetzt etwas hast, das immer laufen soll (z.B. eine Animation), dann musst du dieses etwas in den neuen Thread packen.

Aber eben: was willst du machen? (So wie dein Code aussieht, scheint es ja schon fast gut zu sein, dass der Dialog blockiert?)

 

[Gayson]

Dieser JDialog bietet die Möglichkeit, Daten in einer XML-Datei zu ändern.
Konkreter: In meiner Haupt-GUI gibt es einen Eigenschaften-Button, der beim draufklicken diesen Dialog aufruft, um Daten (in diesem Fall Name, Kürzel, Stunden pro Woche) über einen Mitarbeiter zu ändern. Um diesen Dialog zu verlassen, gibt es entweder Abbrechen (this.dispose()) oder Speichern (speichert geänderte Werte in einer XML-Datei).

 

[Beni]

Aber da macht es doch Sinn, das Prog zu blockieren? Vielleicht nützt es, das "setVisible" nicht im Konstruktor des Dialogs aufzurufen, sondern erst später.

 

[Gayson]

Ich habe jetzt in meiner Hauptklassen, wo der ActionListener des Eigenschaften-Buttons liegt, folgendes geschrieben:

					MitarbeiterEigenschaften mitsettings = new MitarbeiterEigenschaften(selectedMitarbeiter);
					mitsettings.setVisible(true);

Und jetzt funzt es auch ohne Threads!

Ist das deiner Meinung nach die beste Lösung?

 

[Beni]

Jedenfalls besser als extra ein Thread machen :lol: