Artikel-Review: Bayesscher Spam-Filter für Weblogs

[Alexander Brock]

Hallo Christian,

• Du solltest bereits im Artikel erwähnen, dass Deine Code unter der LGPL
     steht und ganz kurz anreißen, was das für den Autor von Software heißt. [...]

[x] done

• Es wäre schön, wenn Du ein paar eindeutige Bezeichnerkonventionen
     verwenden würdest, [...]

[x] done

• Ein Space zur Einrückung? Kann sein, dass das für Dich besser ist, aber
     ich kenne eigentlich sonst _niemanden_, der so wenig einrückt. Nimm Tabs
     (dann kann sich's jeder selbst einstellen) oder zumindest 2 Spaces. Ich
     persönlich empfinde 1 Space nämlich als sehr unübersichtlich und die
     großen Einrückungsdiskussionen drehen sich eigentlich immer um Tab vs.
     2 Spaces vs. 4 Spaces vs. 8 Spaces - 1 Space habe ich noch nie gesehen.

Ich mag weder Tabulatoren, die sind so umständlich zu tippen, noch mehr als ein Leerzeichen, das ist noch umständlicher.

• Mach öfters mal ein paar Leerzeilen, das lockert den Code auf.
• Ein paar mehr Inline-Kommentare wären nett (ich meine keine Docblocks,
     sondern an besonders wichtigen Stellen kommentare, was genau dort
     geschieht / warum das so passiert)

[x] done

• Es ist in meinen Augen sehr verwirrend, dass Du die eine Spalte in Deiner
     Beispielimplementierung ln genannt hast - denn das ist gleichzeitig
     eine MySQL-Funktion, insbs. lustig wird's dann bei sowas wie
        set ln = ln(... + ln)
     und ähnlichem. Benenne die Spalte sinnvollerweise um (z.B. ln_prob oder
     sowas).

[x] done

• »dbwrapper« ist in meinen Augen kein guter Name für eine von der Klasse
     »mysql« abgeleiteten Klasse - zumal der Name überhaupt nicht gut
     beschreibt, wofür die Klasse da ist.

Ich habe die Klassen jetzt in mysql_connector und v_spam_filter_database umbenannt.

Gruß
Alexander Brock