Seltsame Pfadbearbeitung

[Uwe Raabe]

Zitat von Delbor:

Delphi-Quellcode:

procedure TOpenFileFrame.TVFilesExplorerEdited(Sender: TObject; Node: TTreeNode;
  var S: string);
var
  NameRec: PNameRec;
  LBitMap: TBitmap;
  LiconIndex :Integer;
  LIsdirectory,
  LIsDB: Boolean;
  ImageList : TDataObjectlist<TBitmap>;
  LCaption,
  LBmpFile,
  LRealName,
  LDeskIniPath: string;
begin

Bei mir gibt es die Prämisse: Jede Variable bekommt ihren eigenen Typ!

Delphi-Quellcode:

procedure TOpenFileFrame.TVFilesExplorerEdited(Sender: TObject; Node: TTreeNode;
  var S: string);
var
  NameRec: PNameRec;
  LBitMap: TBitmap;
  LiconIndex :Integer;
  LIsdirectory: Boolean;
  LIsDB: Boolean;
  ImageList : TDataObjectlist<TBitmap>;
  LCaption: string;
  LBmpFile: string
  LRealName: string;
  LDeskIniPath: string;
begin

Und wenn einem das zu viele lokale Variablen sind, sollte man über einen Split in mehrere Methoden nachdenken.

Abgesehen davon müssen die Variablendeklarationen auch gar nicht sichtbar sein. Ein Mouse-Hover zeigt schließlich auch den Typ an.

[Delbor]

Hi Uwe Raabe

Zitat von Uwe Raabe:

Und wenn einem das zu viele lokale Variablen sind, sollte man über einen Split in mehrere Methoden nachdenken.

Methoden auszulagern, macht meines Erachtens eh Sinn. Im vorliegenden Fall bin ich allerdings in der Entwicklung - einen guten Teil der lokalen Variablen brauche ich gar nicht. Und gerade in der Pfadbearbeitung bietet es sich an, auch mit anderen Formularen zu arbeiten. So enthält meine Mainform ein Property ProjectPath, bzw ein Property AppDataPath.

Zitat von Uwe Raabe:

Abgesehen davon müssen die Variablendeklarationen auch gar nicht sichtbar sein. Ein Mouse-Hover zeigt schließlich auch den Typ an.

Das hat mich heute morgen erst mal erstmal erstaunt, bis mir klar wurde: man kann sie auch in eine Region packen.

Gruss
Delbor