OOP und ihre Tücken...

[DelphiCreak]

Ich arbeite gerade an einem Dateisystem mit funktionen für Zip's etc, und bin darauf auf ein interessantes verhalten gestoßen:

Ich habe mehrere Klassen für eine Datei(noFile, noFile_Permanent, noFile_Memory, noFile_InZip), um eben die verschiedenen Instanzen von Dateien speichern zu können. Die letzten 3 erben nun wiederum von noFile.

Hat man nun allerdings eine Prozedur a la:

Delphi-Quellcode:

function GetOSFile: noFile;
begin
   Result := noFile_Permanent.Create;
end;

Landet man, wenn man die Rückgabe dieser Prozedur anspricht, bei den Prozeduren der BasisKlasse noFile (z.B. habe ich eine Methode Length, die bei noFile = 0 zurückgibt, bei Permanet die Länge der Datei), statt bei denen von noFile_Permanent.

Nun zur eigentlichen Frage: Warum geht das überschreiben der Prozeduren verloren?
Nach meinem Verständnis von OOP sollte man über das Ergebnis doch jetzt die (überschriebenen) Prozeduren von noFile_Permanent ansprechen. Warum ist das nicht so, oder was mache ich falsch?

[DeddyH]

Ist das eine reguläre Funktion oder eine Methode? Falls Letzteres, wurde sie in der Basisklasse als virtuell deklariert und in der abgeleiteten überschrieben? Im Übrigen bin ich persönlich kein Freund von Funktionen, die eine Objektinstanz zurückgeben, die sie selbst erzeugen.

[DelphiCreak]

In der BasisKlasse ist die Prozedur virtual, und die abgeleiteten Klassen überschreiben die Prozedur dementsprechend...

[DeddyH]

Nunja, der Rückgabetyp ist ja nun einmal vom Typ noFile. Du könntest diesen aber nach dem Funktionsaufruf auf den "Untertyp" überprüfen. Oder man macht es ganz anders und übergibt noch einen Klassentypen, in der Art

Delphi-Quellcode:

type
  noFileClass = class of noFile;

function GetOSFile(aType: noFileClass): noFile;
begin
   Result := aType.Create;
end;

[DelphiCreak]

Werd mal versuchen einen Klassentypen mit einzubauen...auch wenn das imho nicht nötig sein sollte / nichts ändern dürfte...mal schauen

[Edit]
Habs drin gehabt mit klassentyp - keine Veränderung, dann nen Rollback gemacht, und jetzt Läufts...verstehe wer will, ich tus nicht, aber es läuft jetzt.

[Christian Seehase]

Moin Detlef,

Zitat:

Zitat von DeddyH (Beitrag 1182122)

Im Übrigen bin ich persönlich kein Freund von Funktionen, die eine Objektinstanz zurückgeben, die sie selbst erzeugen.

Und wenn sie diese Objekte selber verwalten, wie z.B. TListView das TListItem?

[DeddyH]

Moin Christian,

das ist etwas anderes, aber dann ist der Code auch ausführlicher ;). Als Negativbeispiel sehe ich z.B. GetFormImage an, da man die so ermittelte Instanz selbst freigeben muss, das ist IMO sehr unschön.

[Christian Seehase]

Moin Detlef,

Zitat:

Zitat von DeddyH (Beitrag 1182139)

das ist IMO sehr unschön.

dann sind wir uns ja einig :cheers:

[DeddyH]

Sieht so aus :mrgreen:

P.S.: http://www.detlef-heibing.de/program...rueckgabe.html

[Sir Rufo]

Zitat:

Zitat von DeddyH (Beitrag 1182122)

Ist das eine reguläre Funktion oder eine Methode? Falls Letzteres, wurde sie in der Basisklasse als virtuell deklariert und in der abgeleiteten überschrieben? Im Übrigen bin ich persönlich kein Freund von Funktionen, die eine Objektinstanz zurückgeben, die sie selbst erzeugen.

hmmm, FactoryPattern ... ist eigentlich nicht ungewöhnlich, allerdings so wie es umgesetzt ist ist es keine Factory und da gebe ich dir Recht :)