Swing Focus nach Beendigung eines Modal-JDialogs

[CeJay]

Hallo,

ich plage mich mit folgendem Problem:
Ich habe einen JDialog mit einem Knopf (StartFenster). Bei Aktivierung soll ein neues Fenster2 erzeugt werden. Weil das etwas dauern kann, möchte ich diesen Vorgang mit einer ProgressBar überwachen, der in einem modalen JDialog (Fortschrittsfenster) angezeigt wird.

Nun das Problem. Verwende ich jre1.6.0_17, dann ist der Fokus nach Schließen des Fortschrittsfensters auf dem StartFenster. Das neu erzeugte Fenster2 liegt dahinter.

Verwende ich jre1.6.0, dann klappt es und das Fenster2 liegt oben auf und hat den Fokus.

Ich vermute mal, dass ich irgendwas Im Zusammenhang mit Swing und Threads nicht verstanden habe... Vielleicht kann mir da jemand auf die Sprünge helfen?

Hier der vereinfachte Code StartFenster:

public class Fenster extends JDialog implements ActionListener {
   private JButton knopf;
   private JPanel rahmen;
   public Fenster(String title){
      super();
      setTitle(title);
      knopf=new JButton("Do it!");
      knopf.setBounds(10, 10, 80, 20);
      knopf.addActionListener(this);
      rahmen= new JPanel();
      rahmen.add(knopf);
      this.setBounds(50, 50, 350, 150);
      this.add(rahmen);
      this.setVisible(true);
   }

   @Override
   public void actionPerformed(ActionEvent e) {
      Thread job = new Thread(){
         public void run(){
            Fortschrittsanzeige.max=10;
            for (int i = 0; i<10; i++){
               try {
                  sleep(500);
               } catch (InterruptedException e) {
                  // TODO Auto-generated catch block
                  e.printStackTrace();
               }
               Fortschrittsanzeige.wert=i;   
            }
            new Fenster("Fenster2");
         }
      };
      new Fortschrittsanzeige(job,this);
   }  
}

Hier der Code meiner Fortschrittsanzeige:

public class Fortschrittsanzeige{
   
   public static Popup pop;
   public static JDialog rahmen;
   public static int max =0;
   public static int wert=0;

   private static Thread job_back;
   
   private JProgressBar progress;
   
   public Fortschrittsanzeige(final Thread job, final JDialog fenster){
         rahmen = new JDialog();
         rahmen.setTitle("Fortschritt");
         progress = new JProgressBar();
         System.out.println("Enter");
         job_back=job;
         job_back.start();
      if (max>0){
         progress.setMaximum(max);
         progress.setValue(wert);
         progress.setIndeterminate(false);
      } else {
         progress.setIndeterminate(true);
      }
      
      Thread checker = new Thread(){
         public void run(){
            while(job_back.getState()!=State.TERMINATED){
               if (max>0){
                  progress.setMaximum(max);
                  progress.setValue(wert);
                  progress.setIndeterminate(false);
               }
               
               try {
                  Thread.sleep(60);
               } catch (InterruptedException e) {
                  e.printStackTrace();
               }
            }
            rahmen.dispose();
         }
      };
      checker.start();      

      rahmen.add(progress);
      rahmen.setPreferredSize(fenster.getSize());
      rahmen.setBounds(fenster.getBounds());
      rahmen.setModal(true);
      rahmen.setVisible(true);
}
}

 

[kay73]

Hilft es, wenn Du auf Fenster2 toFront() und/oder setAutoRequestFocus() aufrufst?

 

[KrokoDiehl]

Swing-Objekten sollten immer in der AWT-EventQueue (EDT) modifizert werden. Dafür gibt es z.B.

SwingUtilities.invokeLater()

.

Ferner ist deine "Klasse" Fortschrittsanzeige ziemlich threadunfreundlich, weil sie viele statische Elemente hat.

Im ersten Schritt würde ich das Fortschrittsanzeige-Fenster mal völlig unstatisch machen und im Wartethread als lokales Objekt halten. Wenn du dort (im Wartethread) dann z.b. den Fortschritt aktualisieren willst, kannst du das im o.g.

invokeLater()

