Vcl Annsprecbar bei TDirectory.copy

[ScharfeMietze]

Ich schreibe ein Tool das einen Ordner Überwacht und nach Inhaltsprüfung die Unterordner in verschiedene Verzeichnisse kopiert.
Das Kopieren funktioniert auch alles problemlos.
Die Ordner sind sehr groß dabei kann ein Kopiervorgang auch gern mal 20-30 min dauern.

Das Problem was ich habe ist, das die Dauerschleife während des kopierens nicht Ansprechbar ist. Normal stört das nicht, wenn man einmalig in Sekunden etwas kopiert.
Da das Programm in einer Schleife läuft, komme ich nicht dazu auf den "Stopp" Button zu drücken...
Die Dateien an sich darf ich NICHT verändern (also in rar wandeln z.B.) Ich muss auch am ende den Quell- und Zielorder auf Gleichheit überprüfen.
Hat jemand eine Idee wie ich den Stop drücken/ansprechbar machen kann und am Ende des aktuellen Ordners das Programm stoppt?

Gruß
SM ;)

[Bernhard Geyer]

Einfache (schnellhack) Lösung wäre es zwischendurch ein Application.Processmessage einzufügen um die normale Windows-Messagequeue bedienen zu können.
Besser wäre es diese Aktion in einen Thread zu verlagern.

[Sir Rufo]

Oder man nimmt den BackgroundWorker als Bindeglied zwischen Thread und GUI.

Beispiele findet man hier
http://www.delphipraxis.net/185335-b...ab-xe2%5D.html

[ScharfeMietze]

Das habe ich drin, auch ein Sleep von 1ner Sekunde am Anfang der "repeat" Schleife. Keine Wirkung
wird Grau und keine Rückmeldung... auch nach vollendetem Schleifendurchlauf.

Eigentlich wollte ich vermeiden einen Thread zu nutzen, da "nichts" zu tun ist bis fertig kopiert wurde............
Wenn es einen Weg ohne Threads gibt ... hör ich den gern.

[Daniel]

Zitat:

Zitat von ScharfeMietze (Beitrag 1315673)

Eigentlich wollte ich vermeiden einen Thread zu nutzen, da "nichts" zu tun ist bis fertig kopiert wurde.....

Doch. Du willst die Anwendung "am Leben" halten und das ist aus Sicht des Systems sehr wohl "etwas zu tun". ;-)
Sorry, is' so ...

[hathor]

Man kann auch COPY von CMD.exe nutzen - arbeitet auch unsichtbar im Hintergrund - das Hauptprogramm bleibt bedienbar.
Kann auch zig Terabytes kopieren!

Stark gekürzter Auszug:

Delphi-Quellcode:

var
 FullOutputFileName, SummaryLine, CommandProcLine, FILETOCOPY: string;

SummaryLine:= '/Y /B "';
SummaryLine:=SummaryLine + FILETOCOPY +'"  ';
CommandProcLine:='copy '+ SummaryLine +' "'+ FullOutputFileName+'"';
ExecAndWait(PChar('cmd.exe /C '+ CommandProcLine));

[ScharfeMietze]

Jup daher die "" .. ...

Nun mal die Frage ob man es so tun kann, oder ob das besser geht?

Delphi-Quellcode:

unit UnitCopyThread;

interface

uses
  System.Classes,System.IOUtils;

type
  TCopyThread = class(TThread)
  private
  FSource, FDestination: PChar;
  FFinished: Boolean;
    { Private-Deklarationen }
  public
  constructor Create(aSource,aDestination: PChar);
  function isFinished: Boolean;
    { Public-Deklarationen }
  protected
    procedure Execute; override;
  end;

implementation


constructor TCopyThread.Create(aSource, aDestination: PChar);
begin
inherited Create (false) ;
  FreeOnTerminate:= True;
  FSource:= aSource;
  FDestination:= aDestination;

end;


procedure TCopyThread.Execute;
begin
FFinished:= False;
TDirectory.copy(FSource,FDestination);
FFinished:= True;
  { Thread-Code hier einfügen }
end;

function TCopyThread.isFinished: Boolean;
begin
  if FFinished = True then  Result:= True else Result:= False
end;

end.

und Ausführung

Delphi-Quellcode:

procedure MDir.btnTestClick(Sender: TObject);
var
  LCopyThread: TCopyThread;
begin
 LCopyThread:= TCopyThread.Create('D:\1','D:\2');
  try
  while LCopyThread.isFinished = false do
    begin
      Application.ProcessMessages;
      Sleep(100);
    end;
  finally
    LCopyThread.Terminate;
  end;
end

Das andere schau ich mir gleich mal an von Hathor :)

[Aviator]

Da ein Thread, welcher die Execute Methode vollständig durchlaufen hat, sich selbst "terminiert" ist es schwachsinnig, auf die Beendigung des Threads durch eine Boolean Variable zu warten und dann Terminate aufzurufen. Wenn du das unbedingt machen willst, dann prüfe auf Thread.Terminated. Viel eher solltest du an dieser Stelle LCopyThread.Free aufrufen um den Speicher wieder freizugeben. Oder noch einfacher wäre es, wenn du FreeOnTerminate des Threads auf true setzt. Aber dieses warten darauf, dass eine selbst erstellte public Variable irgendwann true wird, ist nicht der optimale Weg.

Und noch ein Hinweis: Man überprüft Boolean Variablen nicht mit

Delphi-Quellcode:

if Boolean = True then

oder

Delphi-Quellcode:

if Boolean = False then

sondern mit

Delphi-Quellcode:

if Boolean

oder

Delphi-Quellcode:

if not Boolean

.

[ScharfeMietze]

Ist bereits im construcor gesetzt (free on terminate);

Wenn ich LCopyThread.CheckTerminated abfrage, friert mir Windows ein und verabschiedet sich später mit einem Ungültigen Fensterhandle :



Bei der CMD Variante ... kann man dort auch das Ergebnis auslesen?
Wenn ja wie?

[Aviator]

Zitat:

Zitat von ScharfeMietze (Beitrag 1315690)

Wenn ich LCopyThread.CheckTerminated abfrage, friert mir Windows ein und verabschiedet sich später mit einem Ungültigen Fensterhandle :

Ist ja auch klar warum. Im Constructor setzt du FreeOnTerminate auf True (hatte ich übersehen). Was passiert, wenn der Thread fertig abgearbeitet wurde und Terminated auf true gesetzt wird? Genau, der Thread wird freigegeben. Was passiert dann. Du greifst auf einen nicht initialisierten Speicherbereich zu. Infolgedessen schmiert dein Programm natürlich auch ab. Eigentlich sollte (wenn ich jetzt keinen Knoten im Kopf habe) zwar eine AV kommen, aber es kann ja auch anders sein.