Seltsame Pfadbearbeitung

[Delbor]

Hi jaenicke

Zitat von jaenicke:

Wir halten uns schlicht an den Styleguide, dann gibt es auch keine Unklarheiten, der Code ist sauber strukturiert und wir können jederzeit den Codeformatter ausführen (in den aktuellen Quelltexten meistens ohne Änderungen).

In einem Team mit Bestimmtheit unverzichtbar. 12 Mitarbeiter und 13 Formatierungen - schlicht fürchterlich...

Zitat von jaenicke:

Oh, ja, ich sollte nicht ganz so schnell lesen. Die Einrückung ist eigentlich für zu lange Zeilen mit Parametern üblich, so dass ich die Klammerung glatt übersehen habe.

Lokale Deklarationszeilen sollten meines Erachtens aber immer kürzer und deutlich von Kopfzeilen unterscheidbar sein. Hier hab ich das offenbar zuwenig beachtet.

Zitat von jaenicke:

Zitat von Delbor:

Das ist sozusagen mein 'Erbe' aus Delphi-1-Tagen, macht meines Erachtens aber durchaus Sinn, da dadurch oft etliche Zeilen eingespart werden können.

Das Einsparen von Zeilen ist aber genau nicht sinnvoll, unter anderem weil man so Diffs in der Versionsverwaltung viel schlechter lesen kann, von Merges ganz zu schweigen.

An die Versionsverwaltung hab ich dabei nicht gedacht. Auch wenn ich sowas noch korrigiere, wärs in Sachen 'Zeilenverschwendung' nicht ganz so tragisch:

Delphi-Quellcode:

[Prozedurkopf]
var
  desktopini: TIniFile; LDesktopIniParent : String; 
begin

Auf der andern Seite der Akzeptanzskala steht zB. solches:

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

Da geschieht noch rein gar nichts, trotzdem ist ein durchschnittlicher Bildschirm schon zu einem guten Teil 'zugemüllt'.

Gruss
Delbor

[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