Seltsame Pfadbearbeitung

[Uwe Raabe]

Zitat:

Zitat von Delbor (Beitrag 1452241)

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:

Zitat von Uwe Raabe (Beitrag 1452243)

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:

Zitat von Uwe Raabe (Beitrag 1452243)

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