Objekt in Liste mit anderer Instanz ersetzen ohne Referenz zu verlieren

[norwegen60]

Hallo zusammen,

ich habe eine TMyObjectList = class(TObjectList<TMyObject>); .
Jedes TMyObject hat eine eindeutige ID (ID aus der Datenbank)
Jetzt könnte es sein, dass ich an anderer Stelle bei einer Änderung eine Instanz1.2 bekomme, die schon in der Liste enthalten ist (Instanz1.1).
Da es sein kann, dass irgendwo im Code schon auf Instanz1.1 verwiesen wird (in meinem Fall in zwei TVirtualStringTree), kann ich diese ja nicht einfach in der Liste löschen und dafür Instanz1.2 hinzufügen.
Wie schaffe ich es aber einfach, dass Instanz1.1 die eventuell aktuelleren Daten aus Instanz1.2 übertragen bekommt?

Grüße
Gerd

[mkinzler]

Mit .Assign() kann man Werte von einem Objekt auf das andere übertragen.
<Instanz1.1>.Assign( <Instanz1.2>);

[himitsu]

Aber nur wenn Assign/AssignTo implementiert ist.

Aber das Prinzip stimmt schonmal ... du mußt den Inhalt von 2 in die 1 kopieren / ihr zuweisen ... und sei es, dass du es mit jedem Feld/Property einzeln machen mußt.

[norwegen60]

Zitat von himitsu:

Aber nur wenn Assign/AssignTo implementiert ist.

Aber das Prinzip stimmt schonmal ... du mußt den Inhalt von 2 in die 1 kopieren / ihr zuweisen ... und sei es, dass du es mit jedem Feld/Property einzeln machen mußt.

Genau das war es was ich befürchtet habe und nach einer einfacheren Lösung gesucht habe
OK, in dem gerade aufgetretenen Fall sind es nur 5 Felder, die sich ändern können und damit ist die Implementation von Assign überschaubar. Es ging mir aber ums generelle

[mkinzler]

Um was für Objekte hanldet es sich? Wenn es eigene Klassen sind einfach Assign für diese implementieren.

[norwegen60]

Meine Klasse sieht prinzipiell so aus. Hier schon mit dem Assign, wie ich es verstanden hätte

Delphi-Quellcode:

  
  TMyBaseClass = class;
  private
    FID:Integer;
    FName: String;

    procedure SetID(const Value: Integer);
    procedure SetName(const Value: String);
  public
    property ID: Integer read FID write SetID;
    property Name: String read FName write SetName;

    constructor Create;
    destructor Destroy; override;

    procedure Assign(aNewValue:TMyBaseClass);         

    function FunctionB1: String;         
    procedure ProcB1(sMsg: String); 
  end;

  TMyClass = class(TMyBaseClass);
  private
    FInfo:String;

    procedure SetInfo(const Value: String);
  public
    property Info: String read FInfo write SetInfo;

    constructor Create;
    destructor Destroy; override;

    procedure Assign(aNewValue:TMyClass);         
    function Function10: String;         
    procedure Proc10(sMsg: String); 
  end;

..

Procedure TMyBaseClass.Assign(aNewValue:TMyBaseClass);
begin
  self.ID := aNewValue.ID; // Und damit Möglichkeit, TMyClass komplett zu überschreiben
  self.Name := aNewValue.Name;
end;
  

Procedure TMyClass.Assign(aNewValue:TMyClass);
begin
  inherited (TMyBaseClass(aNewValue));

  self.Info := aNewValue.Info;
end;

Oder geht das einfacher?

Mein Problem ist, dass TMyClass aus ca. 40 Properties besteht und noch Unter- und Überklassen bestehen. Nur Arbeit, alle Properties in Assign aufzunehemen, aber schwierig, den aktuelle zu halten wenn mehrere eventuell neue Properties hinzufügen (Selten, aber kann vorkommen)

[himitsu]

Anders geht es nunmal nicht (effektiv).

Man kann ab TPersistent vererben und dort das Streaming missbrauchen
oder via RTTI alles suchen und kopieren.




Bei sowas, wo man etwas mal vergessen könnte, kann man auch
{$IF SizeOf(TMyRecord) <> 123} {$MESSAGE Error '...'} {$IFEND}

Bei Klassen ist das leider nicht ganz so einfach, direkt im Compiler, daher wohl eher
if Self.InstanceSize <> 123 then raise ...;
[edit] Kannst auch mal nachsehn, wo sich die InstanceSize in der TypeInfo versteckt.

Das in dein Assign und 123 auf den aktuellen Wert setzen.
-> neues Feld = Peng

[norwegen60]

War mir nicht sicher ob Markus mit

Zitat:

Wenn es eigene Klassen sind einfach Assign für diese implementieren.

nicht noch was einfacheres gemeint hat

[himitsu]

Nein.

TPersistent.Assign und seine Freunde sind doof und wissen von nichts.
Ob man nun extern jedes Feld/Property kopiert, oder in einer Methode ala Assign macht ... gemacht werden muß es so oder so.

Sie können es auch garnicht, da sie nicht wissen was und wie es kopiert werden muß, sobald es keine einfachen Typen sind.

[mkinzler]

Wenn das in allen Klassen implemnetiert ist, musst Du ja nur für die "neuen" Eigenschaften implementieren

Delphi-Quellcode:

TBasis = class
 a,b: Integer;
 procedure Assign( Source: TPersistent); virtual;
end;

TTeil = class 
  aa, zz: integer;
  procedure Assign( Source: TPersistent); virtual;
end;

TAbg = class( TBasis)
  c: TTeil;
  d, e: Integer;
  procedure Assign( Source: TPersistent); virtual;
end;
...
procedure TBasis.Assign( Source: TPersistent); 
begin
  a := Source.a;
  b := Source.b;
end;
...
procedure TTeil.Assign( Source: TPersistent); 
begin
  aa := Source.aa;
  zz := Source.zz;
end;
...
procedure TAbg.Assign( Source: TPersistent); 
begin
  inherited Assign( Source); //a, b
  c.Assign( Source.c); //c
  d := Source.d;
  e := Source.e;
end;