For-Schleife als Template mit Prozedur als Parameter?

[QuickAndDirty]

Ich glaube er wünscht sich so einen Stil wie Javascript.
Wo man massive Dinge wie höhere Funktionen und bindings als Funktionsparameter verwenden kann.

[LTE5]

Nee. Ich benutze an sehr vielen Stellen im Programm immer wieder denselben Schleifenkopf und vor diesem Schleifenkopf noch eine weitere if-Abfrage.

Etwa so

Delphi-Quellcode:

{ 1 } if Liste.Count > 0 then
  begin
{ 2 }   for i := 0 to Liste.Count - 1 do
   begin
    DieEigentlicheArbeit;
   end;
 end;

Durch das Auslagern des Schleifenkopfes und DieEigentlicheArbeit (das kann alles sein) als Prozedurparameter habe ich { 1 } und { 2 } jetzt nur noch an einer einzigen Stelle.

Delphi-Quellcode:

procedure TForm1.ForLoopList(Func: TFunc<Integer, Boolean>);
var
 i: Integer;
begin
 if Liste.Count = 0 then
  Exit;

 for i := 0 to Liste.Count - 1 do
  begin
   if Func(i) then
    Break;
  end;
end;

ForLoopList(
 function(i: Integer): Boolean
  begin
   Result := DieEigentlicheArbeit;
  end);

Was ich jedoch noch nicht so richtig verstehe, ist die Definition von TFunc. Warum <Integer, Boolean>?

[Bernhard Geyer]

Du weißt aber schon das der erste Teil mit "if Liste.Count > 0" unnötig ist wenn nur die for-Schleife folgt.

[LTE5]

Sähe das sonst aber nicht so aus?
FÜR i = 0 BIS -1 DANN Weil Liste.Count -1 bedeutet ja 0-1 = -1?

Zitat:

if Liste.Count > 0

Warum denn > 0? Ich prüfe im neuen Code auf = 0.

[Bernhard Geyer]

Zitat von LTE5:

Sähe das sonst aber nicht so aus?
FÜR i = 0 BIS -1 DANN Weil Liste.Count -1 bedeutet ja 0-1 = -1?

ja und? D.h. er geht nie in die Schleife.

Zitat von LTE5:

Zitat:

if Liste.Count > 0

Warum denn > 0? Ich prüfe im neuen Code auf = 0.

Also ich sehe da eine Zeile:

{ 1 } if Liste.Count > 0 then

[Delphi.Narium]

Anderer Ansatz: 'nen Nachfahren von List (was auch immer das jetzt genau ist) bauen. Für 'ne TStringList sinngemäß sowas (mit Delphi 7 probiert und funktioniert):

Delphi-Quellcode:

unit Unit1;

interface

uses
  Windows, Messages, SysUtils, Variants, Classes, Graphics, Controls, Forms,
  Dialogs, StdCtrls;

type
  tLoopFunc = Function(AString : String) : Boolean;

type
  tForLoopList = class(TStringList)
  private
    { Private-Deklarationen }
  public
    function DoLoop(AProc : tLoopFunc) : Boolean;
  end;

type
  TForm1 = class(TForm)
    Button1: TButton;
    procedure Button1Click(Sender: TObject);
  private
    { Private-Deklarationen }
  public
    { Public-Deklarationen }
  end;

var
  Form1: TForm1;

implementation

{$R *.dfm}

function DieEigentlicheArbeit(AString : String) : Boolean;
begin
  if aString = 'x' then begin
    // mach noch was ...
    Result := True;
  end else Result := False;
end;

function DieAndereArbeit(AString : String) : Boolean;
begin
  if aString = 'x' then begin
    // mach noch was ....
    ShowMessage('Eine Zeile mit x');
    Result := True;
  end else Result := False;
end;

function tForLoopList.DoLoop(AProc : TLoopFunc) : Boolean;
var
  i : Integer;
begin
  Result := False;
  if Not Assigned(AProc) then Exit;
  for i := 0 to Self.Count - 1 do begin
    Result := AProc(Self[i]);
    if Result then Break;
  end;
