Rückgabewert könnte undefiniert sein und Exception.RaiseOuterException

[Der schöne Günther]

Das geringste Übel wäre meiner Meinung nach den Rückgabewert unnötigerweise im except-Block zu setzen. Das hätte wohl wenigstens keine Auswirkungen auf den Normalfall:

Delphi-Quellcode:

function f(): Byte;
begin
   try
      Result := 42;
   except
      Exception.RaiseOuterException( EProgrammerNotFound.Create('derp') );
      Result := 0; // unnötig, aber compiler ist happy
   end;
end;

Oder - du setzt das Result einfach nach dem try..except.

[Scurra]

Zitat von Der schöne Günther:

Das geringste Übel wäre meiner Meinung nach den Rückgabewert unnötigerweise im except-Block zu setzen. Das hätte wohl wenigstens keine Auswirkungen auf den Normalfall

Ja, das sehe ich auch so.

Zitat von Der schöne Günther:

Oder - du setzt das Result einfach nach dem try..except.

Das wäre ungünstig, weil dann das Result im Erfolgsfall überschrieben wird Man bräuchte zusätzlich ein Exit am Ende des try-Blocks.

[dummzeuch]

So sollte es am wengsten stören:

Delphi-Quellcode:

function SomeFunction: Integer
begin
  try
    // do something and set result
    // ...
    Result := 0;
  except
    Exception.RaiseOuterException(...);
    Result := 0; // damit der Compiler nicht motzt
  end;
end;

Auch wenn die zweite Zuweisung nie ausgeführt wird, so stellt sie doch den Compiler zufrieden. Der erzeugte Code ist ja minimal. Es könnte allerdings jemanden verwirren, der den Code liest.

[himitsu]

Tja, könnte man bei Funktionen/Methoden angeben, dass sie immer eine Excepion werfen, dann könnte der Compiler das wissen, aber das geht nicht, also weiß er es nicht, dass es danach nicht weitergeht.

Statt dem Result zuzuweisen würde ich hier einfach raise nutzen.
Beides wird nicht ausgeführt, aber beim Raise ist das Verhalten "klarer" und man ist unabhängig davon ob die Funktion ein Result hat oder nicht und welchen Typ die hat.

Delphi-Quellcode:

  try
    // do something and set result
    // ...
    Result := 0;
  except
    Exception.RaiseOuterException(...);
    raise; // damit der Compiler nicht motzt
  end;

Ich find's eh bissl komisch, dass die OuterException so unpraktisch verpackt wurde.