Zugriff auf gemeinsame Eigenschaft in unterschiedlichen Klassen

[norwegen60]

Zitat von hoika:

Benutz Interfaces

Mit denen habe ich mich noch nie abgegeben. Könntest du anzeigen, wie das geht. Meine Bedenken sind bei so was, dass es zuerst ganz einfach aussieht und man dann feststellt, dass man vieles beachten muss, was man noch nicht weiß. Ich bin ziemlich unter Zeitdruck

[stahli]

Interfaces bringen jetzt an der Stelle nicht wirklich viel.

Ich würde auch grundsätzlich zu Interfaces raten, aber man muss sich erst damit befassen. Die Klassen muss man dann aber auch differenziert ausgestalten.

Ich würde es so machen:

Delphi-Quellcode:

  TBase = class
   private
     FNr: Integer;
     FParentObject: TObject;
   public
     property Nr: Integer read FNr write FNr;
     property ParentObject: TObject read FParentObject write FParentObject;
   end;
   
   TMessData1List = class(TBase)
   private
     fItems: (TObjectList<TMessData1>);
     function get_Item(const Index: Integer): TMessData1;
     procedure set_Item(const Index: Integer; Value: TMessData1);
   public
     procedure Add(Item: TMessData1);
     procedure Delete(Item: TMessData1); overload;
     procedure Delete(Index: Integer); overload; // alle Methoden, die für die Liste gebraucht werden
   protected
   public
     property Items[const Index: Integer]: TMessData1 read get_Item write set_Item; // weiß nicht, ob die Syntax so richtig ist
   end;

[norwegen60]

Hallo Stahli,

ja, das sieht gut aus. Bedeutet aber auch einiges an Umstellarbeit. Die würde ich gerne verschieben. Deshalb die Frage ob mein Ansatz funktioniert?

Delphi-Quellcode:

TBaseList = class(TObjectList<TBase>)
  privat
    FParentObject: TObject;
  public
    property ParentObject: TObject read FParentObject write FParentObject;
  end;

TMessData1List = class(TBaseList)
  privat
  public
  end;

Oder habe ich da ein Gedankenfehler?

Danke
Gerd

[stahli]

Jetzt ohne Delphi kann ich gar nicht sagen, ob das compiliert (ich würde vermuten, dass nicht).
Wenn es compiliert müsstest Du immer prüfen, von welcher Klasse das jeweilige ParentObjekt ist. Du musst also immer mit is und as casten.


Mein Ansatz ist nicht so aufwendig, wie man zunächst vermuten würde.
Du musst nur eine neu benötigte Listenmethode neu implementieren, wenn Du sie erstmalig brauchst. So schlimm ist das nicht.

Außerdem könntest Du auch Dein Parent als ParentObject als TBase verwalten.

FParentObject: TTBase; Dann kannst Du immer weiter die Parents durchlaufen und musst nicht unterscheiden, was das für ein tatsächliches Objekt ist. Es muss immer nur von TBase abgeleitet sein.

[norwegen60]

Kompilieren lässt es sich und das mit dem Cast ist bekannt. Meine Erfahrung ist nur dass sich die Probleme erst später zeigen. Drum meine Rückfrage ob es etwas gibt, was dagegen spricht.

Die Listen werden jetzt schon recht intensiv genutzt. Umgestellung ist vielleicht wirklich überschaubar. Ich scheue (noch) den Testaufwand.

Bei deinem Ansatz FParentObject: TBase bin ich nicht ganz sicher. Darf bei der Deklaration von TBase eine Eigenschaft auf sich selber zeigen. Oder muss ich dann och eine Klasse davor setzen?
Grüße

[stahli]

... darf es.

[norwegen60]

Zitat von stahli:

Ich würde es so machen:

Delphi-Quellcode:

  TBase = class
   private
     FNr: Integer;
     FParentObject: TObject;
   public
     property Nr: Integer read FNr write FNr;
     property ParentObject: TObject read FParentObject write FParentObject;
   end;
   
   TMessData1List = class(TBase)
   private
     fItems: (TObjectList<TMessData1>);
     function get_Item(const Index: Integer): TMessData1;
     procedure set_Item(const Index: Integer; Value: TMessData1);
   public
     procedure Add(Item: TMessData1);
     procedure Delete(Item: TMessData1); overload;
     procedure Delete(Index: Integer); overload; // alle Methoden, die für die Liste gebraucht werden
   protected
   public
     property Items[const Index: Integer]: TMessData1 read get_Item write set_Item; // weiß nicht, ob die Syntax so richtig ist
   end;

Jetzt wollte ich das mal ausprobieren und beim Durchspielen stellte ich fest, dass entweder du oder ich einen Gedankenfehler haben.
Problem: Wenn ich auf einem der Einträge MessData1 stehe muss ich ja wieder zuerst durch die Liste um zu wissen zu wem die Liste gehört. Jetzt könnte ich natürlich MessData1 als ParentObject TMessData1List zuordnen, aber wie schon gesagt, kann es auch sein dass ich nicht eine Class finden will sondern eine darüber liegende Liste.

Oder sehe ich was nicht?

