Einzelnen Beitrag anzeigen

Benutzerbild von mirage228
mirage228

Registriert seit: 23. Mär 2003
Ort: Münster
3.750 Beiträge
 
Delphi 2010 Professional
 
#5

Re: (Indy) IdFTP: Prüfen ob ein Verzeichnis existiert

  Alt 30. Jan 2010, 07:04
Ich sehe in Deinem Code zudem drei potentielle Probleme
  • Die IdFTP Komponenten wird nicht auf <> nil geprüft (Zugriffverletzung beim ersten Zugriff)
  • Der String wird nicht auf Länge > 0 geprüft (Zugriffverletzung beim Abgreifen des letzten Zeichen). Siehe zusätzlich Delphi-Referenz durchsuchenAnsiLastChar
  • Um die erstellte StringListe ist kein Ressourcenschutzblock (try/finally) (Potentielles Speicherleck)
Weiterhin sind die Variablen TempDir1 bis 3 nicht gerade eindeutige Bezeichner...

Und noch was: Für neuere Delphi Versionen wäre so etwas eventuelle eine schöne Gelegenheit einen Delphi-Referenz durchsuchenclass helper zu verwenden


Viele Grüße
David F.

May the source be with you, stranger.
PHP Inspection Unit (Delphi-Unit zum Analysieren von PHP Code)
  Mit Zitat antworten Zitat