Dialog schliesst sich nicht. Warum?

[Kaladial]

Moin

als erstes wünsch ich euch mal allen nen frohes neues !!!
so und nun zu meinem problem...
ich hatte schon mal nen thread erstellt hab aber leider noch keine passende antwort erhalten, also versuch ich es nochmal...

also ich habe einen JDialog den ich über den aufruf aufrufe...
den JDialog hab ich auch entsprechend gepostet und mein programm geht auch zur richtigen zeit in die Funktion closeWindow()... das problem: er schliesst den dialog nicht automatisch...

mich würde interesieren warum er den nicht schliesst...
wenn ich z.b. dialoge über buttons schliesse funktioniert dispose() immer... nur bei dem automatischen schliessen geht das irgendwie nicht...

thx für eure hilfe
mfg Kala

Aufruf:

DlgComStart dlg = new DlgComStart(this, "Starte Com-Port-Verbindung"); 
dlg.setTitle("Starte Com-Port-Verbindung"); 
dlg.setModal(true); 
dlg.setVisible(true);

DlgComStart:

package ScanControl; 

import javax.swing.JPanel; 

import java.awt.Dimension; 
import java.awt.Frame; 
import java.awt.BorderLayout; 
import java.awt.Rectangle; 
import java.awt.Toolkit; 

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

public class DlgComStart extends JDialog { 

   private static final long serialVersionUID = 1L; 
   private JPanel jContentPane = null; 
   private JLabel jLabelMeldung = null; 
   String meldung=""; 
   /** 
    * @param owner 
    */ 
   public DlgComStart(Frame owner, String meldung) { 
      super(owner); 
      this.meldung=meldung; 
      initialize(); 
   } 

   /** 
    * This method initializes this 
    * 
    * @return void 
    */ 
   private void initialize() { 
      final Dimension screenDimension = Toolkit.getDefaultToolkit().getScreenSize(); 
      final int x_pos = (int) (screenDimension.getWidth() / 2.0f - 400.0f / 2.0f); 
      final int y_pos = (int) ((screenDimension.getHeight() / 2.0f - 200.0f / 2.0f)-100); 

      //this.setBounds(new Rectangle(0, 0, 400, 300)); 
      this.setBounds(new Rectangle(x_pos, y_pos, 400, 200)); 
      this.setResizable(false); 
      this.setContentPane(getJContentPane()); 
      jLabelMeldung.setText(""+meldung); 
       
      do{}while(!ScanControl.sc_global.stop_all); 
      closeWindow(); 
   } 

   private void closeWindow(){ 
      System.out.println("close"); 
      dispose(); 
   } 
    
   /** 
    * This method initializes jContentPane 
    * 
    * @return javax.swing.JPanel 
    */ 
   private JPanel getJContentPane() { 
      if (jContentPane == null) { 
         jLabelMeldung = new JLabel(); 
         jLabelMeldung.setText(""); 
         jLabelMeldung.setHorizontalAlignment(JLabel.CENTER); 
         jLabelMeldung.setBounds(new Rectangle(20, 50, 350, 20)); 
         jContentPane = new JPanel(); 
         jContentPane.setLayout(new BorderLayout()); 
         jContentPane.add(jLabelMeldung, BorderLayout.NORTH); 
      } 
      return jContentPane; 
   } 

}

 

[mimo]

Versuch mal:

super(Frame, "Titel", true);

 

[The_S]

Geht vermutlich darum http://www.java-forum.org/de/viewtopic.php?p=367947 !?

Naja, hast jetzt lang genug geschmort ... will mal nicht so sein:

Hoi,

ich bin aber gern empfindlich . Ne sry, aber des kann ich net ham, wenn ma Hilfesuchender ist, und sich dann so anstellt.

Fehler: Im Konstruktor wird die initialize Methode aufgerufen. Diese ruft am Ende das windowClose auf. Wenn der Konstruktor aber noch nicht durchgelaufen ist, KANN der Dialog noch gar nicht geöffnet sein. Deshalb wird der Dialog wieder disposed, bevor er überhaupt geöffnet wurde => Wirkungslos. Und irgendwann, nachdem der Konstruktor (und somit auch das dispose) durchlaufen wurde, wird der JDialog erst Visible gesetzt.

Seine "Warten"-Schleife scheint vermutlich von Anfang an auf true zu stehen, da der Dialog scheinbar nicht zeitversetzt geöffnet wird (zumindest merkt er das nicht), und er mit einer schleife ohne sleep alles andere blockieren und somit 100%CPU auslast erzeugen würde.

lg

Frohes Neues ...

 

[Kaladial]

@hobbit:
hmmm also ScanControl.sc_global.stop_all auf die ich mich in der do while schleife bezieh wird folgendermasen gesetzt:

                public boolean stop_all=false;
	public synchronized void removeThread(ScannerThread thread) { 
		threads.remove(thread); 
		if (threads.size() == 0) { 
			this.notify(); 
	            stop_all=true;
		} 
	}

also erst wenn alle threads die laufen geschlossen sind wird die variable gesetzt...

also ma ganz kurz die erläuterung wie das laufen sollte:

- ich öffne ne com-verbindung
- ich verändere über einen befehl die baudrate des gerätes das an den com angeschlossen ist
- ich beende die com-verbindung -> setzte die variable auf true so das er aus der warteschleife raus geht.....
- ich erstelle neue comverbindung zu dem gerät mit der nun schnelleren baudrate...

 

[The_S]

Lass uns eins nach dem Anderen machen, so kommt man besser ans Ziel, als wenn man alles auf einmal macht .

Hast du denn verstanden, warum dein Dialog nicht geschlossen wird?

 

[Kaladial]

also nach deiner aussage: weil der sich schliesst bevor er offen ist bzw die methode close aufgerufen wird bevor der dialog überhaupt sichtbar ist

 

[The_S]

ja ... dann besser das erstmal aus un dann reden wir weiter

 

[Kaladial]

hmmm nun dann erklär mir mal wie ich das beheben kann ...

 

[The_S]

Indem du diese beiden Codezeilen

      do{}while(!ScanControl.sc_global.stop_all);
      closeWindow();

nicht im Konstruktor bzw. einer Methode, die vom Konstruktor aufgerufen wird einfügst, sondern erst ausführst, wenn dein Dialog sichtbar ist.

 

[Kaladial]

hmmm und wo soll ich die dann einfügen?
steh jetzt irgendwie auf dem schlauch ...

 

[The_S]

ich weiß nicht, ich kenn die aufrufende Klasse nicht. Aber du wirst ja irgendwo ein "deinDialog.setVisible(true)" haben!? Diese beiden Codezeilen lagerst du dann in eine extra, public Methode deiner DlgComStart-Klasse aus, und rufst selbige auf, nachdem du den Dialog visible gesetzt hast. Alternativ kannst du auch in deinem Dialog die setVisible-Methode überschreiben, dort dann die dazugehörige Methode aus der Elternklasse aufrufen, und anschließend deine beiden Codezeilen in diese Methode einfügen.

 

[Kaladial]

ok geht ..