Redundante Zustandsprüfungen auslagern

[Meisenmann]

Hi,
ich habe in meinem (Pseudo-)Code folgende redundante Zustandsprüfung:

void f1(int){ 
  if(allowed) {
    update(int);
  }
  else {
    throw new Exception("not allowed");
  }
}

void f2(double) {
  if(allowed) {
    update(double);
  }
  else {
    throw new Exception("not allowed");
  }
}

void f3(String) {
  if(allowed) {
    update(String);
  }
  else {
    throw new Exception("not allowed");
  }
}

Wie kann ich diese Redundanz am elegantesten auslagern? Eine vorherige Methode mit switch bzw. if/else finde ich nicht so dolle.

Danke und viele Grüße,
Meisenmann

 

[JCODA]

Ist es möglich die update-Methode generisch zu machen? Damit könnte f ebenfalls generisch sein und die Methode gibt's dann nur noch ein einziges Mal.

 

[Joose]

[ot]
Bitte in Zukunft die

-Tags für Java Code verwenden, danke! :)
[/ot]

 

[JavaMeister]

[ot]
Bitte in Zukunft die

-Tags für Java Code verwenden, danke! :)
[/ot][/QUOTE]

Lohnt sich das jedes mal hinzuschreiben? Man könnte stattdessen ja im Beitrag was schreiben. SO denkt man: Ohh eine Antwort: Nein.

 

[Joose]

Copy&Paste, außerdem ein EDIT im Beitrag des Erstellers wird weniger war genommen als eine Antwort. (mein Gefühl)

 

[Ruzmanz]

Kommt auf die Größe des Projekts an. Bei großen Projekten bietet sich AOP (Aspektorientierte Programmierung) an. Wenn es sich nur um die drei Methoden dreht, ändert man update:

Ist es möglich die update-Methode generisch zu machen? Damit könnte f ebenfalls generisch sein und die Methode gibt's dann nur noch ein einziges Mal.

 

[Meisenmann]

vielen Dank für die Antworten!

Das mit der generischen Implementierung der Methode verstehe ich nicht. Ich möchte ja die Datentypen unterschiedlich behandeln. Für Zahlenwerte möchte ich beispielsweise eine Range-Prüfung durchführen und für
String ein bestimmtes Regex abfragen. Somit müsste ich ja in der Methode mit if und instanceof den Typ des Parameters abfragen.

Wie ist das mit AOP gemeint?

Java-Tags habe ich absichtlich nicht verwendet, weil es sich ja um Pseudo-Code handelt

Danke und viele Grüße
Meisenmann

 

[arilou]

Statt dem

if(allowed)
// ...
else
throw

könnte man eine Methode

checkAllowed()

einfügen; wenn die eine Exception wirft, wird ein nachfolgendes update(...) dann auch nicht mehr ausgeführt.

 

[Joose]

Wie ist das mit AOP gemeint?

Einfach mal schaun was dir das Internet dazu sagen kann.
Wenn es eine konkrete Frage dazu gibt kannst du diese natürlich stellen

Java-Tags habe ich absichtlich nicht verwendet, weil es sich ja um Pseudo-Code handelt

Auch Pseudocodo (vor allem ein längerer) wird dadurch lesbarer und hebt sich von Text ab.