Zitat:
Zitat von traderdoc
"Warum soll das Quatsch sein?"
Na, weil es eine überflüssige, ressourcenfressende und codeverlängernde Funktion ist und weil eine unnötige, ebenfalls speicherplatzintensivere datetime-Variable benötigt wird. Meine Version hat in der Start()-Funktion eine Zeile mehr, dafür insgesamt 8 Zeilen weniger!
Da Deine NewBar()-Funktion zwangsweise auch in der start-Funktion steht, ebenso wie meine Time-Abfrage, wird hier gar nichts überfrachtet. Und in der Regel wird nach unser beiden if-Abfragen eh in irgendwelche Funktionen zur weiteren Verarbeitung gesprungen. Oben gesagtes bleibt aber deshalb weiterhin bestehen.
Und die Lesbarkeit wird nicht immer unbedingt einfacher, indem ich ständig in irgendwelche Funktionen springe, zumal, wenn es Ein- oder Zweizeiler sind, die auch nur an einer! Stelle benutzt werden.
Ok, vor 20 Jahren hätte ich das Buch evtl. gut gebrauchen können.
traderdoc
|
Vor 20 Jahren hätte ich dir recht gegeben bzgl. des Speichers und Ressourcen, aber heutzutage ist das mit dem Speicher und Ressourcen kein Thema mehr, weil es einem quasi hinterher geschmissen wird und auch kein Kostenfaktor mehr ist. Zum Glück sind diese Zeiten vorbei.
Es kommt mit den Methoden immer darauf an. Wenn es häufig wiederverwendet wird, was im Falle meiner Methode NewBar vielleicht nicht der Fall ist, ist das ein sauberer Ansatz. Die Methode ist gut benamst, und sagt genau aus was es macht.
Ansonsten sind beide Ansätze möglich, und bei der Diskussion sollte es auch nicht darum gehen, wer hier recht hat, sondern nur eine Art Hilfestellung auf Fragen der Leute.
Carpe Diem,
Hoschie