Fragen zu DirectoryExists

[Glados]

Ich habe eben bei Google diese DirectoryExists-Alternative gefunden (https://stackoverflow.com/questions/...-xe2-correctly)

Delphi-Quellcode:

 function DirExistsA(const aDirectory: string): Boolean;
 var
  Code: Integer;
 begin
  Code := GetFileAttributesW(PChar(aDirectory));
  Result := (Code <> -1) and (FILE_ATTRIBUTE_DIRECTORY and Code <> 0);
 end;

Sie ist doppelt so schnell wie das Delphi eigene DirectoryExists.
Ich habe keine Einsicht in die Quelltexte von Delphi. Aber ist DirExistsA trotzdem zu gebrauchen? Denn es gibt kein FollowLink als Parameter und außerdem sind das nur zwei mickrige Zeilen.

[Zacherl]

Mal ein genereller Hinweis, weil ich von dir viele Threads dieser Art sehe. Wenn die Delphi RTL eine Funktion für diesen Zweck schon bereitstellt, dann würde ich sie auf jeden Fall verwenden. Es macht nur in äußerst wenigen Situtationen Sinn auf Alternativen umzusteigen oder das Rad neu zu erfinden. Grade Operationen wie

Delphi-Quellcode:

DirectoryExists

wirst du ja kaum so frequent ausführen, dass du tatsächlich Performance-Vorteile daraus ziehen könntest.

Das

Delphi-Quellcode:

DirectoryExists

aus der RTL ruft btw. exakt die gleiche API (GetFileAttributesW) auf. Sollte also keinesfalls langsamer sein als der Code von Stackoverflow.

[Glados]

Wenn ich beide Funktionen 2 Millionen mal aufrufe und die Zeit messe, ist die Delphifunktion mehr als doppelt so langsam.

[Zacherl]

Zitat:

Zitat von Glados (Beitrag 1383882)

Wenn ich beide Funktionen 2 Millionen mal aufrufe und die Zeit messe, ist die Delphifunktion mehr als doppelt so langsam.

Am Ende macht Delphi noch ein GetLastError (und das auch nur, wenn das gewünschte Verzeichnis nicht gefunden wurde).

Aber wie gesagt. Ich bezweifele, dass dieser Performanceunterschied im Produktivbetrieb großartig relevant sein wird.

[Glados]

Wofür soll das GetLastError denn gut sein? Der Rückgabewert ist doch Boolean.

[Zacherl]

Zitat:

Zitat von Glados (Beitrag 1383885)

Wofür soll das GetLastError denn gut sein? Der Rückgabewert ist doch Boolean.

Delphi-Quellcode:

Result := (LastError <> ERROR_FILE_NOT_FOUND) and
      (LastError <> ERROR_PATH_NOT_FOUND) and
      (LastError <> ERROR_BAD_PATHNAME) and
      (LastError <> ERROR_INVALID_NAME) and
      (LastError <> ERROR_BAD_NETPATH) and
      (LastError <> ERROR_NOT_READY) and
      (LastError <> ERROR_BAD_NET_NAME);

Wenn GetFileAttributesW fehlschlägt, dann kannst du halt nicht automatisch daraus schließen, dass es das Verzeichnis nicht gibt. Es kann genausogut sein, dass das Verzeichnis da ist, aber du keine Berechtigung dafür besitzt, etc. Daher musst du um sicherzugehen zwangsweise den Error Code prüfen.

[Glados]

Ich habe das dementsprechend umgestellt. So sollte das dann doch jetzt in Ordnung sein

Delphi-Quellcode:

 function DirExistsA(const aDirectory: string): Boolean;
 var
  Code: Integer;
  LastError: Cardinal;
 begin
  Code := GetFileAttributesW(PChar(aDirectory));

  if (Code <> -1) then
   Result := (FILE_ATTRIBUTE_DIRECTORY and Code <> 0)
  else
   begin
    LastError := GetLastError;
    Result := (LastError <> ERROR_FILE_NOT_FOUND) and (LastError <> ERROR_PATH_NOT_FOUND) and (LastError <> ERROR_BAD_PATHNAME) and (LastError <> ERROR_INVALID_NAME) and
     (LastError <> ERROR_BAD_NETPATH) and (LastError <> ERROR_NOT_READY) and (LastError <> ERROR_BAD_NET_NAME);
   end;
 end;

ich bleibe letzten Endes aber dennoch beim Delphi eigenen Code :P