9. Coding Contest

Binary Test:
+ beim laden einer neuen Datei wird gefragt, ob das aktuelle gespeichert werden soll
- abbrechen button beim speichern funktioniert nicht
+/- Kategorie zwar verwaltbar, aber kein dropdown in der liste selbst
- Umlaute funktionieren beim Excel (CSV) export nicht
- Excel (CSV): Spalten werden nicht erkannt, alles wird in eine Zeile geschrieben (, anstatt ; als Seperator benutzt)
+ XML Export scheint zu klappen (kenn mich mit dem format nicht aus)
+ Tabelle docked an der GUI (Größenvariabel)
+ Anzahl wird auf gültige Zahl geprüft (beim erstellen eines neuen Bestands)
- Kategorie wird nicht auf Duplikat geprüft (ein Dialog für den Namen einer neuen Kategorie wäre auch wünschenswert)
- Anzahl Update in der Tabelle wird nicht auf gültige Zahl überprüft
- negative anzahl möglich
+ Suchfunktionen
+ Sortierfunktion für jede Spalte

Source:
+ Gut in Regions sortiert
- Keine Kommentare
- Doppelter Try-catch Block um cBestandManager.LoadBinary, besser wäre ein try-finally-catch block (finally zum reader closen)
- finally Block zum closen des Streams fehlt
- Parameter werden nicht auf gültigkeit überprüft (null Verweis, leerer String)
- cItem Klasse wird als struct benutzt, sollte weiter ausgebaut werden, getter, setter fehlen, public attribute nicht gut
kein Konstruktor, kein toString
- OutputStreams werden nicht geflushed vor dem schließen
- zum löschen wäre es sauberer den index im Manager mit dem in der Tabelle synchron zu halten, dadurch muss nicht die ganze
Liste durchsucht werden
- Konstanten bzw. enums verwenden für switch case
- zu wenig OOP

+ Die XML-Ausgabe is ok
+ Objekt-Serialisierung verwendet!
+ AutoSave alle 10 Sekunden
+ Programm ist übersichtlich gehalten.

? Kategorie-Liste hat keinen Bezug zum restlichen Programm
? Markiert man mehrere Einträge und drückt löschen, wird nur der letze Eintrag gelöscht.

- Beim Laden einer Datei wird die Kategorie-Liste nicht geleert. Die Kategorien werden einfach eingefügt

Binary:
+ Fehlerbehandlung funktioniert augenscheinlich
- Keine Dialoge zum Speichern, Laden
- Dateien können nicht überschrieben werden
- csv export erste spalte leer
- doc format wird nicht richtig eingehalten

Source:
- beim stringvergleich auf leerstring immer trim() vorher aufrufen
- Ungültiger Name Fehlermeldung wird jedesmal neu definiert, eine Methode die auf gültigen String prüft und dann eine Exception oder
passende Fehlermeldung ausgibt wäre besser, dasselbe gilt für die anderen fehlermeldungen
- keine Kommentare
- name besser im Konstruktor überprüfen und dann passende Exception werfen
- anstatt einer ausgabe methode überschreibt man besser die tostring methode
- sauberer wäre es in der Verwaltungsklasse nur exceptions zu werfen und die passenden Fehlermeldungen in
der GUI Klasse auszugeben
- Methoden zum laden/speichern besser static machen und anstatt success zu setzten direkt true oder false zurückgeben
- Parameter der setter werden nicht überprüft
- finally block zum flushen/closen fehlt
- Kein Dialog um den Speicherpfad auszuwählen alles wird auf den Desktop geklatscht
- Datei kann nicht überschrieben werden!
- Datei wird nicht auf File, beschreibarkeit geprüft
- für jeden stream wird ein objekt erzeugt, new BlaReader/Writer dem Konstruktor des überliegenden Streams übergeben, dann müssen auch kein 100 Streams
geschlossen werden
- Laden kompliziert gelöst, file wird nicht auf file, lesbarkeit geprüft
- Beim Exportieren kommt immer das gleiche raus

Binary:
+ schöne GUI, mit schönen Icons, kompakt und überschaubar
+ Wenn auch wenig funktionen scheint es zu funktionieren

Source:
- keine Kommentare
naja durch autoit halt sehr unsauber

Binary:
- Informationen Message Box hat ein Error Logo?!
- speichernder Name?! Kein Dialog zum pfad auswählen, keine überprüfung des pfades
- speichernder name wird anscheinend auch zum laden benutzt
- Keine Erklärung wie das Programm funktionieren soll (auch nicht selbsterklärend)
- x,y sollen anscheinend den index in der tabelle angeben, keine überprüfung auf korrekte zahl (weder numerisch noch vom index her)
- Nach Klick auf Eingabe, muss bei Zelle löschen auf nein geklickt werden, ich versteh den gesamten dialog nicht

Source:
- kein OOP, alles in einer GUI Klasse
+ try finally prinzip eingehalten
- tabelle löschen mit konstanten column, row counts?!
- TForm1.Edit4Focus?

+ Speichern und Laden geht

? Dialog nach klick auf Eingabe nervt^^

- StringGrid wird resettet sobald man Reihen oder Zeilen hinzufügt
- Teilweise Dirty Code (clear_grid)
- Unklare Objekt-Bezeichnungen