verliere Zeiger bei setlength(array,length(array)+1)

[Jan]

Hallo DPler,
ich habe einen Array, und eine TListview, dessen Tlistitem Einträge jeweils einen Zeiger auf die werte des arrays haben. Komischerweise verliere ich den Zeiger auf das letzte Element des arrays, wenn ich den Array um 1 Element erweitere, also setlength nutze. Eine Dereferenzierung des Zeigers auf das vorher letzte element des Arrays löst jetzt eine exception aus. Mache ich da jetzt was falsch, oder ist das ein Bug?
[Edit]

Delphi-Quellcode:

      
      showmessage((pstringlist(form1.listview1.Items[form1.ListView1.Items.Count-1].Data))^.strings[0]);
      setlength(unit1.workarray, length(unit1.workarray)+1);
      showmessage((pstringlist(form1.listview1.Items[form1.ListView1.Items.Count-1].Data))^.strings[0]);

Das erste showmessage gibt den namen des letzten Films in dem array aus, das zweite löst eine exception aus.
[/Edit]
Gruß
Jan

[maximov]

Wenn du ein dynamisches array erweiterst, dann kann es sein das der gesammte speicher, des arrays, verschoben werden muss, weil hinter dem array kein platz mehr ist. Dh. alle pointer, die direkt auf array-einträge referenzierten, sind kaputt. Ich vermute, genau das ist dier passiert.

[Jan]

Hallo Maximov!
Warum gilt dieses problem dann nur für das letzte element des arrays? Alle anderen Zeiger sind noch erhalten.

[jim_raynor]

Zitat:

Zitat von Jan

Hallo Maximov!
Warum gilt dieses problem dann nur für das letzte element des arrays? Alle anderen Zeiger sind noch erhalten.

Sollte eigentlich nicht sein. Beim vergrößern eines Arrays wird immer ein neuer Speicherbereich angefordert, der alte übertragen und anschliessend der alte gelöscht.

P.S:

Delphi-Quellcode:

