[FMX] Thread läuft nicht weiter

[Devil1925]

Moin,
ich habe in einer Android App einen Thread erstellt, um zwei Querys zu öffnen, welche teilweise länger brauchen. Dies tue ich, damit Android nicht meldet dass die App nicht mehr Reagiert.
Da ich gerade erst Anfange mit Threads zu arbeiten, bitte ich darum böse anfängerfehler zu entschuldigen /.\

Den Thread Erstelle ich Folgendermaßen:

Delphi-Quellcode:

type TLoadingThread = class(TThread)
  protected
    procedure Execute; override;
  end;

procedure TLoadingThread.Execute;
begin
  FrmKundenwahl.FilterSetzen;
  Synchronize(FRMKundenwahl.ShowErgebnis);
end;

procedure TfrmKundenwahl.BT_SuchenClick(Sender: TObject);
var ThLoading:TLoadingThread;
begin
  R_Loading.Visible := True; //Rectangle mit einem Ladekreis drauf, überdeckt die gesamte Anwendung.

  ThLoading := TLoadingThread.Create(True);
  ThLoading.FreeOnTerminate := True;
  ThLoading.Start;

End;

Die Prozedur "FilterSetzen" sieht folgendermaßen aus:

Delphi-Quellcode:

procedure TfrmKundenwahl.FilterSetzen;
var filt:String;
Begin
  filt := '1=1';

  if TRIM(E_Sortierfeld.Text) <> '' then
    filt := filt + ' AND Sortierfeld like '+AnsiQuotedStr(Trim(E_Sortierfeld.Text), #39)+' +''%''';

  if TRIM(E_Name.Text) <> '' then
    filt := filt + ' AND (Name1 like '+AnsiQuotedStr(Trim(E_Name.Text), #39)+' +''%'''
                     + ' OR Name2 like '+AnsiQuotedStr(Trim(E_Name.Text), #39)+' +''%'')';

// ... noch ein paar weitere solcher abfragen und das öffnen der Querys.
End;

Die "ShowErgebnis" Prozedur sieht folgendermaßen aus:

Delphi-Quellcode:

procedure TfrmKundenwahl.ShowErgebnis;
Begin
  Show_Tab_Ergebnis.ExecuteTarget(self);
  R_Loading.Visible := False;
end;

Wird das Ganze beim ersten mal aufgerufen, so läuft alles ohne Probleme durch. Rufe ich das allerdings später ein zweites mal auf, so läuft es nur noch bis zu folgender Stelle in "FilterSetzen":

Delphi-Quellcode:

  if TRIM(E_Sortierfeld.Text) <> '' then
    filt := filt + ' AND Sortierfeld like '+AnsiQuotedStr(Trim(E_Sortierfeld.Text), #39)+' +''%''';

wenn ich hier nun im Debug versuche mit F8 weiter zu gehen, kommt keine Fehlermeldung oder Ähnliches, und ich lande nicht beim nächsten Punkt. Die Prozedur wird nie zu einem Ende gebracht.

Kann mir hier einer Verraten, wo ich hier meinen bösen Fehler mache? oder liegt der womöglich in einem anderen bereich vom Quellcode und der gezeigte ist vollkommen richtig?

[Rollo62]

Ich weiss jetzt nicht genau was du da mit den Threads machst,
aber vielleihct versuchst du einfach die DB erst zu connection wenn FormShow aktiviert wurde.
Dann sollten die mobilen Geräte nicht mehr direkt abbrechen ...

Rollo

[Devil1925]

Ich mache das Connect zur DB direkt vor dem öffnen der Querys. (habe ich vergessen zu erwähnen)

Das Thema dabei ist, dass das öffnen der Query länger dauert, und wenn ich das Öffnen der Query dann über den MainThread laufen lasse, ist die App solange blockiert. Während dieser Zeit läuft dann auch kein Ladekreis weiter (TAniIndikator) und auf antippen der App in diesem zusatand erscheint nach einiger Zeit die Meldung "Anwendung Reagiert nicht mehr. Soll die App geschlossen werden?"
Das Versuche ich mit diesem Thread zu verhindern.

[HolgerX]

Zitat:

Zitat von Devil1925 (Beitrag 1383590)

Delphi-Quellcode:

procedure TLoadingThread.Execute;
begin
  FrmKundenwahl.FilterSetzen;
  Synchronize(FRMKundenwahl.ShowErgebnis);
end;

Delphi-Quellcode:

procedure TfrmKundenwahl.FilterSetzen;
var filt:String;
Begin
  filt := '1=1';

  if TRIM(E_Sortierfeld.Text) <> '' then
    filt := filt + ' AND Sortierfeld like '+AnsiQuotedStr(Trim(E_Sortierfeld.Text), #39)+' +''%''';

  if TRIM(E_Name.Text) <> '' then
    filt := filt + ' AND (Name1 like '+AnsiQuotedStr(Trim(E_Name.Text), #39)+' +''%'''
                     + ' OR Name2 like '+AnsiQuotedStr(Trim(E_Name.Text), #39)+' +''%'')';

// ... noch ein paar weitere solcher abfragen und das öffnen der Querys.
End;

Kein Wunder!

Dur greifst aus dem Thread (Execute) über die Funktion FilterSetzen auf die Komponenten (E_Sortierfeld / E_Name) deines MainThreads zu!

Ändere deinen Thread-Aufruf, so dass Du diese Parameter für den Filter an den Thread übergibst und setzte den Filter IM Thread zusammen, ohne auf Edits und Co zuzugreifen!

Oder Übergebe den kompletten SQL inclusive des Filterstrings an den Thread!

Zusätzlich braucht der Thread sein eigenes Query und kann nicht das Query vom Mainthread verwenden! Denn schließlich wird es dort wohl mit einer Visuellen Komponente verknüpft sein!

[Devil1925]

Danke für die Antwort!

Nur um das Richtig zu Verstehen: ich kann nicht aus einem Thread auf Komponenten des MainThread zugreifen? (Also z.B. die Edit Felder) Ich hatte gedacht das das kein Problem wäre, da ich diese ja nur lese und nicht schreibe.

Thema Query: ich kann also das laden der Daten in der Query nich in einen Thread auslagern wenn diese auf einer Visuellen Komponente dargestellt werden?

[mjustin]

Daumenregel: alles in den Thread packen, was er benötigt, so dass er vom "Rest der Welt" unabhängig arbeiten kann.

Sobald irgendetwas zur Threadlaufzeit an ihn übergeben, oder von der Aussenwelt ausgelesen wird, oder auf die Außenwelt zugegriffen wird, ist Alarmstufe Rot :)

Rückgaben von End-Ergebnissen an die Aussenwelt kann man völlig risikolos über die OnTerminate Methode durchführen, die im Kontext des Hauptthreads ausgeführt wird, das funktioniert daher auch ohne Synchronize.

[Devil1925]

Vielen Dank euch, ihr habt mir super weitergeholfen! Hier nochmal wie ich es jetzt gelöst habe (ich hoffe ich habe da jetzt soweit alles Richtig)

Delphi-Quellcode:

type TLoadingThread = class(TThread)
  private
    ThreadQuery:TUniQuery;
    ThreadFilter, ThreadStation:String;
  protected
    procedure Execute; override;
  public
    property Filter : String read ThreadFilter write ThreadFilter;
    property Station : String read ThreadStation write ThreadStation;
  end;

procedure TfrmKundenwahl.FilterSetzen;
Begin
  Filter := '1=1';

  if TRIM(E_Sortierfeld.Text) <> '' then
    Filter := Filter + ' AND Sortierfeld like '+AnsiQuotedStr(Trim(E_Sortierfeld.Text), #39)+' +''%''';

  if TRIM(E_Name.Text) <> '' then
    Filter := Filter + ' AND (Name1 like '+AnsiQuotedStr(Trim(E_Name.Text), #39)+' +''%'''
                     + ' OR Name2 like '+AnsiQuotedStr(Trim(E_Name.Text), #39)+' +''%'')';
  // und halt der Rest der Abfragen
  // ...

  ThLoading := TLoadingThread.Create(True);    //ThLoading ist im Private Teil der Unit Deklariert
  ThLoading.Filter := Filter;
  ThLoading.Station := IntToStr(Station_ID);
  ThLoading.OnTerminate := ShowErgebnis;
  ThLoading.FreeOnTerminate := True;
  ThLoading.Start;
End;
  
procedure TLoadingThread.Execute;
begin
  Verbindungsaufbau;

  Daten.MSSQL_Connection.ExecSQL('if Object_ID(''MOBILE_Kunden_'+ThreadStation+#39+') is not null Drop Table MOBILE_Kunden_'+ThreadStation);
  Daten.MSSQL_Connection.ExecSQL('Select distinct Kunden_Nr as kdn, Sortierfeld as Sortierfeld_Vorschau, Name1 as Name1_Vorschau, Strasse as Strasse_Vorschau, PLZ as PLZ_Vorschau, Ort as Ort_Vorschau, 1 as Selected into MOBILE_Kunden_'+ThreadStation+' from Kundenst where '+ThreadFilter);
  ThreadQuery := TUniQuery.Create(Daten);
  ThreadQuery.Connection := Daten.MSSQL_Connection;
  ThreadQuery.SQL.Text := 'Select * from MOBILE_Kunden_'+ThreadStation;
  ThreadQuery.Open;
End;

procedure TfrmKundenwahl.ShowErgebnis(Sender:TObject);
Begin
  qVorschau := ThLoading.ThreadQuery;
  qVorschau.AfterScroll := Q_VorschauAfterScroll;

  BS_Vorschau.DataSet := qVorschau;
  BS_Vorschau.DataSet.Open;

  Show_Tab_Ergebnis.ExecuteTarget(self);
  R_Loading.Visible := False;
end;

Wenn ich da jetzt noch irgendwelche Bösen Fehler drin habe, die ich bis jetzt nur noch nicht gesehen habe und die mir in zukunft böse auf den Kopf fallen könnten, wäre es super nett, wenn mir die einer Zeigen könnte :)

[Neutral General]

Ich kann falsch liegen, aber bei uns hieß es immer 1 DB-Connection pro Thread.
D.h. Du könntest nicht Daten.MSSQL_Connection benutzen.

Aber da bin ich mir nicht zu 100% sicher.

Das andere Problem wo ich mir ziemlich sicher bin dass du das nicht willst/brauchst:

Delphi-Quellcode:

// qVorschau = ThLoading.ThreadQuery = BS_Vorschau.DataSet
// Die Query wird nicht kopiert! Das ist alles immer noch die selbe wie aus dem Thread.

qVorschau := ThLoading.ThreadQuery;
qVorschau.AfterScroll := Q_VorschauAfterScroll;

BS_Vorschau.DataSet := qVorschau; // <-- ist das gleiche wie BS_Vorschau.DataSet := ThLoading.ThreadQuery
BS_Vorschau.DataSet.Open;  // <-- D.h. das ist im besten Fall unnötig weil deine ThreadQuery schon im Thread geöffnet wurde

Show_Tab_Ergebnis.ExecuteTarget(self);
R_Loading.Visible := False;

Da ist die Frage was du tatsächlich machen/haben willst.
Entweder geht das einfacher oder es tut etwas was du nicht willst. (Je nachdem was du willst)

[Devil1925]

also das mit der MSSQL_Connection funktioniert soweit, kann ich aber mal ändern, damit ich dann auch Sicher bin.

ist die ThreadQuery nicht nach dem ausführen der ShowErgebnis Prozedur weg? weil der Thread dann freigegeben wird?

Ausserdem nutze ich qVorschau noch in anderen Stellen nach dieser zuweisung, indem ich über diese noch felder Editiere etc. geht dass denn dann auch über die ThreadQuery?

[Neutral General]

Wenn du Variablen von Objekten zuweist, dann wird nie das Objekt kopiert.
"ThLoading.ThreadQuery", "qVorschau" und "QueryBS_Vorschau.DataSet" sind 3 verschiedene Namen für das exakt gleiche Objekt was du im Thread erstellt hast.