Programm führt Methoden gleichzeitig aus

[Riley12121]

Hallo,
ich bin gerade dabei ein Programm mit Java zu programmieren, welches ein anderes mit Robot fernsteuert. Dort soll das Java Programm unter anderem automatisch Textfelder erstellen und diese dann nach bestimmten Vorgaben formatieren.
Nun bin ich auf das Problem gestoßen, dass sich nicht an die Reihenfolge der Programmcodes gehalten wird. Ich erstelle mit einer Methode ein beschriebenes Textfeld, dieses soll dann mit einer weiteren Methode markiert werden, nur damit danach der Reiter für die Formatierung geöffnet wird. Alles alleine klappt auch einwandfrei.
Nun ist es aber leider so, dass mein Java Programm teilweise schon den Reiter für die Formatierung öffnet, obwohl das Textfeld noch nicht beschrieben wurde, etc. Ich hatte auch schon etwas gegoogelt, wie man das verhindern kann, und bin auf Threads gestoßen und die Funktion join(). Eigentlich hatte ich es so verstanden, dass bei join() so lange gewartet wird, bis der Thread durchgelaufen ist. Allerdings klappt es leider bei mir nicht.
Kann mir vielleicht jemand erklären, was ich falsch mache, und wie ich es wirklich verhindern kann, dass der Ablauf durcheinander gerät?

Das ist meine Methode, mit der Überschriften erstellt werden sollen:

public static void überschriftGruppe (String nameG,)throws AWTException, InterruptedException{
MyThread t1 = new MyThread (nameG);
t1.start();
t1.join();
Robby.markieren(nameG);
Robbybaut.farbeWählenÖffnen();
Robbybaut.farbeWählenTextGruppe();
}

Das ist die Thread-Klasse, die ich aktuell gebaut habe:

public class MyThread extends Thread {
String nameG;

MyThread (String nameG){
this.nameG = nameG;
}

public void run (){
try {
Robbybaut.textfeld(nameG);
} catch (AWTException ex) {
Logger.getLogger(MyThread.class.getName()).log(Level.SEVERE, null, ex);
} catch (InterruptedException ex) {
Logger.getLogger(MyThread.class.getName()).log(Level.SEVERE, null, ex);
}
}

}

 

[KonradN]

Die Beschreibung ist relativ schwer zu verstehen. Was ich verstanden habe ist, dass Du mit Deiner Java Anwendung eine andere Anwendung fernsteuerst.

Ein mögliches Problem könnte hier evtl. sein, dass Dein Steuerungs-Programm schlicht zu schnell ist. Unter dem Strich passiert ja, dass durch die Aktionen in Deinem Programm nur Events für das andere Programm erzeugt werden. Danach läuft Dein Programm direkt weiter, so dass es sein kann, dass Du weitere Events für das andere Programm erzeugst, ehe das andere Programm das erste Event abgeschlossen hat.
Das kann funktionieren, so das andere Programm die Events entsprechend der Reihenfolge des Eingangs abarbeitet. Es gibt meines Wissens nur wenige Events, die bevorzugt behandelt werden. Aber es kann natürlich zu Abhängigkeiten kommen, so Veränderungen vorgenommen werden, die existenziell für das nächste Event sind. (Wie will man z.B. ein Control verändern, das noch nicht erstellt wurde?)

Aber ohne die ganz genauen Informationen zu kennen, was Du genau wie machst, ist das nur ein raten. Das kann so sein, aber muss nicht. Evtl. liegt die Problematik auch an etwas anderer Stelle.

Ansonsten kann man nur paar allgemeine Dinge zu dem Code sagen:

MyThread t1 = new MyThread (nameG);
t1.start();
t1.join();

Sowas ist in der Regel nicht sinnvoll. Wenn Du eh auf den anderen Thread warten willst, dann kannst Du es auch gleich auf dem gleichen Thread ausführen.

Dann ist der Code so schlicht unleserlich auf unnötig aufgebläht. Es gibt ja nicht ohne Grund mit Java 8 die Funktionalen Elemente.

Statt eine Klasse von Thread erben zu lassen und dann run() zu überschreiben kannst Du einfach eine Methode schreiben, die dann einen Namen hat, der auch besagt, was da gemacht wird. Und die Methode kannst Du dann über die Methodenreferenz oder über eine Lambda Expression aufrufen.

Also von mir aus sowas in der Art:

public void createTextField(String nameG) {
    try {
        Robbybaut.textfeld(nameG);
    } catch (AWTException ex) {
        Logger.getLogger(MyThread.class.getName()).log(Level.SEVERE, null, ex);
    } catch (InterruptedException ex) {
        Logger.getLogger(MyThread.class.getName()).log(Level.SEVERE, null, ex);
    }
}

Und Nutzung per new Thread( () -> createTextField("Whatever") )

Dann ist es aus meiner Sicht eher Üblich, den Logger als statische Variable zu haben und nicht immer neu mit getLogger zu holen. Also etwas wie:
private static Logger logger = Logger.getLogger(MyThread.class.getName());
Um dann den logger zu nutzen, wo man ihn braucht.

Und dann auch noch der Hinweis: Du hast seit Java 7 die Möglichkeit, mehrere Exception Typen anzugeben:

public void createTextField(String nameG) {
    try {
        Robbybaut.textfeld(nameG);
    } catch (AWTException | InterruptedException ex) {
        Logger.getLogger(MyThread.class.getName()).log(Level.SEVERE, null, ex);
    }
}

 

[Riley12121]

Tatsächlich hast du gerade schon mein Problem damit gelöst.
Ich hatte zwar berücksichtigt, dass mein Programm zwischen einzelnen Klicks der Robot Klasse Zeit zum Reagieren braucht, ausgerechnet am Ende von Robbybaut.textfeld (nameG) habe ich es allerdings vergessen. Jetzt, wo ich dort auch eine kurze Pause einlegen lasse, funktioniert mein Programm wieder.