Array mit Recordinhalten löschen

[himitsu]

Gerade für die Datensicherheit beim Bearbeiten von Arrays hab ich endlich mal einen Vorschlag gemacht. (darf man gerne für voten)
http://qc.embarcadero.com/wc/qcmain.aspx?d=110391
Die schnelleren Zugriffe über Move und Co. sind da ja schonmal ganz schön fehleranfällig.

Derartige Codes kann man aber auch schon in älteren Delphis nutzen, außer daß man sich dort die Funktionen für jeden Arraytypen einzeln implementieren müste/sollte.

[Sir Rufo]

Zitat von himitsu:

Gerade für die Datensicherheit beim Bearbeiten von Arrays hab ich endlich mal einen Vorschlag gemacht. (darf man gerne für voten)
http://qc.embarcadero.com/wc/qcmain.aspx?d=110391

Kannst du mir erklären, wie das hier funktioniert, denn das verstehe ich nicht (in der class procedure Swap)

Delphi-Quellcode:

  if @Value = @Value then
    exit;

Müsste das nich eher

Delphi-Quellcode:

  if @Value1 = @Value2 then
    exit;

heißen?

[himitsu]

Dieser böse Copy&Paste-Notepadprogrammierung-Codevervollständigung funktioniert auch nich immer.


Hmm, warum kann ich via WebQC nix editieren, was über QCApp geschrieben wurde?

[Bjoerk]

Himitsu, in der Insert ist das überflüssig und in der Remove sogar falsch?

FillChar(Values[Index], SizeOf(T), 0);

[himitsu]

Ins Insert muß es rein, denn wenn das z.B. ein Array of String oder IInterface ist, dann muß dort der Speicher freigegeben/genullt werden, ohne die Referenzzählung zu nutzen.
Aber beim Remove hast du Recht, da geht es auf den falschen Index. (muß Values[High(Values)] nullen)

Statt FillChar könnte man zwar auch InitializeRecord verwenden, aber FillChar dürfte locker ausreichen.

Bei Remove und Insert wurden Felder kopiert, womit eines der Felder nun doppelt existiert, aber die doppelte "Referenz" nicht in eventuell vorhandenen Referenzzählungen beachtet wurde, darum muß diese eventuelle Referenz "vernichtet" werden, bzw. es wirde für gewisse Typen die Speicherverwaltung nicht angepaßt.
(dyn. Array, String, Interface, Variant und Records mit den Zuvorgenannten)



Nja, Emba wird doch hoffentlich seine neu eingefühten Unittests darauf anwenden, bzw. bei Übernahme der Codes nochmal einen Blick drauf werfen.