Thread und TimerTask, Verstädnisproblem

[Chefselber]

Guten Morgen Gemeinde,

ich habe anscheinend ein Verständnisproblem.

Ich rufe aus der Main mittels Timer die Klasse Aufgabe auf und diese holt sich Daten aus einer SPS. Funktioniert auch soweit und es werden dann auch die Daten nach BTB in einer Liste geschrieben.
Mache ich das über einen Thread, macht er das einfach nicht,
dass er Daten an die Liste übergibt.
Was ich nicht verstehe ist, dass es bei Zeitgesteuert funktioniert und über einen Thread nicht.

Hier die Klassen:

Ich rufe über einen Button

private void jButton1ActionPerformed(java.awt.event.ActionEvent evt) {                                        
     // Verbinden();
       Thread verbinden = new Thread(new Verbinden());
       verbinden.start();
     
 
    }

einen Thread auf, der genau dasselbe macht (bzw. machen sollte wie Klasse Aufgabe)






[CODE lang="java" title="Main Form"]public class BTB extends javax.swing.JFrame {
....


}


public static void main(String args[]) {
/* Set the Nimbus look and feel */
//<editor-fold defaultstate="collapsed" desc=" Look and feel setting code (optional) ">
/* If Nimbus (introduced in Java SE 6) is not available, stay with the default look and feel.
* For details see http://download.oracle.com/javase/tutorial/uiswing/lookandfeel/plaf.html
*/
try {
for (javax.swing.UIManager.LookAndFeelInfo info : javax.swing.UIManager.getInstalledLookAndFeels()) {
if ("Nimbus".equals(info.getName())) {
javax.swing.UIManager.setLookAndFeel(info.getClassName());
break;
}
}
} catch (ClassNotFoundException ex) {
java.util.logging.Logger.getLogger(BTB.class.getName()).log(java.util.logging.Level.SEVERE, null, ex);
} catch (InstantiationException ex) {
java.util.logging.Logger.getLogger(BTB.class.getName()).log(java.util.logging.Level.SEVERE, null, ex);
} catch (IllegalAccessException ex) {
java.util.logging.Logger.getLogger(BTB.class.getName()).log(java.util.logging.Level.SEVERE, null, ex);
} catch (javax.swing.UnsupportedLookAndFeelException ex) {
java.util.logging.Logger.getLogger(BTB.class.getName()).log(java.util.logging.Level.SEVERE, null, ex);
}
//</editor-fold>
Timer timer = new Timer();

timer.scheduleAtFixedRate(new Aufgabe(),10000,2000);


java.awt.EventQueue.invokeLater(new Runnable() {
public void run() {
new BTB().setVisible(true);

}
});
}[/CODE]


Die Aufgabe Klasse funktioniert

