Einzelnen Beitrag anzeigen

freimatz

Registriert seit: 20. Mai 2010
1.442 Beiträge
 
Delphi 11 Alexandria
 
#1

Clean Code - Default benutzen

  Alt 10. Aug 2020, 07:57
Hallo,
ich bin gerade am grübeln zu der besten Variante. Gegeben sei ein
Delphi-Quellcode:
  TBlaResult = record
    one, two: Double;
  end;
Dann gibt es ein function irgendwo, da will man das zurücksetzen mit den Werten 0. (In anderen function abers). Was ist da nun am Besten?
Delphi-Quellcode:
//a)
function TBlaNullStrategy.DetermineBla(): TBlaResult;
begin
  Result.one := 0.0;
  Result.two := 0.0;
end;
//b)
function TBlaNullStrategy.DetermineBla(): TBlaResult;
begin
  Result := Default(TBlaResult);
end;
//c)
function TBlaNullStrategy.DetermineBla(): TBlaResult;
begin
  Result := Default(TBlaResult);
  Result.one := 0.0;
  Result.two := 0.0;
end;
Für mich ist bei a) schön, dass man explizit darstellt, dass es in diesem Fall die Werte 0 sind. Der Nachteil ist, dass wenn man mal TBlaResult erweitert und vergisst das hier anzupassen, man undefinierte Inhalte bekommt. (Hat FixInsight auch nicht gemerkt.)
Bei b) kann das nicht passieren und ist auch schön kurz. Es ist allerdings nicht explizit, man muss wissen dass der Default-Wrt von TBlaResult eben dann auch 0 ist.
c) wäre eine Kombination. Es kann nicht passieren und es ist auch schön explizit. Irgendwie aber auch DRY und auch mehr Code.
Meinungen?
  Mit Zitat antworten Zitat