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.

Ich hatte diese Konstellation bisher so noch nicht, aber ich meine folgendes Problem macht das in der gewünschten Form unmöglich:

SQL Connections sind i.A. an den Thread gebunden, in dessen Kontext man sie erstellt. Heisst: Ein TThread kann/darf eine Connection, die auf dem Formular liegt, in keiner Weise nutzen. Das beinhaltet auch, dass im Thread keine Query benutzt werden kann/darf, die als Connection die auf dem Formular benutzt. Umgekehrt das selbe: Wird die Connection im Kontext des TThreads erzeugt, darf nichts auf dem Formular etwas anfassen, was diese Connection benutzt.
Folgerung:
Connection auf Formular -> Kein Query.Open im Thread.
Connection im Thread -> Keine Benutzung der Query als Dataset für das Grid

Man könnte jetzt an eine vermittelnde Pufferstruktur denken, die du im Thread voll machst, und dann dem Hauptthread am besten per Message mitteilst, dass es vollbracht ist. Der füllt dann ein Grid damit... und schon braucht dieser vermutlich ähnlich lange, genau dies zu tun. Darüber hinaus kann das dann kein DBGrid mehr sein, bzw. ist jeglicher Bezug zur DB dahin -> Editieren wird so einfach nichts.

Alles was man versuchen würde das eigentliche Problem zu kaschieren, wird mindestens extrem wackelig. Mach es lieber richtig, und filter wie schon irgendwo vorher gesagt wirklich nur die Daten, die du gerade zur Anzeige brauchst heraus. Dauert das auch zu lange, ist dein DB Design optimierungsbedürftig. Mit einem Thread wird das aber definitiv nur eine Zugriffsfehlerhölle ohne Gleichen.

Oder um deine Frage zu beantworten:

Zitat:

wie kann ich das an dieser Stelle korrekt abbilden?

Nein

hatten wir das Thema nicht schon vor ein paar Tagen?
http://www.delphipraxis.net/170838-o...ry-thread.html