Änderungen im Frame werden zu spät durchgeführt

[Durandir]

Hallo, ich habe ein Problem und komme leider nicht weiter. Ich hatte mal ein Spiel gesehen, wobei man sich in einer bestimmten Zeit eine Zahl einprägen und diese dann nachher wiedergeben musste. Dieses Spiel wollte ich nachprogrammieren und insoweit "verbessern", dass man die Zeit, in der die Zahl sichtbar ist, variieren kann.
Leider klappt es aber nicht so, wie ich es möchte. Zu dem Kode unten: Die Methode losgehts ist eine Framemethode und wird über einen Button aufgerufen. Darin wird je nach Größe des Eingabeparameters i eine Zahl zufällig erstellt und sollte dann im Label angezeigt werden. Danach wird ein Threadobjekt erzeugt, darin 2000ms gewartet und am Ende sollte die Zahl im Label durch "**" ersetzt werden.
Mein Problem ist, dass zwar die richtige Zahl pünktlich auf der Console ausgegeben wird, aber im Labelfeld immer nur die "**" erscheinen. Lasse ich die letzte Zeile aber weg, dann erscheint auch die richtige Zahl.
Ich hatte es auch schon mit sowas wie "busy waiting(for-Schleife)" versucht, aber es klappt irgendwie nicht.

Kann mir einer da weiterhelfen?

Vielen Dank schon einmal
Gruss
Durandir

public void losgehts(int i){
     
     int j;
     
     int f[] = new int[10];
     
     AFrame.this.c="";    //Stringvariable des Frames
     
    
     for(j=0;j<i;j++){
         f[j]=(int) (10*Math.random());
         AFrame.this.c= String.valueOf(AFrame.this.c+""+f[j]);
     }
          
     System.out.println(AFrame.this.c);         //die kommt pünktlich
     AFrame.this.jLabelZahl.setText(""+AFrame.this.c+"");   //diese Zahl erscheint niemals
     
     
    Warten warten = new Warten(AFrame.this);   //Threadobject zum Warten von 2000ms
    
    try {
   
        warten.start();
        warten.join();
     
    }catch(Exception e){
        System.out.println("Ochnö"+e);
    }
    
    AFrame.this.jLabelZahl.setText("**");  //dies ist immer schneller, als die eigentliche Zahl
     
 }

 

[Beni]

Die Logik (dein "losgehts") muss in einem anderen Thread als das Fenster laufen.

Klassisch sieht das dann so aus:

import java.awt.EventQueue;

import javax.swing.JFrame;
import javax.swing.JLabel;

public class Test{
    public static void main( String[] args ) {
        JFrame frame = new JFrame();
        frame.setDefaultCloseOperation( JFrame.EXIT_ON_CLOSE );
        final JLabel label = new JLabel( "0" );
        frame.getContentPane().add( label );
        frame.setBounds( 20, 20, 300, 200 );
        frame.setVisible( true );
        
        // Die Logik in einen eigenen Thread
        Thread thread = new Thread(){
            public void run(){
                int index = 0;
                
                // Hier kann man alles wüste machen
                while( true ){
                    final int current = index++;
                    
                    // AWT/Swing ist nicht threadsicher, mit diesem Konstrukt
                    // lagert man den Aufruf von "setText" in eine sichere 
                    // Umgebung aus.
                    EventQueue.invokeLater( new Runnable(){
                        public void run(){
                            label.setText( String.valueOf( current ) );
                        }
                    });
                    
                    try{
                        // Diesen Thread kann man pausieren.
                        Thread.sleep( 1000 );
                    }
                    catch( InterruptedException ex ){
                        ex.printStackTrace();
                    }
                }
            }
        };
        thread.start();
    }
}

 

[Durandir]

Besten Dank für die schnelle Antwort. Für mich als Anfänger sieht dein Kode zwar noch etwas wirr aus, aber ich werde mal versuchen mich da reinzuwühlen. Ich habe nun erstmal die eine eigene Klasse erstellt und wenn ich die laufen lasse, dann klappt das auch mit der zuerst erscheinenden Zahl. Wenn ich nicht möchte, dass das Label dann allerdings nicht mehr private ist, weil halt auch die andere Klasse draufzugreifen will, dann muss ich via call-back die Kontrolle an das Frame übergeben, die Änderung durchführen lassen und wieder zurück in die Klasse wechseln, oder?

Kannst du mir auch sagen, warum die Logik ausgelagert sein muss? Ich meine, selbst wenn man keine Threads verwendet, sondern die Wartezeit mittels 1000 Ausgaben auf der Konsole "simuliert", dann klappt es ja auch nicht.

Danke nochmals.....
Gruss
Durandir

 

[Beni]

Zu 1: Praktisch kannst du das EventQueue-Zeugs in eine Methode des Frames kopieren, die das Label verändert. Das wird etwa das sein, was du mit "call-back" meinst?

Zu 2: Es schwirren viele Threads durch die Gegend. Einer dieser Threads ist der AWT-Thread. Dieser Thread ist dafür zuständig, dass Frames gezeichnet werden, Mouse-Events bei den Listeners ankommen, etc...
Wenn man Pech hat, arbeitet man in diesem AWT-Thread, und damit kann der AWT-Thread nicht neuzeichnen. Deshalb muss man die "Logik" in einen eigenen Thread stecken.

 

[Durandir]

Hmmm....also ich fürchte, ich habe es immer noch nicht so recht.

Ich habe nun einmal eine Klasse erstellt, darin das Frame erzeugt und es mit einer 5-stelligen Zahl durchlaufen lassen..klappt.

Danach wollte ich das Programm wieder zusammenbauen. Also erzeuge ich in der neuen Klasse ein Objekt davon und erstelle danach ein neues Frame, wobei ich diesem Frame die Adresse des Objekt mitgebe. In dem Frame lasse ich nun auf Knopfdruck die Stelligkeit der Zahl hochzählen und rufe die Methode der neuen Klasse auf, wo nun die Zahl berechnet und die Labels geändert werden. Als Parameter übergebe ich wieder die Adresse des Frames sowie die Stelligkeit, was jeweils in Attributen der Klassen gespeichert wird.

Aber irgendwie bin ich wieder da, wo ich vorher auch war. Das Frame wird nun in der neuen Klasse erstellt und auch die Labels werden von da geändert, aber wieder habe ich das Anfangsproblem.

 

[Durandir]

Ach, nun habe ich es verstanden. Es ist wichtig, dass man sich nicht mittels einer neuen Klasse Zutritt verschafft, sondern sich in einem anderen Thread befindet. Dein letzter Absatz war sehr hilfreich. Ich habe nun in der neuen Klasse einen neuen Thread ins Leben gerufen und nun klappts. Nur ist das Programm ziemlich wirr und ich muss nun mal schauen, dass es wieder übersichtlich wird.

Danke nochmals....jetzt klappts endlich =)

Gruss
Durandir