Wie mit dem Fehler in einer Klasse umgehen?

[Sir Rufo]

Wenn ich Klassen erstelle, dann wird im constructor als Erstes die Gültigkeit der Argumente geprüft.

Delphi-Quellcode:

constructor TFoo.Create( ABar : TBar );
begin
  // Guard-Section
  if not Assigned( Bar ) then
    raise EArgumentNilException.Create( 'ABar' );

  inherited Create;

  FBar := ABar;
  FSomething := TSomething.Create;
end;

destructor TFoo.Destroy;
begin
  FSomething.Free;
  inherited;
end;

Wird jetzt eine Instanz erzeugt, dann ja nach diesem Pattern:

Delphi-Quellcode:

LFoo := TFoo.Create( nil ); // Bei einer Exception ist hier schon Ende
try
  // mit LFoo arbeiten
finally
  LFoo.Free;
end;

In einer Schleife

Delphi-Quellcode:

LList := TObjectList.Create;
try
  while LList.Count < 1000 do
  begin
    LList.Add( TFoo.Create( nil ) ); // Bei einer Exception ist hier schon Ende
  end;

  // mit LList arbeiten

finally
  LList.Free; // aber das hier wird auch noch abgearbeitet
end;

Also immer alles sauber

@bernau

Mit diesem Konzept wünsche ich viel Glück bei der Fehlersuche ... denn das wirst du brauchen

[bernau]

Zitat von Sir Rufo:

@bernau Mit diesem Konzept wünsche ich viel Glück bei der Fehlersuche ... denn das wirst du brauchen

Popov hatte nicht geschrieben, daß er den Durchlauf komplett abbrechen möchte, wenn bei einem Objekt etwas schief geht. Kann ja sein, daß er mit den "guten" Objekten arbeiten will und die "schlechten" Objekte auswerten will.

Delphi-Quellcode:

LGuteList := TObjectList.Create;
LGuteList.ownsobjects := True;
LSchlechteList := TObjectList.Create;
LSchlechteList.ownsobjects := True;
try
  for a := 0 to 1000 do
   begin
     // hier irgendwie die Daten sammeln, die dem Objekt übergeben werden
     lFoo := TFoo.Create( irgendwelcheDaten );
     if lfoo.CreateError then
       LSchlechteList.add(lfoo)
     else
       LGuteList.add(lfoo)
   end;

  MachJetztWasMitDerGutenList;

  if IchWillDieSchlechteListAuswerten and (LSchlechteList.count>0)   then
    MachJetztWasMitDerSchlechtenList;
   
finally
  LGuteList.free;
  LSchlechteList.free;
end;

Was ist daran nicht sauber und warum brauche ich hierbei Glück?

[Dejan Vu]

Leute, einfach keine Logik in den Konstruktor.

Wenn es eine Logik für die Eigenschaft 'bin ich gültig?' gibt, wieso sollte man das über den Konstruktor abbilden? Das ist doch... nun ... ungünstig.

[stahli]

So etwas wollte ich vorhin auch schreiben. Hatte es aber nicht so schön auf den Punkt gebracht.

[Der schöne Günther]

Siehe auch:

Zitat von Der schöne Günther:

Etwas off-Topic:
Eins habe ich anfangs auch immer gemacht, mittlerweile bereue ich es heftigst: Im Konstruktor eines Objekts anfangen, irgendwelche Dinge zu tun. Hier sogar Kommunikation starten, auswerten, was auch immer. Das ist einer der heftigsten Fehler den ich durchgehend getan habe.

Eine Zusammenfassung in ein paar Zeilen warum nicht liefert der Google C++ Styleguide: Link.
Auf Delphi trifft das natürlich genauso zu.

Wochenend-(Urlaub?)Lesestoff dazu: Link.

[Sir Rufo]

Zitat von Dejan Vu:

Leute, einfach keine Logik in den Konstruktor.

Wenn es eine Logik für die Eigenschaft 'bin ich gültig?' gibt, wieso sollte man das über den Konstruktor abbilden? Das ist doch... nun ... ungünstig.

Die einzige Logik die da rein gehört ist das Validieren der Argumente und wenn da etwas nicht passt, dann werfen wir eine Exception.