Controller wird zu groß, was tun ?

[Joob]

Ich habe ein kleines Programm geschrieben.
Dabei verwende ich TabPane.
Die Controller habe ich aufgeteilt, für jeden Tab.
Jetzt ergibt es sich aber das ich mehrere Tableviews in einem Tab verwenden möchte.
Das ist etwas unübersichtlich.

1. Was kann ich tun um die Steuerung der Tableviews noch einmal zu unterteilen.

2. Wenn ich die Methoden abstrahieren will, komme ich immer wieder an die Stelle in der ich unterschiedliche Methoden aufrufen muss, also Klasse.Methode. Kann man den Methodenaufruf mit Parametern regeln.
pulic classname (tabelview, Methodarray) {

dann bei Bedarf
CLASS1 class1 = new CLASS1();
class1.Methodarray[1] ;

Geht natürlich nicht so, aber dahin ziehlt meine Frage.


}

 

[mrBrown]

1. Was kann ich tun um die Steuerung der Tableviews noch einmal zu unterteilen.

Naja, einfach den großen Controller aufteilen auf mehrere kleinere. Genauso, wie du es für die einzelnen Tabs gemacht hast.

2. Wenn ich die Methoden abstrahieren will, komme ich immer wieder an die Stelle in der ich unterschiedliche Methoden aufrufen muss, also Klasse.Methode. Kann man den Methodenaufruf mit Parametern regeln.
pulic classname (tabelview, Methodarray) {

dann bei Bedarf
CLASS1 class1 = new CLASS1();
class1.Methodarray[1] ;

Geht natürlich nicht so, aber dahin ziehlt meine Frage.

Ehrlich gesagt versteh ich absolut nicht, was du damit meinst...

Wenn du unterschiedliche Methoden auf unterschiedlichen Objekten aufrufen musst, dann mach das doch einfach?
Hast du irgendeinen Beispielfall, wo man sowas wie dein Konstrukt bräuchte?

 

[Joob]

Hallo,

1. du meinst weiter verschachteln, ich mach ja mit Inject und dann das Tabelview mitteilen und dort dann die Funktionalität unterbringen ?

zu 2 schreib ich morgen noch was, muss gerade weg und kann das nicht verschieben.

 

[mrBrown]

1. du meinst weiter verschachteln, ich mach ja mit Inject und dann das Tabelview mitteilen und dort dann die Funktionalität unterbringen ?

Wenn das dafür nötig ist, dann ja.

Was machen denn bisher bei dir die Controller?
MVC-konform nur Behandlung von Eingaben oder ist dort die gesamte Logik untergebracht?

 

[Joob]

Hallo,

erst einmal zu 2. und wahrscheinlich hast du mit deiner Frage hinsichtlich der Controller mein Problem schon angeschnitten.

public void changeLevelCellEvent(CellEditEvent editLevel) {
     
        MaintenanceLevelsTVC levelSelected = tvmainlevels.getSelectionModel().getSelectedItem();
        // Value vor Änderung wird für mustSafeLanguage und fehlerhaftem Update benötigt
        levelByGoin = levelSelected.getLevel();                       
        levelSelected.setlevel(editLevel.getNewValue().toString());

        // auf Änderung pürfen
        if (!levelSelected.getLevel().equals(levelByGoin)) {
            goinLevelTVO.setlevel(levelSelected.getLevel());                // Zur Sicherung der letzten Änderung vor dem Verlassen
            mustSafeLevel = true;
        }
        tvmainlevels.requestFocus();                                         // Set Focus on edited element
    }

Es handelt sich um eine Methode aus einem Tabelview mit Pulldown und enum in der Zelle

in der Zeile 5 levelByGoin... beziehe ich mich auf ein Objekt in einer ObsList aus der das Tableview besteht.
Wenn ich diese Logik in eine seperate Klasse packen möchte und die Klasse für verschiedene Objekt verwenden möchte habe ich das Problem das die Methode ..getLevel() natürlich in jeder Klasse ObsListen anders heißt, das müsste ich der Methode dann sagen, ich kenne aber die Notation dafür nicht.

Ich denke du hast schon auf den problematischen Punkt mit deiner Frage hinsichtlich der Logik hingewiesen, ich habe noch keinen Plan wie ich die Logik komplett aus dem Controller heraushalten kann. Vielleicht kannst du mir ja einen Hinweis geben wie das angehen soll.

 

[mrBrown]

Wenn ich diese Logik in eine seperate Klasse packen möchte und die Klasse für verschiedene Objekt verwenden möchte habe ich das Problem das die Methode ..getLevel() natürlich in jeder Klasse ObsListen anders heißt, das müsste ich der Methode dann sagen, ich kenne aber die Notation dafür nicht.

Man muss ja nicht die gleiche Klasse für verschiedene Views nutzen, grad auch wenn die unterschiedliche Dinge darstellen

*wenn* nötig, kann man das z.T. mit Generics und ein wenig Vererbung/Komposition lösen - würde ich aber in dem Teil nicht unbedingt machen.

Ich denke du hast schon auf den problematischen Punkt mit deiner Frage hinsichtlich der Logik hingewiesen, ich habe noch keinen Plan wie ich die Logik komplett aus dem Controller heraushalten kann. Vielleicht kannst du mir ja einen Hinweis geben wie das angehen soll.

Naja, dass kommt ganz auf die Logik an.
Oftmals kann man die in den entsprechenden Objekten unterbringen, ansonsten sind da z.B. die GRASP-Patterns und dabei die Controller (die ungleich den MVC-Controllern sind ) einen Blick wert.