Brauche Bewertung + Tipps für mein kleines Projekt

[GambaJo]

Ich wollte mich wieder in OOP einarbeiten (habe das schon ewig nicht mehr gemacht) und habe vor einiger Zeit mit Java angefangen (leider immer wieder schleifen lassen) und habe auch schon eine kleine Desktop-Anwendung erstellt. Sie ist bei weitem noch nicht fertig (lauffähig schon), aber bevor ich viel falsch mache, dachte ich mir, es könnte sich mal der eine oder andere Profi kurz anschauen, ob das zumindest schon mal in die richtige Richtung geht, und mir vielleicht Tipps und Ratschläge geben kann.

Mir geht es vor allem um „sauberes“ Java (Konventionen, Lesbarkeit, …) und OOP (Vererbung, etc.), Performance (nicht zu viele Objekte, etc.), MVC, der richtige Umgang mit Swing (vor allem JTable und TableModel) und Datenbanken.

Es ist eine kleine Rezeptverwaltung, die später sowohl unter Windows wie auch Linux genutzt werden soll. Erstellt wurde die Anwendung mit Netbeans, als Datenbank wird db4o genutzt.
Bitte Projekt hier runterladen: Klick mich
Es ist ein gepacktes Netbeans-Projekt. Im Ordner /dist findet sich die ausführbare jar-Datei. Die Datenbank wird beim ersten Starten der Applikation im Benutzerverzeichnis angelegt.

Es würde mich freuen, wenn sich der eine oder der andere sich die Mühe macht, und mal kurz drüber schaut und mich in die richtige Richtung „schubst“.

 

[maki]

Wo hast du denn deine Unittests?

Standard Kommentare (// methods, // propertys) sind bääh.

 

[till123]

Hallo,

ich würde mcih mal um deine warnings kümmern
Deine Fehlermeldung beim Absichern (wenn Rezept fehlt) beinhaltet ein Rechtschreibfehler.
...

 

[Landei]

Habe leider keine Zeit, das durchzusehen, aber hast du schon mal FindBugs bemüht? Das findet eine Menge Schwachfug und meldet auch relative wenige "Falsche Fehler" (false positives)

http://findbugs.sourceforge.net/

 

[GambaJo]

Danke schon mal für eure Beiträge.
Es geht mir nicht um Fehlerfindung im sinne von Bugs. Ich weiß ungefähr, wo sie sind, und ungefähr, was ich noch machen muss.
Es geht mir eher darum den Code unter den oben genannten Gesichtspunkten zu betrachten und dazu auch Tipps zu bekommen. Nur weil eine Applikation lauffähig ist, heißt nicht, dass sie intern auch gut ist. Ich hab noch wenig Erfahrung mit OOP allgemein und speziell mit Java. Meine Sorge ist, dass man als Java-/OOP-Programmierer einige Punkte vielleicht anders löst (weil das der Standard oder weil es einfach besser ist).

Vielleicht formuliere ich konkrete Fragen:

1. Habe ich OOP sinnvoll eingesetzt (Objekte richtig genutzt, gekapselt, etc.)?
2. Sind meine Methoden sinnvoll konzipiert und befinden sie sich an den richtigen Stellen (könnte mir vorstellen, dass einige davon vielleicht in eine Extra-Klasse gehören würden)? Sind es nicht zu viele für den Zweck?
3. Habe ich nicht zu viele Objekte erzeugt, was den Speicher voll laufen lassen würde (z.B. beim Füllen der JTable über TableModel und Vectoren)?
4. Beim konzipieren einer Applikation nach MVC bin ich noch sehr unsicher. Ich habe versucht eine Trennung zwischen Daten, Geschäftslogik und Darstellung hinzukriegen, aber an manchen Stellen bin ich mir einfach nicht sicher.
5. Ich habe mich entschieden anfangs Swing zu nutzen, weil ich damit am wenigsten Portierungsschwierigkeiten habe (nutze Win32 aber Linux64). Sind die einzelnen Komponenten sinnvoll gewählt und eingesetzt? Fülle ich die JTable richtig (Vectoren)?
6. Habe mich für db4o als Datenbank entschieden, weil ich das Prinzip dahinter sehr interessant finde. An sich klappt der Zugriff auf die Datenbank ganz gut. Ich bin mir nur halt nicht sicher, ob man das Handling so machen kann oder besser gesagt soll (siehe auch der Punkt mit MVC).