end;

procedure TForm1.Button1Click(Sender: TObject);
var
  sl : tForLoopList;
begin
  sl := tForLoopList.Create;
  // irgendwie jetzt für Inhalt in sl sorgen.
  // sl.LoadFromFile('x.txt');
  sl.Add('x');
  case sl.DoLoop(DieEigentlicheArbeit) of
    true  : sl.DoLoop(DieAndereArbeit);
    false : ShowMessage('war nix');
  end;
  // Ergbenis eventuell abspeichern?
  // sl.SaveToFile('x.txt');
  sl.Free;
end;

end.

[LTE5]

Das mit dem Nachfahren wird nix denn DieEigentlicheArbeit ist immer eine andere und dafür Prozeduren zu erstellen lohnt sich nicht.

[uligerhardt]

Zitat von LTE5:

Sähe das sonst aber nicht so aus?
FÜR i = 0 BIS -1 DANN Weil Liste.Count -1 bedeutet ja 0-1 = -1?

Selbst wenn das ein Problem wäre (was es ja nicht ist), würde die cLoopCount-Variante auch dein if mit abdecken.

Immer noch kein Bedarf für schwerer lesbaren und vermutlich deutlich langsameren Code.
(Mal abgesehen davon traue ich den derzeitigen Profis bei Embarcadero zu, dass sie bei neueren Sprachfeatures einige Überraschungen für dich einbauen. )

[LTE5]

Zitat:

Immer noch kein Bedarf für schwerer lesbaren und vermutlich deutlich langsameren Code.
(Mal abgesehen davon traue ich den derzeitigen Profis bei Embarcadero zu, dass sie bei neueren Sprachfeatures einige Überraschungen für dich einbauen. )

Was genau meinst du mit beiden Sätzen?

[HolgerX]

Hmm..

Ich würd es so machen (old school, D6)

Delphi-Quellcode:

unit Unit1;

interface

uses
  Windows, Messages, SysUtils, Variants, Classes, Graphics, Controls, Forms,
  Dialogs, StdCtrls;

type
  TForm1 = class(TForm)
    Button1: TButton;
    procedure Button1Click(Sender: TObject);
  private
    function DoWas(AStrings : TStrings; AIndex : integer):boolean;
  public
    { Public-Deklarationen }
  end;

var
  Form1: TForm1;

implementation

{$R *.dfm}
type
  TExecuteStrings = function(AStrings : TStrings; AIndex : integer):boolean of Object;
  TExecuteList = function(AList : TList; AIndex : integer):boolean of Object;

procedure ExecuteList(AStrings : TStrings; AFunc : TExecuteStrings);overload;
var
  i : integer;
begin
  for i := 0 to AStrings.Count-1 do
    if AFunc(AStrings,i) then break;
end;

procedure ExecuteList(AList : TList; AFunc : TExecuteList);overload;
var
  i : integer;
begin
  for i := 0 to AList.Count-1 do
    if AFunc(AList,i) then break;
end;


procedure TForm1.Button1Click(Sender: TObject);
var
  s : TStringList;
begin
  s := TStringList.Create;
  try
    s.Add('Test1');
    s.Add('Test2');
    s.Add('Test3');
    ExecuteList(s, DoWas);
  finally
    s.Free;
  end;
end;

function TForm1.DoWas(AStrings: TStrings; AIndex: integer): boolean;
begin
  ShowMessage(AStrings[AIndex]);
  Result := AStrings[AIndex] = 'Test2';
end;

Da ich der Übergebenden Funktion auch die Liste selber mitgebe (nicht nur den Index), kann die Funktion direkt mit der richtigen Liste arbeiten.

Im Beispiel oben ist nur zwischen TList und TStrings unterschieden.

Wenn Du vorhast (wie z.B. in Lua möglich) eine Funktion zu übergeben, welche unterschiedliche Anzahl/Typen an Parameter hat, dann dürfte dies schwierig werden, vor allem, da deine Loop-Funktion nicht wüste, was an welcher Stelle im Funktionsaufruf übergeben werden muss.