@Omm: Die Namensschreibweisen entsprechen nicht dem
Style Guide for Python Code.
Was bedeutet das CL in `CL_Logfilemanager`? Man sollte bei Namen nicht raten müssen, denn Namen sind ja dazu da dem Leser zu vermitteln um was es sich bei dem Wert von der Bedeutung her handelt. Den Namenszusatz `manager` würde ich auch dringend überdenken, denn das ist meistens überflüssig oder weil einem nichts besseres eingefallen ist und man da noch einmal drüber nachdenken sollte. Jede Klasse verwaltet irgendeinen Zustand, deshalb könnte man `manager` auch bei jedem Klassennamen dranpappen, es enthält aber eigentlich keine Information die für den Leser wichtig wäre.
Ich bin mir ziemlich das `returnLastEntry()` eigentlich `getLastEntry()` (bzw. `get_last_entry()`) heissen sollte.
Wenn man etwas an einen Namen bindet und den Namen dann nur in der nächsten Zeile für ein ``return`` verwendet, kann man sich den Namen sparen.
`update_infobox()` aktualisiert gar nichts, das ist also auch ein falscher, irreführender Name.
Ich hätte es so geschrieben:
Code: Alles auswählen
def get_info(self):
entry = self.cl_logfile_manager().get_last_entry()
return '{} {} {}\n{}'.format(
*map(entry.get, ['Weekday', 'DateStr', 'DateStr', 'ProjectName'])
)