Grüße
Gerd

[stahli]

Zitat von norwegen60:

Wenn ich auf einem der Einträge MessData1 stehe muss ich ja wieder zuerst durch die Liste um zu wissen zu wem die Liste gehört. Jetzt könnte ich natürlich MessData1 als ParentObject TMessData1List zuordnen, aber wie schon gesagt, kann es auch sein dass ich nicht eine Class finden will sondern eine darüber liegende Liste.

Ich sehe da jetzt nicht das Problem.
Du könntest in der Basisklasse eine Suchfunktion, die dann nach oben iteriert:

Delphi-Quellcode:

TBase = class
private
  FNr: Integer;
  FParentObject: TBase;
public
  function FoundParentClass(aClassName: String): TBase;
  function FoundParentItem(aItem: TBase): Boolen;
  property Nr: Integer read FNr write FNr;
  property ParentObject: TBase read FParentObject write FParentObject;
end;


function TBase.FoundParentClass(aClassName: String): TBase;
begin
  Result := nil;
  if Assigned(ParentObject) then
  begin
    if (ParentObject.ClassName = aClassName) then
      Exit(ParentObject);
    Result := ParentObject.FoundParentClass(aClassName);
  end;  
end;

function TBase.FoundParentItem(aItem: TBase): Boolen;
begin
  Result := False;
  if Assigned(ParentObject) then
  begin
    if (ParentObject = aItem) then
      Exit(True);
    Result := ParentObject.FoundParentItem(aItem);
  end;  
end;

So kannst Du dich von einer beliebigen Stelle aus weiter nach oben hangeln.
Ob die Listeneinträge die Liste als ihren Parent sehen sollen, musst Du abhängig von der Programmlogik entscheiden.

Zitat von norwegen60:

Auch dieses Konstrukt hat einen großen Nachteil. Was vorher so aussah
[DELPHI] lBaseSample := TBaseSample(FSamplingList[i]);

Das ist aber auch nicht mein Vorschlag.


Kannst Du mal ein Unit hochladen, in der diese Klassen vollständig enthalten sind?
Es bräuchte ja keine GUI vorhanden sein - nur eine kompilierbare Unit. Dann könnte man (ich?) die mal entsprechend umbauen.

[norwegen60]

Hallo Stahli,

zuerst mal Danke für deine Unetrstützung.

Zitat von stahli:

Delphi-Quellcode:

TBase = class
private
  FNr: Integer;
  FParentObject: TBase;
public
  function FoundParentClass(aClassName: String): TBase;
  function FoundParentItem(aItem: TBase): Boolen;
  property Nr: Integer read FNr write FNr;
  property ParentObject: TBase read FParentObject write FParentObject;
end;

function TBase.FoundParentItem(aItem: TBase): Boolen;
begin
  Result := False;
  if Assigned(ParentObject) then
  begin
    if (ParentObject = aItem) then
      Exit(True);
    Result := ParentObject.FoundParentItem(aItem);
  end;  
end;

Ich glaube ich weiß wo mein Denkfehler liegt. Ich hatte bei MessData1 als Parent weiterhin die Liste (FItems) eingetragen. Damit musste ich natürlich wieder durch diese Liste. Sobald ich aber sage MessData1.ParentObject := MessData1List mit MessDataList = TMessDataList =class(TBase) sollte es gehen.

Zitat von stahli:

Das ist aber auch nicht mein Vorschlag.

Ich weiß. Ich will nur verstehen und habe auch da mal mit meinem Ansatz weiter getestet.

Zitat von stahli:

Kannst Du mal ein Unit hochladen, in der diese Klassen vollständig enthalten sind?
Es bräuchte ja keine GUI vorhanden sein - nur eine kompilierbare Unit. Dann könnte man (ich?) die mal entsprechend umbauen.

Der Code ist in 20 Jahren gewachsen und wir sind aktuell daran, ihn aufzuräumen. Noch aber sind viel zu viele Abhängigkeiten da. Wenn ich nicht weiter komme, versuche ich mal was zu extrahieren.

Ich hatte noch folgenden Ansatz:

Delphi-Quellcode:

  TMyObjectList<T> = class(TObjectList<T>)
  private 
    FParentObject: TBase;
  public
    property ParentObject: Integer read FParentObject write FParentObject;
  end;

  TBaseList = class(TMyObjectList<TBase>);
 
  // Und die Suche nach einer bestimmten Klasse, z.B. TAnalysis  
  lObject := ParentObject;
  repeat
    if lObject is TAnalysis then               // gefunden
      Result := TAnalysis(lObject)
    else if ParentObject is TBase then         // Wenn TBase eine Klasse höher
      lObject := TBase(lObject).ParentObject
    else if ParentObject is TBaseList then     // Wenn Liste eine Klasse höhr
      lObject := TBaseList(lObject).ParentObject
    else
      lObject := nil;
  until (Result <> nil) or (lObject = nil);

Der code

Delphi-Quellcode:

else if ParentObject is TBaseList then
      lObject := TBaseList(lObject).ParentObject

funktioniert aber nicht nicht. Geht das irgendwie?