Zugriffsverletzung beim TObjectDictionary.Add

[Stevie]

Zitat von haentschman:

Zitat:

Ich glaub, ich mach mal für die Forentage 2020 einen "Fehleranalyse 101" Vortrag

...Danke für die aufbauenden Worte. Kannst du dir vorstellen, daß ich für das Debuggen von "Bibliotheken" keinen Nerv/Zeit habe?

Ich verwende Dictionaries ständig. Einen Comparer habe ich noch nie benutzt. Ich erstelle die Instanz wie immer meistens mit [doOwnsValues].

Danke...

Hilfe zu leisten kostet natürlich keine Zeit...

Es geht nur darum, dass wenn bestimmte grundlegenden Dinge schon vorab passiert sind (und das ist nun keineswegs nur auf dich und diesen Fall bezogen) man entweder schon selbst drauf kommt oder es anderen erleichtert, Hilfe zu leisten. Das ist glaube ich, im Interesse aller.

Ist wie mit dem Fisch geben oder Angeln beibringen. Klar, wenn man keine Lust oder Zeit auf Angeln hat, kann man auch andere nach nem Fisch fragen (oder man kauft ihn im Supermarkt)

Punkt ist, dass man keinen comparer angeben muss, der wird intern gebaut/bezogen. Wenn der also plötzlich nil ist, kann es sich nur um Vorzeitiges Freigeben des Dictionaries handeln oder was ich eher für unwahrscheinlich halte Speicherüberschreiben dieses Objekts durch eine andere Stelle.

[Der schöne Günther]

Und was ist "TSEAMUserRoleRightDict"? Ist das immer noch ein Alias? Oder eine eigene Unterklasse welche z.B. nur "inherited" oder "inherited Create" im Konstruktor sagt?

Ein Codeschnippsel zum Nachstellen wäre echt super-hilfreich.

[haentschman]

TSEAMUserRoleRightDict war ein Tippfehler.
Korrekt wäre wie in den Codeschnippseln TUserRoleRightDict.

Danke...

[haentschman]

Moin...

Zitat:

Eine Idee habe ich noch...aber erst morgen früh.

...Fazit: Eine Objektkopie mit Marshal/UnMarshal nimmt den Comparer nicht mit.

Danke für eure Hilfe.

[Incocnito]

Objektkopie?
Hat das Objekt ein "Assign", oder wie genau?

Delphi-Quellcode:

StringList1 := TStringList.Create();
StringList2 := StringList1;
StringList1.Free();
StringList2.Add('Something'); // <- Alles Kaputt, so geht das natürlich nicht!

Für den höchst unwahrscheinlichen Fall, dass du das meintest.

[haentschman]

Zitat:

Hat das Objekt ein "Assign", oder wie genau?

Normal macht man das klassisch mit Assign. Meine "Datenobjekte" haben auch Listen mit Objekten. Deshalb benutze ich gern folgendes:

Delphi-Quellcode:

class function TToolsJson.ObjectCopyJson(Value: TObject): TObject;
var
  MarshalObj: TJSONMarshal;
  UnMarshalObj: TJSONUnMarshal;
  JSONValue: TJSONValue;
begin
  Result := nil;
  MarshalObj := TJSONMarshal.Create;
  try
    UnMarshalObj := TJSONUnMarshal.Create;
    try
      JSONValue := MarshalObj.Marshal(Value);
      try
        if Assigned(JSONValue) then
          Result := UnMarshalObj.Unmarshal(JSONValue);
      finally
        JSONValue.Free;
      end;
    finally
      UnMarshalObj.Free;
    end;
  finally
    MarshalObj.Free;
  end;
end;

...nur leider funktioniert das nicht immer. Wie man an dem o.g. Fehler erkennen kann, werden Dictioniaries nicht wieder richtig erzeugt. Ich habe deshalb wieder nur auf Liste umgestellt.

[Der schöne Günther]

Zitat von haentschman:

Moin...

Zitat:

Eine Idee habe ich noch...aber erst morgen früh.

...Fazit: Eine Objektkopie mit Maschal/UnMarschal nimmt den Comparer nicht mit.

Ja, leider.
Siehe auch hier:
https://stackoverflow.com/a/49636496/2298252

[Incocnito]

Sowas geht auch nur ansatzweise?
Ich hätte gesagt, da es sich ja nur um den Typen "TObject" handelt,
weiß Delphi eh nicht, was zu tun ist und stellt nur irgendeinen Mist her.
Ich bin total verwirrt! Das ist für mich schon quasi EDV-Magie!

[Uwe Raabe]

Zitat von Incocnito:

Das ist für mich schon quasi EDV-Magie!

Das dazugehörige Zauberwort heißt RTTI.

[Stevie]

Zitat:

Eine Objektkopie mit Marshal/UnMarshal nimmt den Comparer nicht mit.

Unwesentliches Detail, was bisher fehlte

Lösung kann auch so aussehen:

Delphi-Quellcode:

TUserRoleRightDict = class(TObjectDictionary<Integer, TUserRoleRight>)
  constructor Create; overload;
end;

constructor TUserRoleRightDict.Create;
begin
  inherited Create;
end;

Dann ruft das JSON dings den richtigen ctor auf.

P.S. Das Problem wird übrigens auch bei TObjectList<T> passieren. Denn dann wird der parameterlose ctor von TList<T> aufgerufen und dann bleibt FOwnsObjects auf false. Nur als Hinweis, falls mal irgendwo massive Leaks auftreten.