Du bist da offenbar bei Deiner Fragestellung etwas durcheinander gekommen.

Wird fListe nun im Thread neu createt oder wird Liste der fListe zugewiesen (und warum überhaupt)?

Im ersten Fall musst Du fListe freigeben, was keine Auswirkungen auf Liste hat.
Im zweiten Fall entspricht fListe = Liste und Du darfst diese nicht freigeben.

Ich denke, Du musst Dein Problem nochmal konkretisieren...

Entweder das:

zusammenfalten · markieren

Delphi-Quellcode:

thread.Create(const liste:tstringlist);
begin
   fListe:=tstringlist.create;
   fListe.Assgin(Liste);
end;

 thread.execute;
begin
   machwasmitfliste;
    
   //Hier muss fListe aber noch zurück in liste

  fliste.free {ja!!!!}
end;

oder das

markieren

Delphi-Quellcode:

thread.Create(const liste:tstringlist);
begin
   fListe:=liste;
end;

 thread.execute;
begin
   machwasmitfliste;
  //fliste.free {nein!!!
end;

Ein fListe:=TStringList.Create macht ja keinen Sinn, wenn ich danach den der Variable fListe einfach mal auf eine andere STringList zeigen lassen. Und wenn ich das dann noch Freigebe, ist die Globale Liste weg.

Mit der zweite Variante wirst du dir aber über kurz oder lang ärger einhandeln.

Zitat von stahli:

Du bist da offenbar bei Deiner Fragestellung etwas durcheinander gekommen.

Ich denke, Du musst Dein Problem nochmal konkretisieren...

Nö, ich hab mir da nur selbst auf der Leitung gestanden.
Hobbycoder hat mich auf den richtigen Weg gebracht, wobei mir nicht so ganz kar ist warum fListe=liste; mir früher oder später Ärger bereiten sollte, wenn sicher ist, daß nicht mehrere Threads gleichzeitig zugreifen.

Gruß
K-H

Zitat:

markieren

Delphi-Quellcode:

thread.Create(const liste:tstringlist);
begin
  fListe:=tstringlist.create;
end;

thread.execute;
begin
  machwasmitfliste;
 fliste.free {------------------ ja oder nein????}
end;

Nein.

Freigabe natürlich möglichst immer in der selben Ebene, wo es auch erstellt wurde.

Create -> Destroy

Zitat von p80286:

Hobbycoder hat mich auf den richtigen Weg gebracht, wobei mir nicht so ganz kar ist warum fListe=liste; mir früher oder später Ärger bereiten sollte, wenn sicher ist, daß nicht mehrere Threads gleichzeitig zugreifen.

Ich meinte jetzt nicht explizit in diesem Fall, sondern eher im allgemeinen.
Ich kenn ja deinen Restcode nicht. Wenn alles wirklich synchronisiert abläuft, dann sollte es ja gehen. Trotzdem versuch ich sowas zu vermeiden. Schon allein, der Wiederverwendbarkeit halber.

Ein synchronisierter Zugriff auf Daten außerhalb des Thread ist ja nicht unüblich, und letztlich spielt es dabei ja keine Rolle ob es sich bei dem Object um eine Stringlist handelt, ein GUI-Object oder sonst irgendeine Variable, solange es synchronisiert abläuft.

Interessant wird das allerdings beim Debuggen, wenn der Inhalt der Liste von verschiedenen Threads verändert wird. Aber wie gesagt, das hängt natürlich stark von dem Restcode ab.

Und Last-but-not-least, ist das mit dem Freigeben gerade in größeren Projekten so eine Sache, wenn man auf Objecte zugreift, deren Existenz nicht in der Kontrolle des zugreifenden Thread liegt. Entweder es wird einem mal das Object unter dem Arsch weggezogen oder es bleiben Speicherleichen zurück. Das muss zwar nicht grundsätzlich so sein, aber in großen Projekten kommt das schneller vor, als man denkt. Und im zweiten Fall kommt es ja noch nicht mal zu einer Exception.

Vielen Dank an alle!

Gruß
K-H