OOP GUI aus anderer Klasse updaten

[TheMan]

Guten Tag,

ich hoffe ihr könnt mir mit meinem kleinen Anliegen helfen. Und zwar arbeite ich zum ersten mal mit einer GUI in Java und bin nun auf ein Problem gestoßen, bei dem ich weiß wo der Fehler liegt, aber nicht wie ich in behebe.

Habe zwei Klassen

• GUI Class
• ScriptClass

Nun werden einzelne Schritte der ScriptClass durchlaufen und der aktuelle Status soll an die GUI in eine JTextArea eingetragen werden. Das Problem ist, das ich innerhalb der ScriptClass nicht auf die GUI Klasse zugreifen kann, da diese dann wieder den Kontruktor ausführt was zu folgendem Fehler führt.

Exception in thread "AWT-EventQueue-0" java.lang.StackOverflowError
at ScriptClass.<init>(ScriptClass.java:22)
at GUI_Class.<init>(GUI_Class.java:37)
at ScriptClass.<init>(ScriptClass.java:22)
at GUI_Class.<init>(GUI_Class.java:37)

Code-Beispiel
GuiCLass erstellt mit WindowBuilder und Eclipse

public class GUI_Class{

    JFrame GuiClass;

//------------------------------------------------------------------------------------------------------------------------------------------   
    /**
     * Launch the application.
     */
    public static void main(String[] args)
    {
        EventQueue.invokeLater(new Runnable() {
            public void run() {
                try {
                    GUI_Class window = new GUI_Class();
                    window.GuiClass.setVisible(true);
                } catch (Exception e) {
                    e.printStackTrace();
                }
            }
        });
    }
//------------------------------------------------------------------------------------------------------------------------------------------    
      /**
      * Getter / Setter Function
      */
    public void setGUILog (String xMsg)
    {
        LogArea.append(xMsg + "\n");
    }

//----------------------------------------------------------------------------------------------------
    /**
     * Create the application.
     */

    public GUI_Class()
    {
        initialize();

        // Settings for File Chooser -> Logs
           openFileChooser = new JFileChooser();
           openFileChooser.setCurrentDirectory(new File("C:\\Sandboxes"));
           openFileChooser.setFileSelectionMode(JFileChooser.DIRECTORIES_ONLY);
       
        // Settings for File Chooser -> Logs
           openFileChooser2 = new JFileChooser();
           openFileChooser2.setCurrentDirectory(new File("C:\\Sandboxes"));
           openFileChooser2.setFileSelectionMode(JFileChooser.FILES_AND_DIRECTORIES);      
    }

ScriptClass

public class ScriptClass{

 GUI_Class Gui = new GUI_Class ();   [B]<--- führt zu Fehler, da neuer Aufruf von init function[/B]

//Beispiel für euch
//Remove duplicated entries in array list
   void removeDuplicatedEntries(ArrayList<String> arrayList, ArrayList<String> sortList)
   {
       // Log Message
       LogWorker.setLogMsg("INFO", "Check duplicated entries and remove them");
       String xLog = LogWorker.getLogMsg();
       Gui.setGUILog(xLog);  [B]<--wollte dann mit der Set-Function arbeiten [/B]

       ....
   }
}

Ich hoffe ihr versteht mein blödes Problem, welches wahrscheinlich einfach zu lösen ist.

Kurz: Ich muss innherlab der Script Klasse auf die Set-Funktion der Gui zugreifen

Vielen dank im voraus

 

[mrBrown]

Einfache Lösung: Übergib der ScriptClass dein GUI_Class Object.


Schöne Lösung: Guck dir MVC an

 

[TheMan]

Vielen Dank,

könntest du kurz aufzeigen wie die Object Übergabe der GUI_Class aussieht? Ich meine das heute schon vergeblich versucht zu haben

Das Prinzip des Model View Controller schaue ich mir bei Gelegenheit auch an

 

[mrBrown]

könntest du kurz aufzeigen wie die Object Übergabe der GUI_Class aussieht? Ich meine das heute schon vergeblich versucht zu haben

Das ist etwas, was du aus dem Effeff beherrschen solltest, bevor du irgendwas mit GUI versuchst

Irgendwo musst du einen Setter oder Konstruktor haben, der das entsprechende Objekt entgegen nimmt, und dem musst du das übergeben.
In deinem Code wird allerdings nirgends ein Objekt der ScriptClass erstellt, das ist sicher keine Absicht, oder?

 

[TheMan]

Wird in der GUI_Class erstellt, hatte das nur rausgenommen, weil es mir ja um den weg aus Script_Class zu GUI_Class geht

Ich hatte es folgendermaßen versucht

Die GUI/Main Klasse

public class GUI_Class{

    JFrame GuiClass;
    Script_Class mScript = new Script_Class();

//------------------------------------------------------------------------------------------------------------------------------------------  
      /**
      * Getter / Setter Function
      */
    public void setGUILog (String xMsg)
    {
        LogArea.append(xMsg + "\n");
    }
}

Die Script Klasse, die als Übergabe die GUI_Class bekommt, bzw. eine Referenz darauf.

class ScriptClass {
    private GUI_Class gui;

    public GUI_Class(GUI_Class gui) {
        this.gui = gui;
    }

        GUI_Class gui = new GUI_Class();
        ScriptClass test = new ScriptClass(gui);



//Remove duplicated entries in array list
   void removeDuplicatedEntries(ArrayList<String> arrayList, ArrayList<String> sortList)
   {
       // Log Message
       LogWorker.setLogMsg("INFO", "Check duplicated entries and remove them");
       String xLogMsg = LogWorker.getLogMsg();
       test.setGUILog(xLogMsg); 

       ....
   }

   //Methoden

Das hat aber meine ich auch nicht funktioniert, ich stehe irgendwo auf dem Schlauch wo ich was erstellen muss -_-

Ich glaube ein kleines Beispiel würde mir helfen oder eine kurze Änderung an dem von mir geposteten Code.

Ich danke dir

 

[mrBrown]

Wird in der GUI_Class erstellt, hatte das nur rausgenommen, weil es mir ja um den weg aus Script_Class zu GUI_Class geht

Dafür sind beide Stellen relevant

Das hat aber meine ich auch nicht funktioniert

Natürlich funktioniert das nicht.
In Gui_Class (guck dir mal bitte Konventionen zur Benennung von Klassen an) erstellst du ein Objekt von ScriptClass. In ScriptClass hast du einen Konstruktor für Gui_Class definiert, was so nicht mal kompiliert. Dann erstellst du in dem ScriptClass-Object ein weiteres Gut_Class-Objekt und noch ein neues ScriptClass-Object, dem du ein GUI_Class-Object übergibst, wofür es allerdings keinen Konstruktor gibt.

Ich glaube ein kleines Beispiel würde mir helfen oder eine kurze Änderung an dem von mir geposteten Code.

In Gui_Class:

ScriptClass mScript = new ScriptClass(this); //Hier erstellst du ein Script_Class-Object und übergibst dem das aktuelle Gui_Class-Objekt

In ScriptClass:

Gui_Class gui_Class; //eine Variabel vom Typ Gui_Class deklarieren

public ScriptClass(Gui_Class gui_Class) { //Konstruktor von ScriptClass, welcher ein Gui_Class-Objekt entgegen nimmt
    this.gui_Class = gui_Class; //dieses Gui_Class-Objekt wird der Variablen zugewiesen
}

 

[TheMan]

Oh wie dämlich von mir. Ich danke dir vielmals.

Habe mich gestern auch in das MVC Konzept eingelesen und werden wohl nun alles darauf umstellen.

Vielen Dank noch einmal