[REVIEW] HomeView

[kelvan]

Anmerkung: ich hab mal alles reviewed, auch wenn manches vermutlich genieriert ist (glaube aber nur ein kleiner Teil)

Review: HomeView
- label; label_1; label_2 haben keine Aussage
- Nicht-final Variablen sollten keinen underscore im namen haben
- lblSieSindEingeloggt ist nett aber ein bissl lang ^^
- e1.printStackTrace(); gehört auch hier noch gefixed
- Zwischenspeichern eine Zeile vor dem return kann durch direktes return ersetzt werden
- prependNews() wird nicht verwendet
- dispose() überschreibt parent methode und ruft nur diesselbe per super auf

[Mog]

Alles bis auf den printStackTrace ausgebessert. Dafür braucehn wir erst eine allgemeine Lösung, damit es sich lohnt das anzufassen.

[nudelsalat]

statt e.printStackTrace() sollte normal immer logger.error() oder logger.fatal() verwendet werden aber ich weiß jetzt leider nicht wie man den inso log4j wrapper dazu bringt die messages auf die richtige konsole auszugeben.
auserdem sollt das ganze dann auch noch in ein file geschrieben werden. wenn sich keiner die arbeit antun will rauszufinden wie man den insologger bändigt dann hätt ich vorgeschlagen, wir haun den raus und verwenden gleich direkt log4j.

[kelvan]

dann könnten wir auch die nervigen debug msgs gezielt ausschalten