Konstantenlänge

[angos]

Hi,

und ich gebe dir damit recht. Ich empfinde das persönlich auch als bescheiden lesbar, sobald die Konstante irgendwo im Code, zB einer if-Abfrage, benutzt wird. Lediglich bei der Deklaration ist das schön, weil absolut eindeutig.
Ich denke, dass ich schon Konstanten mit recht langen Namen vergebe, meine "Schmerzgrenze" liegt so bei ca 30Zeichen.

Großschreibung und Unterstriche nutze ich allerdings auch!

Gruß

[Blup]

Konstanten eines bestimmten Bereichs werden bei mir in jeweils einer Unit zusammengefasst.
Die Namen bischen kürzen und schon ist es lesbar.

Delphi-Quellcode:

unit BENUTZER_EINSTELLUNG_VISUALISIERUNG_PLANUNG;

interface

const
  VIRTUELLE_AUFTRAEGE_EINBEZIEHEN = 123;

[HeikoAdams]

Was für die Namen von Variablen gilt, sollte auch für Konstanten gelten: möglichst kurz und aussagekräftig

[Sir Rufo]

Ansonsten mit ein wenig englisch

Code:

USER_SET_VIS_PLAN_INCLUDEVIRTUALORDER

[Luckie]

Konstanten schreibe ich grundsätzlich mit Großbuchstaben, damit ich sie von Variablen unterscheiden kann. Und diese langen Bezeichnungen sind eher kontraproduktiv, da sie man nicht mit einem Blick erfassen kann. Und was sollen das für lange Zeilen werden, wenn sie im Code verwendet werden?

[nachti1505]

Ich halte es mal mit Martin Fowler, welcher meinte:

- Je größer der Scope einer Variablen / Konstanten / Funktion umso aussagekräftiger (und damit einhergehen auch umso länger) sollte der Name sein. -

[himitsu]

Eigntlich sollte es eher so sein:
aussagekräftig/verständlich und dennoch möglichst kurz und übersichtlich

- je länger etwas ist, umso länger braucht man für das verstehen/erkennen

- sind die Namen nicht aussagekräftig genug, dann weiß man nicht worum es geht
(und müßte eventuell erst irgendwo nachlesen, worum es geht, falls es überhaupt irgendwo erwähnt wurde)

Zitat von Luckie:

Und was sollen das für lange Zeilen werden, wenn sie im Code verwendet werden?

siehe mein letzer Post

[BUG]

Zitat von Teekeks:

Jedoch graust es mich schon, Konstanten zu nutzen, die länger als dieses hier sind ...

Nicht umsonst gibt es die Codevervollständigung.

Ich habe nicht den Eindruck, das

Delphi-Quellcode:

const
  BENUTZER_EINSTELLUNG_VISUALISIERUNG_PLANUNG_VIRTUELLEAUFTRAEGEMITEINBEZIEHEN = 123;

besonders sprechend ist. Das ist irgendwie der Namensraum mit in den Konstantennamen geschrieben.

Wie wäre es denn mit einer Unit BenutzerEinstellungVisualisierungPlanung, in der die Konstante VIRTUELLE_AUFTRAEGE_EINBEZIEHEN zu finden ist. In der Unit könnten dann alle Konstanten liegen, die im Moment mit BENUTZER_EINSTELLUNG_VISUALISIERUNG_PLANUNG_ beginnen.

Sollte die Konstante allerdings nur in einer Unit benötigt werden, die sich mit Visualisierung der Planung befasst, dann könnte man sich doch auf BENUTZER_VIRTUELLE_AUFTRAEGE_EINBEZIEHEN einigen.

@Himitsu:
Man muss ja nicht alles in eine Zeile schreiben:

Delphi-Quellcode:

MeineMachDiesesUndJenesProzedur(
    DerErsteParameterMitDenEinstellungen,
    BENUTZER_VIRTUELLE_AUFTRAEGE_EINBEZIEHEN,
    BENUTZER_VIRTUELLE_AUFTRAEGE_EINBEZIEHEN);

Außerdem wäre schon eine Funktion mit mehr als 2 Parametern suboptimal

[bernau]

Ich finde komplette Grossschreibung sehr unleserlich. Ich verwende Grosskleinschreibung ohne Unterstriche.

IchBinEineVariableOderEineKonstante

finde ich besser lesbar als

ICH_BIN_EINE_VARIABLE_ODER_EINE_KONSTANTE

Ist aber letztendlich Geschmacksache. Maximale Länge ist ca. 40 Zeichen.

Grade bei Zahlen, mit denen auch gerechnet wird, halte ich die Bezeichner kurz, weil damit ja auch gerechnet wird. Man stelle sich eine Formel vor, die 5 Variablen mit je einer Länge von je 70 Zeichen hat. Völlig unleserlich.

[himitsu]

Also im Prinzip ist alles über der halben "Bildschirmbreite" (standardmäßig so 80-120 Zeichen) unleserlich, denn man muß ja bedenken,
daß diese Konstantan auch irgendwo verwendet werden und wenn man nun 2 Konstantan an eine Prozedur übergibt, dann sollten die Einrückung,
Leerzeichen, Prozedurname UND diese Konstanten in eine Zeile passen.

Außerdem haben zu lange Konstantennamen, auch wenn sie "sprechender"/verständlicher und übersichtlicher zu scheinen mögen,
dennoch einen unübersichtlich Beigeschmack.
MeineMachDiesesUndJenesProzedur(DerErsteParameterMitDenEinstellungen, BENUTZER_EINSTELLUNG_VISUALISIERUNG_PLANUNG_VIRTUELLEAUFTRAEGEMITEINBEZIEHEN, BENUTZER_EINSTELLUNG_VISUALISIERUNG_STEUERUNG_VIRTUELLEAUFTRAEGEMITEINBEZIEHEN); .
Oder sieht hier jemand auf die Schnelle, was hier gemacht wird?
Man stelle sich alles in einer Zeile vor und versuche möglichst schnell den Unterschied zwischen den fast gleich
klingenden Konstanten zu finden, denn je mehr Zeichen, umso weniger fallen kleine Unterschiede auf.
MachDieses(Parameter, VIRT_BEN_PLAN, VIRT_BEN_STRG); .
Im Notfall kann man bei der Deklaration und/oder in der Dokumentation
eine ausführlichere Beschreibung zur Konstante angeben.

Ansonsten nehm ich persönlich für Typen, Prozeduren, Variablen und Konstanten CamelCase,
jenachdem mit einem passenden Präfix. (für Typen T und passend dazu das P und für Konstanten meist ein c)

Diese ALLES_GROß_MIT_UNTERSTRICH-Variablen nutze ich eigentlich nur bei APIs,
wo diese Konstanten irgendwo dokumentiert sind, um keine doppelten (verwirrenden) Bezeichner zu bekommen.
Denn hier *_VIRTUELLEAUFTRAEGEMITEINBEZIEHEN hab ich immer Probleme dieses
zu lesen und vorallem schnell zu verstehen, da es keine Übergänge gibt.