Hallo,
Zitat von
gandime:
... was kann ich an dem Code verbessern ...
der Event Handler für Button.OnClick sollte nur Code beinhalten, der im engen Zusammenhang mit der Benutzerschnittstelle steht. In deinem Fall lässt sich das Kopieren einer Datei auch in einer einzigen Zeile erledigen: CopyFile(). Stehst du aber auf Handarbeit, dann würde ich dir die Stream-Methode CopyFrom() empfehlen.
An deinem Code fällt mir spontan folgendes auf:
(1) Der Aufruf von Read() liefert die tatsächlich gelesene Zeichenanzahl zurück und du verwirfst diese wichtige Information.
(2) Die Iteration über Buffer geht auf jeden Fall um einen Schritt zu weit.
(3) Die Endgröße für den String ist bekannt und du erweiterst den String jewils um ein Zeichen, anstelle die Engröße mit SetLength() festzulegen.
(4) Statt pchar(@buffer[i]) kannst du Chr(buffer[i]) schreiben oder du deklarierst deinen Buffer gleich mit dem richtigen Basistyp Char.
(5) Du hast dir die Dokumentation der Stream-Komponente bestimmt noch nicht genau angesehen.
Grüße vom marabu