Inhalt TStringList in andere TStringList kopieren

[Antonov]

Das mit der Groß-Kleinschreibung muss mir nicht gefallen, aber Recht hast du schon. Ist wie in Physik mit den Einheiten oder Beschriftungen. Ohne sie versinkt man im Chaos.

Das mit Result:= TStringlist.Create ist mir ziemlich neu, ich habe noch nicht gesehen, dass man sie erst deklarieren muss. Andere Funktionen von mir kommen ohne diese Deklaration aus.
Und ich möchte den Inhalt einer Stringliste kopieren, und nicht nur einen Zeiger oder auf sie referenzieren. Lustigerweise macht man nicht Result.Free; am Ende, obwohl man die Liste kreeiert hat. Hm.

p.s. Das mit Try Finally habe ich erst seit kurzem gelernt und habe es in diesem Relikt von Programm noch nicht ... Betonung auf "noch"

[mkinzler]

Zitat:

Lustigerweise macht man nicht Result.Free; am Ende, obwohl man die Liste kreeiert hat. Hm.

Du willst diese ja auch zurückgeben und nicht nur ein Nil

[Popov]

Zitat von Antonov:

Das mit Result:= TStringlist.Create ist mir ziemlich neu...

Wieder was gelernt

Wenn du nur eine 08/15 Variable zurück geben willst, wie String oder Integer, das sind keine Objekte, da musst du ja auch nicht machen. Wenn es eine String-Funktion ist, dann ist Result eben ein String. Anders sieht es aber aus wenn die Funktion ein Objekt ist. In dem Fall muss man das Objekt erst erzeugen (und den Speicher reservieren). Ohne das hast du nur eine Adresse, im Grunde ein Integer, aber keinen reservierten Speicher. Das Ergebnis ist ein Error.

Das Result.Free habe ich nicht vergessen. Das findet sich in der anderen Prozedur (siehe slResult.Free; ). Man kann Objekte auch über mehrere Prozeduren verteilen. Nur darf man nicht vergessen es frei zu geben.

Ansonsten macht AddStrings das wofür es gedacht ist: es fügt die Liste von einer TStringList an eine andere an. Man kann es aber auch zum Kopieren "missbrauchen".

[haentschman]

Hallöle...

mal ein wenig Lesestoff zum Thema: Objektinstanzen als Rückgabewert von Functionen und warum man das nicht tun sollte.
http://www.detlef-heibing.de/program...rueckgabe.html
http://forum.delphi-treff.de/index.p...-von-Funktion/

Grundsatz:
Wer erzeugt, gibt frei.
Ausnahme:
Wenn aus dem Funktionsnamen hervorgeht das ein Objekt erzeugt wird. function CreateBlubbList: TSTringlist; Diese Funktion macht dann auch nix anderes als das Objekt zur Verfügung zu stellen. Damit wird deutlich das ich mich selbst um die Freigabe des Objektes kümmern muß. Man darf nicht nur an sich denken sondern auch an den der die Funktion ggf. mal verwendet.

[p80286]

Bitte lass das!

Zitat von haentschman:

Grundsatz:
Wer erzeugt, gibt frei.

Die Konsequenz hiervon ist es, die benötigte Liste als Parameter an eine Procedure zu übergeben. Aus eigener Erfahrung weiß ich, das Funktionsergebnisse manchmal anfangen ein Eigenleben zu führen.
Also nicht
Persliste:=HolePersliste; Sondern

Delphi-Quellcode:

persliste:=TStringlist.Create;
FuellePersliste(persliste);
..
persliste.Free;

Gruß
K-H

[FarAndBeyond]

Ich würde in Verbindung mit TStringlisten immer FreeAndNil verwenden.

FreeAndNil(stringlist_xyz);
oder
stringlist_xyz.Free;
stringlist_xyz := Nil;

Dann kann man auch ohne Probleme Assign benutzen wenn man's braucht...
Und wann immer möglich mit Nil initialisieren...


Gruß
Martin

[vagtler]

Zitat von haentschman:

[...] Man darf nicht nur an sich denken sondern auch an den der die Funktion ggf. mal verwendet.

"Code as if the next guy to maintain your code is a homicidal maniac who knows where you live."

[BUG]

Zitat von p80286:

Die Konsequenz hiervon ist es, die benötigte Liste als Parameter an eine Procedure zu übergeben. Aus eigener Erfahrung weiß ich, das Funktionsergebnisse manchmal anfangen ein Eigenleben zu führen.

Ein in C++ übliches Muster ist es, die Referenz auch wieder zurückzugeben. Das macht den Datenfluss schön sichtbar.

Delphi-Quellcode:

persliste := FuellePersliste(TStringlist.Create);
..
persliste.Free;

[bernau]

Ich sehe es so wie haenschman. Wer erzeugt gibt frei.

Allerdings gibt es auch Fälle, bei denen ich in einer Funktion ein Objekt erzeuge und dann bestimmte Parameter setzte.

Anstatt eine sortierte Stringliste herkömmlich zu erzeugen:

Delphi-Quellcode:

lStringlist:=TStringlist.create;
lStringlist.sorted:=true;
lStringlist.duplicates = dupIgnore;

mache ich mir eine Funktion

Delphi-Quellcode:

function TStringlistSortedCreate:TStringlist;
begin
  result:=TStringlist.create;
  result.sorted:=true;
  result.duplicates = dupIgnore;
end;

Verwendet wird es so:

  lStringlist:=TStringlistSortedCreate;
Wie gesagt. Nur ein Kurzes Beispiel um aus drei Zeilen eine Zeile zu machen.

Finde ich ganz angenehm.

[Popov]

Eine Funktion die als Result ein Objekt zurück gibt ist ja schön und manchmal gibt es keine andere Möglichkeit, aber das muss man immer zu gut aufpassen, nichts zu vergessen.

Ich persönlich favorisiere diese Methode. Da wird nichts verteilt:

Delphi-Quellcode:

procedure MachWas(Bmp: TBitmap);
begin
  Bmp.Canvas.Brush.Color := clRed;
  Bmp.Width := 400;
  Bmp.Height := 200;
end;

procedure TForm1.Button1Click(Sender: TObject);
var
  Bmp: TBitmap;
begin
  Bmp := TBitmap.Create;
  try
    MachWas(Bmp);
    Canvas.Draw(10, 10, Bmp);
  finally
    Bmp.Free;
  end;
end;