Hier der Code: https://github.com/toxinman/stuff/blob/ ... notizen.py
Danke
Notizverwaltung
@Xfd7887a: wie schon im anderen Thread geschrieben, gehören das Laden und Speichern Deines Notizbuchs nicht in die Controller-Klasse. Als "Datenbank" eignet sich SQLite auch besser als Pickle. Pickeldateien sollten immer im Binärmodus geöffnet werden. Ein "exit" hat außerhalb eines, naja, allem, nichts zu suchen, das macht den Programmablauf nur sehr unübersichtlich. Intern sollte ein Datum immer als Datums-Objekt und nicht als String vorliegen, das macht das Weiterarbeiten viel einfacher. Das Menü hat in main nichts verloren, das ist eindeutig Aufgabe des Controllers. Statt einer Liste für jeden Menüeintrag ist ein (Named-)Tuple die bessere Wahl. input sollte man nicht verwenden.
Du solltest noch an Deiner Fehlerbehandlung arbeiten. NotizbuchDB ist keine Datenbank, sondern eine Aufbewahrungsliste mit Ausgabefunktion: Laden und Speichern findet außerhalb statt, notiz_anlegen bekommt schon ein NotizObjekt, der Aufrufer muss also schon den Datentyp kennen, notiz_bearbeiten versteckt den Fehler vor dem Aufrufer und notiz_suchen sucht nicht, sondern gibt aus. Damit ist die ganze Logik der Klasse gar nicht in der Klasse selbst und man kann sie nicht wiederverwenden, weil man dazzu zu viel internes wissen muß und falls Du mal eine GUI schreiben willst, oder ein Web-Interface, Dir die Bildschirmausgabe überhaupt nicht weiterhilft.
Du solltest noch an Deiner Fehlerbehandlung arbeiten. NotizbuchDB ist keine Datenbank, sondern eine Aufbewahrungsliste mit Ausgabefunktion: Laden und Speichern findet außerhalb statt, notiz_anlegen bekommt schon ein NotizObjekt, der Aufrufer muss also schon den Datentyp kennen, notiz_bearbeiten versteckt den Fehler vor dem Aufrufer und notiz_suchen sucht nicht, sondern gibt aus. Damit ist die ganze Logik der Klasse gar nicht in der Klasse selbst und man kann sie nicht wiederverwenden, weil man dazzu zu viel internes wissen muß und falls Du mal eine GUI schreiben willst, oder ein Web-Interface, Dir die Bildschirmausgabe überhaupt nicht weiterhilft.
Erstmal danke für die Antwort. Einige Fragen:
Ich denke mal, du meinst, dass sie einzelne Funktionen sind, die dann in main aufgerufen werden, oder?Laden und Speichern Deines Notizbuchs nicht in die Controller-Klasse
Das dachte ich mir schon, aber ich kann kein SQL, deshalb erstmal diese Notlösung. Edit: Werde ich mir aber noch anschauen.Als "Datenbank" eignet sich SQLite auch besser als Pickle.
Ja, aber wie kann ich sonst die äußere Schleife verlassen?Ein "exit" hat außerhalb eines, naja, allem, nichts zu suchen, das macht den Programmablauf nur sehr unübersichtlich.
Die Schleife dazu auch? Muss das in die __init__ Methode?Das Menü hat in main nichts verloren, das ist eindeutig Aufgabe des Controllers
Was ich versuche deutlich zu machen, ist, dass andere Funktionen und Klassen keine Kenntnis vom internen Aufbau einer Klasse haben dürfen. Das Interface sollte möglichst generisch und leicht verständlich sein. Im Idealfall solltest Du Deine NotebookDB-Klasse durch eine Klasse ersetzen können, die auf einer Datenbank aufbaut, ohne irgendeine andere Stelle im Code ändern zu müssen.