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

[Christian Seiler]

Hallo Alexander,

Der Artikel an sich gefällt mir gut, nur ein paar Anmerkungen zu Deiner Beispielimplementation ganz am Ende:

- 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.
   (Mit ein paar wenigen Ausnahmen sind alle Beispiele bei SELFHTML aktuell
   Public Domain - ich habe nichts dagegen, dass Du die LGPL verwendest,
   allerdings sollte das den Leuten auch klar gemacht werden)
 - Es wäre schön, wenn Du ein paar eindeutige Bezeichnerkonventionen
   verwenden würdest, ich zähle nämlich drei verschiedene Arten von
   Methodenbezeichnern:
      * addHam <- 1. Wort klein, 2. Wort groß
      * addwords <- Beide Wörter klein, nicht getrennt
      * get_ranks <- Beide Wörter klein, durch Unterstrich getrennt
 - 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.
 - 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)
 - 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).
 - »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.

Viele Grüße,
Christian