* aus einem Thread darf man nicht auf VCL Objekte des Hauptthreads zugreifen - ausser wenn der Zugriff mit synchronize geschützt wird
* im Thread MThread.Synchronize(doprogress) aufzurufen ist unnötig, stattdessen einfach Synchronize(doprogress) genügt
* Sind die Datenbankconnection threadsafe? Falls nein oder unbekannt, muss der Thread eine eigene Connection verwenden

Für Grid-Anzeigen empfiehlt es sich mit Limit/Top zu arbeiten um die Anzahl der Datensätze und die Dauer des Abarbeitens zu beschränken.
Dann braucht man keine Threads.

Zitat von Bernhard Geyer:

Für Grid-Anzeigen empfiehlt es sich mit Limit/Top zu arbeiten um die Anzahl der Datensätze und die Dauer des Abarbeitens zu beschränken.
Dann braucht man keine Threads.

Weil wir hier wirklich sehr Datenbanklastig arbeiten, wollen wir diese Dinge durch Threads "kapseln", da diese immer, gerade bei den langsameren Rechnern für ein Hängen der Rechner sorgen.

Gruss

Mc

Zitat von mjustin:

* aus einem Thread darf man nicht auf VCL Objekte des Hauptthreads zugreifen - ausser wenn der Zugriff mit synchronize geschützt wird

Genau dafür ist der Thread ja da - er soll an dieser Stelle das Hängen verhindern.

Zitat:

* im Thread MThread.Synchronize(doprogress) aufzurufen ist unnötig, stattdessen einfach Synchronize(doprogress) genügt
* Sind die Datenbankconnection threadsafe? Falls nein oder unbekannt, muss der Thread eine eigene Connection verwenden

Der Thread hat ja eine eigene Connection bzw. diese Form, wo der Thread gestartet wird.
Ob die Developer Expresstools und die Oracle Komponenten von Devart threadsafe sind kann ich nicht sagen. Gilt es zu checken.

Gruss
Mc

So wie der Thread jetzt geschrieben ist ist er nicht nur sinnfrei sondern auch gefährlich ...

Zitat von Bummi:

So wie der Thread jetzt geschrieben ist ist er nicht nur sinnfrei sondern auch gefährlich ...

Es ist ja auch noch eine Spielwiese, wo ich die generelle Funktionalität teste.

Besserer Vorschlag?

Gruss

Mc

Zitat:

markieren

Delphi-Quellcode:

procedure TMyThread.Execute;
begin
  ...
  MThread.Synchronize(doprogress);
  ...

Das mit dem MThread wurde ja schon gesagt.
Wenn man in einer Klasse ist, dann Self verwenden (oder weglassen und das Self implizit nutzen).

Denn rate mal was passiert, wenn zu z.B. 2 Threads erstellst.
Dann greifst du nicht mehr auf den "aktuellen" Thread zu, sondern auf den, welcher in der globalem Variable liegt.

Nja, das mit der VCL und Threads wurde auch schon genannt.
Was du machen kannst, ist das Query vom Grid (der VCL) trennen, zu aktualisieren (im thread) und es dann wieder zu verbinden (im Hautthread).

Oder die Daten im Hintergrund zu laden und dann synchronisiert in ein weiteres Dataset zu kopieren, welches am Grid hängt.



kleiner Trick, für kurze Sync-Funktionen:

markieren

Delphi-Quellcode:

procedure TMyThread.Execute;
begin
  ...
  Synchronize(procedure
    begin
      FormUsed.Update;
    end);
  ...

Zitat von mcinternet:

Zitat von mjustin:

* aus einem Thread darf man nicht auf VCL Objekte des Hauptthreads zugreifen - ausser wenn der Zugriff mit synchronize geschützt wird

Genau dafür ist der Thread ja da - er soll an dieser Stelle das Hängen verhindern.

Damit meinte ich speziell diese Aufrufe von FormUsed aus Execute:

markieren

Delphi-Quellcode:

procedure TMyThread.Execute;
begin
  ...
  FormUsed.OraSessCommon.Connect;
  ...
  FormUsed.qry_searchMA.Open;
...
end;

Das ist ein Zugriff auf Objekte, die gleichzeitig auch im Hauptthread angesprochen werden.

Zitat:

Damit meinte ich speziell diese Aufrufe von FormUsed aus Execute:

markieren

Delphi-Quellcode:

procedure TMyThread.Execute;
begin
  ...
  FormUsed.OraSessCommon.Connect;
  ...
  FormUsed.qry_searchMA.Open;
...
end;

wie kann ich das an dieser Stelle korrekt abbilden?

Gruss
MC

Zwei Möglichkeiten:
1. Du übergibst dem Thread als Parameter im Create (oder als Property) eine Oracle Connection
2. Du lässt den Thread im Execute eine eigene Connection aufbauen.