machen. Grob gesagt meine ich etws derartiges:

    public void actionPerformed(ActionEvent event) {
        Runnable runnable = new Runnable() {
            @Override
            public void run() {
                final int            max  = 10;
                final ProgressDialog prog = new ProgressDialog(Fenster.this, max);

                // Wartefenster anzeigen
                SwingUtilities.invokeAndWait(new Runnable() {
                    @Override
                    public void run() {
                        prog.setVisible(true);
                    }
                });
                
                // Warten ...
                for (int i = 0; i < max; i++) {
                    try {
                        Thread.currentThread().sleep(500L);
                    }
                    catch (InterruptedException iExc) {
                        iExc.printStackTrace();
                        break; // schleife abbrechen
                    }
                    
                    // Fortschrittsbalken erhöhen
                    SwingUtilities.invokeAndWait(new Runnable() {
                        @Override
                        public void run() {
                            prog.setProgress( prog.getProgess()+1 );
                        }
                    });
                }
                
                // Wartefenster schließen und neues Fenster2 anzeigen
                SwingUtilities.invokeAndWait(new Runnable() {
                    @Override
                    public void run() {
                        prog.setVisible(false);
                        prog.dispose();
                        new Fenster("Fenster2");
                    }
                });
            } //run()
        };

        this.getDoItButton().setEnabled(false);
        new Thread(runnable, "Warte auf Fenster2").start();
    }

Der Code ist nun sicher ein Extremfall. Ich kann auch nicht garantieren dass damit dein Fokusproblem behoben wird. Sicher muss dur dir auch über das Parenting der Fenster (auch Fenster1 und Fenster2) Gedanken machen und die Hinweise von kay73 beachten.
In jedem Fall aber ist das ein besserer Umgang mit Thread(s) und dem EDT.

 

[CeJay]

Vielen Dank für die ausführliche Hilfe.

Ich seh da aber immer noch ein Problem. Ich möchte doch eigentlich den Fortschrittsbalken anzeigen, während das Fenster2 erzeugt wird und nicht vorher! Mein Problem ist doch, dass der Vorgang des Erzeugens von Fenster2 eine ganze Weile dauern kann und ich dafür den Fortschrittsbalken brauche.

Oder hab ich da wieder was falsch verstanden?

Im Grunde hätte ich gerne eine Klasse, der ich (in einem Thread?) eine Aufgabe übergeben kann und die mir anzeigt, wie weit die Aufgabe fortgeschritten und wann sie beendet ist.

Und diese Aufgabe kann z.B. darin bestehen, ein Fenster zu erzeugen.

 

[KrokoDiehl]

Okay, du kannst natürlich das "new Fenster2()" dahin legen, wo du willst. Oder du nimmst einen SwingWorker, der ist vermutlich einfacher weil er die "process()"-Methode direkt im EDT ausführt. Das könnte etwa so aussehen:

public final class Fenster2Lader extends SwingWorker<Fenster2, String> {

    private final DialogMitProgresBar fortschritt;
    
    ...
    
    @Override
    protected Fenster2 doInBackground() throws Exception {
        this.publish("Lade Fenster...");
        Fenster2 fenster = new Fenster2();
        this.publish("Lade Gui...");
        fenster.initGui();
        this.publish("Lade Grafiken...");
        fenster.initImages();
        this.publish("Lade Daten...");
        fenster.initData();
        //...
        
        // fertig!
        SwingUtilities.invokeAndWait(new Runnable() {
            @Override
            public void run() {
                fortschritt.setVisible(false);
            }
        });
        return fenster;
    }
    
    @Override
    protected void process(List<String> chunks) {
        if ( ! chunks.isEmpty()) {
            fortschritt.setProgress(chunks.get( chunks.size()-1 ));
        }
        
        if ( ! fortschritt.isVisible()) {
            fortschritt.setVisible(true);
        }
    }
}

Eine andere Alternative ist auch der ProgressMonitor.
Ich hoffe beide Möglichkeiten helfen dir weiter.

 

[CeJay]

Servus,

ich hab jetzt schon einiges dazu gelernt und ausprobiert. An sich klappt das auch, aber das mit dem "Verlegen" von new Fenster2() vor "prog.dispose" bewirkt wiederum, dass mein neues Fenster nicht mehr im Vordergrund steht. Der modale Dialog in dem sich die PrograssBar befindet, blockiert offensichtlich.

Oder es stimmt wirklich etwas mit dem Parenting nicht...

Die andere Variante mit SwingWorker muss ich erst noch ausprobieren. Ich geb noch nicht auf.

Danke jedenfalls für die Unterstützung!