Eine Art umgekehrte Prozedur

[Popov]

Delphi-Quellcode:

function Test1(x: Integer): Integer;
begin
  Result := 10; //Stellvertretend für viel Code

  Result := Result * x; //<<<<<< Nur diese Zeile ist anderes <<<<<<

  Result := Result * 10; //Stellvertretend für viel Code
end;

function Test2(y: Integer; s: String): Integer;
begin
  Result := 10; //Stellvertretend für viel Code

  Result := Result * x + StrToInt(s); //<<<<<< Nur diese Zeile ist anderes <<<<<<

  Result := Result * 10; //Stellvertretend für viel Code
end;

In der Regel ist es so: man hat einen Code der sich wiederholt und der immer wieder genutzt wird, also lagert man das in eine Prozedur oder Funktion aus und greift dann auf die zurück.

Nun habe ich aber ein anderes Problem. Ich habe x Prozeduren die fast identisch sind, nur in der Mitte unterscheiden sich (sagen wir mal) zwei Code-Zeilen.

Hier ist das Problem verdreht. In der Regel ist der innere Teil gleich, also lagert man es aus. Hier ist es umgekehrt. Hier ist der äußere Teil gleich, nur der Innere anders.

If-Abfragen oder Case bringen hier nichts, denn die Parameter sind immer unterschiedlich. Nun könnte man den Vorderen Teil in eine Prozedur auslagern, den hinteren Teil in eine andere. Das würde gehen, aber irgendwie gehören beide Teile zusammen. Vor allem werden lokale Variablen und Objekte genutzt, was es etwas kompliziert macht es aufzuteilen.

Gibt es also die Möglichkeit den äußeren Teil auszulagern und dann den inneren darin auszuführen?

[Valle]

Vielleicht sowas wie ein Callback? Du übergibst einen Methodenpointer, der dann tut was er soll.

Oder du implementierst das mit Klassen. Abstrakte Basisklasse ruft Methode auf, die erst implementiert werden soll. (Schablonenmethode)

[Uwe Raabe]

In meinem Delphi würde ich das mit einer Anonymen Methode lösen...

[Sir Rufo]

Zitat von Uwe Raabe:

In meinem Delphi würde ich das mit einer Anonymen Methode lösen...

Dazu fehlen Popov allerdings ein X und ein E in der Delphi-Version ... sonst wäre das der richtige Tip.

[himitsu]

Wobei eine anonyme Methode auch "nur" ein Callback ist.

PS: Man kann natürlich auch den vielen Code in zwei Methoden/Prozeduren auslagen und dann überall aufrufen.

[BUG]

Zitat von Popov:

If-Abfragen oder Case bringen hier nichts, denn die Parameter sind immer unterschiedlich.

Das wurde hier bisher noch nicht beachtet.

Ich sehe 2 mögliche Lösungsansätze:

• Übergebe der umschließenden Funktion ein Callback-Objekt abgeleitet von einer gemeinsamen Basisklasse. Dieses kapselt den inneren Aufruf und die dafür nötigen Argumente.
• Spalte die umschließende Funktion in 2 Methoden auf; die lokalen Variablen kannst du dann in Member umwandeln. Im Destruktor nicht vergessen aufzuräumen sollte zwischendurch irgendwas schiefgehen

[Thomasl]

Eine ganz andere aber einfache Möglichkeit ist, den gleichen teil in eine IncludeDatei zu packen

Delphi-Quellcode:

function Test1(x: Integer): Integer;
begin
  {$I aaa.inc}
  Result := Result * x; //<<<<<< Nur diese Zeile ist anderes <<<<<<
  {$I bbb.inc}
end;

function Test2(y: Integer; s: String): Integer;
begin
  {$I aaa.inc}
  Result := Result * x + StrToInt(s); //<<<<<< Nur diese Zeile ist anderes <<<<<<
  {$I bbb.inc}
end;

aaa.inc

Code:

Result := 10; //Stellvertretend für viel Code

bbb.inc

Code:

Result := Result * 10; //Stellvertretend für viel Code

[stahli]

Include-Dateien lassen sich allerdings schlechter (oder gar nicht? - ich weiß nicht mehr genau) debuggen.

[Popov]

Evtl. werde ich versuchen es mit einer Callback-Funktion zu lösen. Muss klein wenig experimentieren.

[Thomasl]

Debuggen geht genauso.
aber das suchen, öffnen dahin springen ist beschissen.

Das ist ja nur eine mögliche Lösung unter vielen