Der Autor freut sich sicherlich über eine Mail mit konstruktiver Kritik, mit deren Hilfe er das Skript verbessern kann, um noch mehr Leute glücklicher zu machen.

moritz+2008-05-15 21:37:00--

Also csvdiff ist mir schon noch den ersten paar Codezeilen suspekt:

Ok, bei deiner Code-Inspection durchgefallen, aber wie sieht's denn mit der Funktion aus? Ich hab's mal mit 2 Dateien a 300MB versucht und war zufrieden.
Was denkst du denn wie's in anderem Sourcecode aussieht? Im Linux Sourcecode steht tausende male "fuck" wenn ich nicht irre, wie's bei Windows aussieht kann dir keiner sagen.
Die von dir bemängelten Zeilen haben zumindest noch eine Funktion, und sind kommentiert. Ausserdem kenne ich zumindest keine alternative die diese Funktion(en) bietet. Es geht doch darum ein Problem zu lösen, nicht wer den schönsten Code schreibt, oder?

Gast+2008-05-16 21:43:51--

moritz+2008-05-15 21:37:00--

Also csvdiff ist mir schon noch den ersten paar Codezeilen suspekt:

Die von dir bemängelten Zeilen haben zumindest noch eine Funktion, und sind kommentiert.

Zeile 4 und 5 haben wohl keine Funktion - zumindest nicht die gewünschte. - Oder?

Grüße, Christoph

Gast+2008-05-16 21:43:51--

Im Linux Sourcecode steht tausende male "fuck" wenn ich nicht irre, wie's bei Windows aussieht kann dir keiner sagen.

tested with linux 2.6.24 (gentoo-sources-2.6.4-r4)

So böse ist der Linuxcode gar nicht ;o)

Und über die bemängelten Zeilen mag man sich streiten; TIMTOWTDI dürfte an diesem Punkt stark einsetzbar sein. Aber ich gebe zu, ich habe mir den Quelltext des Skriptes (noch) nicht zu Gemüte geführt und die zitierten Zeilen lösen bei mir auch keine Freude aus; ich weiß nicht, ob ich es suspekt nennen würde; aber ich würde die Zeilen so auf jeden Fall nicht in einem meiner Skripte einsetzen...

Übrigens freut sich der Autor auch über Mails, in denen man zum Ausdruck bringt, dass einem das Skript etwas gebracht/geholfen hat. (Ich durfte ihn letztens kennenlernen und er sagte, dass die Menge des Feedbacks in keiner Relation zur Menge der Downloads steht...)

Gast+2008-05-16 21:43:51--

[Die von dir bemängelten Zeilen haben zumindest noch eine Funktion, und sind kommentiert.

naja, zum einen ist das string-eval nicht notwendig, es könnte auch ein block-eval sein,
zum anderen sind zeile 4 und 5 überflüssig, denn sie kommen in jedem fall zu spät, das use
wird schon vorher ausgeführt. es kann natürlich trotzdem sein, dass ein programm mit solchen
unschönenheiten gut ist, aber wenn man mehr von solchen sachen findet, kann man sich vielleicht
gedanken machen.