[CODE lang="java" title="Aufgabe Klasse"]public class Aufgabe extends TimerTask implements Runnable{


public BTB btb = new BTB();

byte[] data;

@Override
public void run() {

Einlesen();
// connect();
System.out.println("MuafA");

}


public void Einlesen()
{
byte[] data = new byte[2];
btb.Verbinden();
data =btb.Start(10);
btb.bits_uebernehmen(data);
btb.aktualisieren();

}

[/CODE]



Hier die Thread Klasse

public class Verbinden implements Runnable{
     static  Boolean[]   bit  ;
  public   BTB btb = new BTB();
private static final S7Client Client = new S7Client();
    @Override
    public void run() {
       
       
        connect();
    }
    public void connect()
    {
       
          String ip = btb.ip_holen();
          Client.SetConnectionType(S7.PG);
       
         
          int Result = Client.ConnectTo(ip,0,3);
          btb.Status(Result);
         
           byte[]  data = new byte[2];
      //  btb.Verbinden();
        data =Start(10);
        btb.bits_uebernehmen(data);
        btb.aktualisieren();
        btb.jLabel1_aendern("efefrgrgargegaergaregergsergrsgreger");
        System.out.println("ettrttertt");
     
    }

Hier noch die aktualisieren Methode

public void aktualisieren()
    {
 
       for (int v=1;v<= 16;v++)
       {
            if (bit[v-1] ==true)
            {
                stoerungen.addElement(liste.getElementAt(v));
            
                list1.add(liste.getElementAt(v).toString());
              
            }
          
       }
 
        
    }

Vielleicht kann mir wer helfen?!

MFG Kevin

 

[mrBrown]

Du erzeigst mindestens 3 verschiedene Instanzen von BTB, das ist vermutlich nicht gewollt?

 

[Chefselber]

1x zum Formaufruf.

Wenn ich die Timer.schedule raus nehme, gehts es ja trotzdem nicht.

Mir gehts eigentlich darum, warum aus einem Thread die Form nicht bearbeitet wird. Jlabel, list usw..
Braucht man hier ActionListener usw.?

Der Aufruf der TimerTask Klasse funktioniert ja und beschreibt mir die Liste.

Variablen usw. werden ja in der Instanz von BTB im Thread beschrieben.

 

[mrBrown]

Du erzeugst drei(!) verschiedene(!!) Fenster.

Wenn du ein Fenster anzeigst, aber in dem anderen die Label aktualisierst, siehst du das natürlich in dem angezeigtem Fenster nicht, das ist ja ein anderes.

 

[Chefselber]

Eieieieiei, ja weiss was du meinst, eins wird in der Main erzeugt und angezeigt.

Wie löse ich das dann z.B. im Thread?

Den Zugriff aufs Main Fenster?
ohne new?

public class Verbinden implements Runnable{

     static  Boolean[]   bit  ;

  public   BTB btb;

private static final S7Client Client = new S7Client();

    @Override

    public void run() {

      

      

        connect();

    }

    public void connect()

    {

      

          String ip = btb.ip_holen();

          Client.SetConnectionType(S7.PG);

      

        

          int Result = Client.ConnectTo(ip,0,3);

          btb.Status(Result);

        

           byte[]  data = new byte[2];

      //  btb.Verbinden();

        data =Start(10);

        btb.bits_uebernehmen(data);

        btb.aktualisieren();

        btb.jLabel1_aendern("efefrgrgargegaergaregergsergrsgreger");

        System.out.println("ettrttertt");

    

    }

 

[Chefselber]

Das funktioniert schon mal nicht.

Exception in thread "Thread-1" java.lang.NullPointerException: Cannot invoke "Moka7Demo.BTB.ip_holen()" because "this.btb" is null
    at Moka7Demo.Verbinden.connect(Verbinden.java:28)
    at Moka7Demo.Verbinden.run(Verbinden.java:23)
    at java.base/java.lang.Thread.run(Thread.java:832)

@mrBrown
Was ich immer noch nicht verstehe, warum die TimerTask Klasse Aufgabe funktioniert. Hier werden ja dann auch 2 Fenster erstellt?

Wie kann ich dann dem Thread beim erstellen eine Verknüpfung aus BTB mitgeben?

 

[fhoffmann]

Ich habe nicht genau angesehen, wie dein Projekt aufgebaut ist. Und hier liegen vermutlich auch noch ein paar Probleme (ich sehe auch noch zu oft das Wort "static").

Aber grudsätzlich kannst du natürlich deinem Thread (Runnable) die Oberfläche im Konstuktor mitgeben:

public class Verbinden implements Runnable {
  // public BTB btb = new BTB(); // kommt weg
  private BTB btb;
  public Verbinden(BTB btb) {
      this.btb = btb;
  }
  public void run() {
     // benutze btb
  }
}

Und dann rufst du den Thread mit der richtigen BTB auf

  BTB btb = new BTB()
  Verbinden verbinden = new Verbinden(btb);
  // nun benutz du an dieser Stelle und das erzeugte Verbiden die gleiche Instanz von BTB

 

[Chefselber]

Vielen Dank, hier hats gehakt.
Klar, ich muss dem die Instanz mitgeben. 😩
Statische Variablen sind halt immer einfach, aber wird nicht gerne gesehen.

 

[fhoffmann]

Statische Variablen sind halt immer einfach, aber wird nicht gerne gesehen.

Das hört sich wie ein Vorwurf an, dass Leute, die deinen Code ansehen, Vorurteile gegen das Wort "static" haben.

Ich will es mal an Hand dieses Threads beschreiben:
Du hast heute morgen um 10:15 Uhr um Hilfe gebeten, und hast etwa nach zehn Stunden geschrieben:

Vielen Dank, hier hats gehakt.
Klar, ich muss dem die Instanz mitgeben.

Wie, wo und wann man eine Instanz mitgeben muss, gehört zur Planung eines Programms. Und das Wort "static" kommt oft dann vor, wenn sich der*die Programmierer*in darüber keine Gedanken gemacht hat.
Und hättest du dir frühzeitig darüber Gedanken gemacht, hättest du möglicherweise zehn Stunden gespart.

 

[Chefselber]

So meinte ich das nicht, sollte keinesfalls ein Vorwurf sein. Ich weiss, dass man statische Sachen eher vermeiden sollte, aber vorallem am Anfang es einfacher ist zwischen Klassen Daten zu teilen. Was natürlich nicht schön ist, vorallem wenn mehr Leute an einem Projekt arbeiten.

 

[fhoffmann]

sollte keinesfalls ein Vorwurf sein

Ich habe es auch nicht als Vorwurf gegen mich empfunden.
Vielmehr wollte ich sagen, dass du dir wahrscheinlich selbst ins Knie schießt, wenn du "static" benutzt.