((pstringlist(form1.listview1.Items[form1.ListView1.Items.Count-1].Data))^

pStringList und die anschliessende derefenzierung ^ ist überflüssig. TStringList ist bereits ein pointer.

Tstringlist(form1.listview1.Items[form1.ListView1.Items.Count-1].Data) müsste also reichen.

[Jan]

Zitat:

Zitat von jim_raynor

Sollte eigentlich nicht sein. Beim vergrößern eines Arrays wird immer ein neuer Speicherbereich angefordert, der alte übertragen und anschliessend der alte gelöscht. .

Das weiß ich das das nicht sein sollte, komischerweise ist es aber so, dass wirklich nur der Zeiger auf den letzten Eintrag des arrays verloren geht.

Zitat:

Zitat von jim_raynor

P.S:

Delphi-Quellcode:

((pstringlist(form1.listview1.Items[form1.ListView1.Items.Count-1].Data))^

pStringList und die anschliessende derefenzierung ^ ist überflüssig. TStringList ist bereits ein pointer.

Tstringlist(form1.listview1.Items[form1.ListView1.Items.Count-1].Data) müsste also reichen.

Sollte eigentlich, dachte ich auch, aber tuts nicht. Gibt ne exception zur runtime.

Danke dass du dich damit befasst!
Gruß
Jan

[maximov]

hmmm...

Das kommt vielleicht daher, dass delphi den Speicherbereich eben nicht löscht, sondern nur zur späteren verwendung aufbewart. Dh. deine alten daten liegen dort nach wie vor und sind in deinem prozess ein gültiger speicherbereich. Was mit dem letzten element ist, weiss ich nicht.

Warum speicherst du dort überhaupt pointer? Warum refenzierst du mit den Data-pointern nicht gleich die Springlisten? Ich würde dir raten das konzept zu ändern. Auch wenn es schmerz. Dafür hast du dann später weniger ärger.

[maximov]

Genau

Zitat:

Code:

Tstringlist(form1.listview1.Items[form1.ListView1.Items.Count-1].Data)

ist viel besser, wobei die referenzen dann nix mehr mit dem array zu tun haben...was auch ganz gut ist :wink:


//edit:

PS:

Schreib es so:

Delphi-Quellcode:

with form1.ListView1 do
  TStringList(Items[pred(Items.Count)].Data)

[Jan]

In data kann man aber nur pointer speichern. Also kann ich da leider garnix dran machen. Ganz optimal ist das alles nicht. Aber dazu kommt dann bald version 2 von dem Tool. Nur sollte die erste Version halt mal klappen.
Gruß
Jan

[maximov]

Zitat:

Zitat von Jan

In data kann man aber nur pointer speichern. Also kann ich da leider garnix dran machen. Ganz optimal ist das alles nicht. Aber dazu kommt dann bald version 2 von dem Tool. Nur sollte die erste Version halt mal klappen.
Gruß
Jan

doch klar: Du kannst einfach einen untypisierten pointer auf alles casten:

Delphi-Quellcode:

TStringList(DeinPointer) := deineSL;
...
deineSL := TStringList(DeinPointer);

Hart casten!

[Muetze1]

Moin!

@maximov: Das dynamische Array arbeitet intern meines Wissens nach wie TList und reserviert nur Speicher für die Ablage der Pointer auf die Elemente. Daher ist eine Vergrösserungen oder Verkleinerung des Arrays nie mit dem Verlust bzw. umkopieren der Daten verbunden - diese bleiben an ihren Speicherplatz. Ansonsten würde es ja nach deiner Erklärung bedeuten, dass ich mir nie ein Element eines Arrays irgendwo merken kann wenn es noch irgendwo anders verändert werden kann (das Array).

Ausnahme: packed Array Of - dann werden diese Daten mit kopiert.

Oder liege ich ganz falsch?

MfG
Muetze1

[Jan]

Zitat:

Zitat von maximov

Genau

Zitat:

Code:

Tstringlist(form1.listview1.Items[form1.ListView1.Items.Count-1].Data)

ist viel besser, wobei die referenzen dann nix mehr mit dem array zu tun haben...was auch ganz gut ist :wink:


//edit:

PS:

Schreib es so:

Delphi-Quellcode:

with form1.ListView1 do
  TStringList(Items[pred(Items.Count)].Data)

Das klappt ja wie gesagt nicht, da bekomme ich eine exception, wenn ich das schreibe:

Delphi-Quellcode:

      with form1.ListView1 do
      showmessage(TStringList(Items[pred(Items.Count)].Data).strings[0]);

Jan

[maximov]

@Muetze: Meines wissens sind arrays immer zusammenhängende speicher bereiche, die sich linear adressieren lassen. Von einem verfahren wie bei TList hab ich noch nie was gehört. Ausserdem würde ich mich dann fragen, warum sie dann dort ein dynamisches array, mit hilfe einer statischen pointer-array-deklaration, faken.

Wäre ja auch unpraktisch. Oder hast du beweise? :mrgreen:

[maximov]

@Jan: Wie ermittelst du den Pointer denn, der in Bla.blas(Blub).data steht?

[Jan]

Die "Data" property wird folgendermaßen gesetzt:

Delphi-Quellcode:

Listview1.Items[counter].Data:=@inarray[i];

inarray ist ein array of tstringlist;
Gruß
Jan

[Muetze1]

Moin!

Nimm das @ Zeichen weg und probiers von neuem. Wenn er dann wegen dem Typ meckert, dann klammere inarray[i] mit Pointer()...

MfG
Muetze1

[maximov]

Noch mal von vorne: :shock:

Die möglichkeit des arrays sind nicht sehr rosig, da trotz dynamischer grösse, immer ein grosser klumpen bleibt. Es wird eine liste von pointern benötigt. Worum nicht eine TListe nehmen oder TObjektliste nehmen? Das spart ärger und zeit. Wenn man aber vorerst nicht zu grossen wert auf die trennung von Model und Control legt, dann könnte man die infrastruktur der TreeNodes nutzen und die daten darin kapseln. Dh. im klartext jeder treenode hat zB. eine property movies. Und TreeView.items dient als grosse liste für den zugriff:

Delphi-Quellcode:

 TMovies = class(TStringList)
  private
  end;

  TMyTreenode = class(TTreeNode)
  private
    function GetMovies: TMovies;
  public
    destructor destroy; override;
    procedure AfterConstruction; override;
  published
    Property Movies:TMovies read GetMovies;
  end;

var
  Form1: TForm1;

implementation

{$R *.dfm}

{ TMyTreenode }

procedure TMyTreeNode.AfterConstruction; override;
begin
  data := TMovies.create;
end;

destructor TMyTreenode.destroy;
begin
  TObject(data).Free;
  data := nil;
  inherited;
end;

function TMyTreenode.GetMovies: TMovies;
begin
  result := TMovies(data);
end;

{ TForm1 }

// event des treeViews
procedure TForm1.TreeView1CreateNodeClass(Sender: TCustomTreeView;
  var NodeClass: TTreeNodeClass);
begin
  // sagen, dass die nodes von der folgenden klasse sein sollen
  NodeClass := TMyTreeNode;
end;

procedure TForm1.CreateNode;
begin
  with TreeView1 do
    if assigned(Selected) then
      Items.AddChild(Selected, InputBox('Sag', 'wie soll mein name sein', 'Node'));
end;

procedure TForm1.Button1Click(Sender: TObject);
begin
  CreateNode;
end;

fertig. und einfach benutzen:

Delphi-Quellcode:

(treeView.items[i] as TMyTreeNode).Moview.Add('delphi der film')

was meint ihr?


//edit:bugs\\
//edit2:Leider noch keine generics

[jbg]

Zitat:

Zitat von maximov

procedure AfterConstruction; override;

Da hat wohl jemand bei Borland ein virtual hinter dem Konstruktor vergessen.

Zitat:

was meint ihr?

Das ist dann die Variante für die, die die Oberfläche mit den Daten verschmelzen wollen und sich dann später Wundern, wo sie jetzt die benötigten Informationen wieder herbekommen. Mit deiner Variante ist das noch ein wenig leichter herhauszubekommen.

[maximov]

Zitat:

Zitat von jbg

Zitat:

was meint ihr?

Das ist dann die Variante für die, die die Oberfläche mit den Daten verschmelzen wollen und sich dann später Wundern, wo sie jetzt die benötigten Informationen wieder herbekommen. Mit deiner Variante ist das noch ein wenig leichter herhauszubekommen.

Ja genau! Glaube aber, dass Jan erstmal besseres zu tun hat, als die saubere trennung von Model, View und Control. Grundsätzlich kann man aber TMyTreeView für eine lose kopplung nutzen, indem man nicht direkt agregiert, sondern eine Observer einsetzt oder dergleichen.