Hallo Daniel, das mit den 'schweren Fehlern' darf man nicht so wörtlich nehmen. Ich hatte vor langer Zeit schon mal einen Code Optimizer ausprobiert, und was man auf keinen Fall machen darf ist es den automatisiert an den Code ran zu lassen. Es ist aber tatsächlich so das man auf die bemängelten Stellen mal gucken sollte, und dann mit dem Hintergrund das man den Code an der Stelle auch versteht die vorgeschlagenen Änderungen durchführt. Insbesondere für die nicht-null-zurückgeben - Warnings gilt es: bloss nicht zu schnell was ändern. Zurückgegebene Null-Werte werden in YaCy oft als 'sanfte Exception' interpretiert, d.h. ein Methoden-Caller testet auf Null und entscheidet dann ggf. das die Sache schief gelaufen ist. Das ist leider in YaCy nicht ganz konsistent. De kannst ja die Analyse zum Anlass nehmen dir einzelne Codestellen mal genau anzugucken und dann ggf. was machen. Ich installiere auch mal den tptp und schaue mal wie das im Detail aussieht.
Viele Grüße, Michael Am 04.03.2008 um 13:20 schrieb Daniel: > Hallo Liste, > > ich habe in Eclipse mal eine statische Code Analyse angestoßen > (mittels > TPTP: http://www.eclipse.org/tptp/ ). Dabei wird geprüft, ob die > "Best > Practices" eingehalten werden. Dabei sind einige schwerwiegende Fehler > zu Tage getreten. Die "Kleinigkeiten" habe ich deshalb extra > aufgeführt. > > Das Bild zeigt die schweren Fehlermeldungen. Im XML sind dann auch die > Stellen im Code angegeben. Mittels dem XLS kann man sich die XML- > Dateien > formatiert im Browser betrachten, allerdings ist die "Full" mit 25.000 > Einträgen recht lang ;) > > Ich habe mir das bisher nicht detailliert angeschaut, aber bin der > Meinung, dass die schweren Fehler oft wichtige Stellen betreffen. Ich > würde mich bereit erklären Kleinigkeiten zu fixen, falls nicht jemand > total gegen ordentlichen Code ist :) > > Wie seht ihr das? Spielkram oder wichtig? > > Gruß > Daniel > < > CodeReview > .tar > .gz > ><YaCy_CodeReview.png>_______________________________________________ > YaCy-svn mailing list > [email protected] > https://lists.berlios.de/mailman/listinfo/yacy-svn _______________________________________________ YaCy-svn mailing list [email protected] https://lists.berlios.de/mailman/listinfo/yacy-svn
