2014-09-14T06:51:26 rosti

In Teamarbeit ist es wichtig, dass Einer, der den Code liest, erkennen kann, was sich der Andere dabei gedacht hat. Wenn Rückgabewerte von Funktionen genutzt werden, ist das der Fall und kein Voodoo, da ist OPTISCH zu sehen, wo die Variable herkommt, wann sie einen Wert bekommt und das kann mit den Möglichkeiten einer IDE oder einer anderweitigen Suchfunktion zurückverfolgt werden ohne dass die Zeit wegläuft.

Dass Kollegen dann bei sowas mitdenken sollen. Optisch zu sehen, Oh weh!
Teamarbeit besteht nicht daran, verfreakten Code der Kollegen zu verstehen, sondern mittels guter Dokumentation und eines Styleguides zu programmieren.

2014-09-14T06:51:26 rosti

Code (perl): (dl )

my $config = require MyConfig;

drückt rein optisch in einer einzigen Zeile so ziemlich eindeutig aus, was da gemacht wird. Sicher geht das auch mit mehreren Zeilen genauso verständlich, aber wenn $config einmal im Scope liegt, wird wohl in beiden Fällen keiner auf die Idee kommen, die Config ein zweites Mal laden zu müssen.

Wenn mir jemand sowas zeigt oder hinterlässt: my $config = require MyConfig; dann denke ich eher dran denjenigen zu erwürgen, weil sowas PerlBadPractice entspricht und nicht sofort nachvollziehbar ist.
Denn ich denke, da hat jemand eine Funktion names require() geschrieben, die dann den Parameter MyConfig verwendet.

OK, dein Team ist bestimmt viel intelligenter als ich und erkennt sofort, was du meintest. ;)

Was du da machst mit require ist eher ein Proof of Concept als wirklich praktikabel, um sauber zu programmieren.
Last edited: 2014-09-15 11:49:04 +0200 (CEST)