NewAction und EditAction

[lenilsas]

Ich habe eine NewAction und eine EditAction erstellt die viele vorhandene Actions ersetzen könnten. Ich habe es erstmal nur für die BuchungsKlasseAction gemacht. Wenn es auf Zustimmung stößt setze ich es für die anderen Actions um.

[JohannMaierhofer]

Ich würde vorschlagen, dass man auch bei der EditAction wie bei der NewAction die ObjectClass übergibt. Dann kann man in handleAction überprüfen ob der context eine Instanz dieser Klasse ist. Ist es das nicht sollte es eine Fehlermeldung geben. Diese Prüfung gibt es ja auch bei den bisherigen Edit Actions.

[lenilsas]

Ich würde vorschlagen, dass man auch bei der EditAction wie bei der NewAction die ObjectClass übergibt. Dann kann man in handleAction überprüfen ob der context eine Instanz dieser Klasse ist. Ist es das nicht sollte es eine Fehlermeldung geben. Diese Prüfung gibt es ja auch bei den bisherigen Edit Actions.

Das hatte ich auch schon versucht, allerdings weiß ich nicht wie das geht.
Der Aufruf wäre: new EditAction(BuchungsklasseView.class, Buchungsklasse.class) und in der EditAction müsste ich context instanceof objectClass machen, das geht aber nicht. Wenn du eine Idee hast wie ich das lösen kann, kann ich es gerne ändern.

[JohannMaierhofer]

Du hast Recht, das geht wohl nicht. Dann müssen wir das aber im View überprüfen. Ich habe mal testweise eine Buchungsart übergeben, da gab es dann eine Exception im BuchungsklasseControl.
So hats functioniert mit den ersten Zeilen im View:
if (getCurrentObject() instanceof Buchungsklasse)
{
GUI.getView().setTitle("Buchungsklasse");
}
else
{
GUI.startPreviousView();
GUI.getStatusBar().setErrorText("Keine Buchungsklasse ausgewählt");
return;
}

Die Frage wäre dann nur ob man die Edit Actions nicht doch lässt wie sie waren und nur die NewAction macht.

[JohannMaierhofer]

Eigentlich sollte das aber nicht passieren wenn kein Fehler im Code ist.
Es gibt nur das Problem, was ich habe, dass manchmal der Context bei Edit Actions null ist obwohl der Menüpunkt angezeigt wird.

[lenilsas]

Eigentlich sollte das aber nicht passieren wenn kein Fehler im Code ist. Es gibt nur das Problem, was ich habe, dass manchmal der Context bei Edit Actions null ist obwohl der Menüpunkt angezeigt wird.

Ja, eigentlich gibt es keinen Fehler. Den Context auf null testen kann man ja trotzdem.
Außerdem baue ich noch einen Test auf instanceof Object[] ein, falls der Eintrag bei Mehrfachauswahl aufgerufen wird.

[JohannMaierhofer]

Ja, so sollte es reichen.