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

[Christian Seiler]

Hallo Alexander,

• 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.

Naja, wie umständlich etwas zu tippen ist ist ja wohl kaum ein Argument bezüglich der Übersichtlichkeit des Codes. Und ich bin normalerweise absolut nicht der Typ, der sich in Coding-Stil-Diskussionen einmischt (ob nun 2 oder 4 oder 8 Leerzeichen ist mir eigentlich egal) - allerdings empfinde ich Dein "nur 1 Leerzeichen" als derart unübersichtlich, dass es mich _deutlich_ mehr Konzentration kostet, bei Deinem Code die Übersicht zu behalten, als bei so gut wie jedem anderen Code, den ich mal lesen musste.

• »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.

Tja, damit hast Du die Hälfte meiner Kritik umgesetzt. ;-) Die andere Hälfte war die: Deine Klasse mysql_connector (früher mysql) ist eine OO-Abkapselung der MySQL-Datenbankfunktionen. Wenn Du jetzt eine Klasse davon ableitest, dann sollte im Klassennamen der abgeleiteten Klasse auch die Information auftauchen, dass es sich um eine MySQL-spezifische Klasse handelt - denn Dein einfaches Datenmodell lässt sich in so gut wie jeder relationalen DB umsetzen, die es gibt. Ich käme nämlich vom Namen v_spam_filter_database (oder vorher dbwrapper) *nie* auf die Idee, dass es sich um eine MySQL-spezifische Klasse handelt. Wie wäre es mit v_spam_filter_mysql_storage oder etwas ähnlichem (vielleicht fällt Dir ja etwas kürzeres ein)?

Viele Grüße,
Christian