Thread MP3Tagger: Ein MP3-Tag-Editierungsprogramm (13 answers)
Opened by Hrhon at 2006-04-22 23:39

Hrhon
 2006-04-23 21:25
#38703 #38703
User since
2006-04-22
5 Artikel
BenutzerIn
[Homepage] [default_avatar]
Quote
ich dachte, du suchst leute, die vielleicht mithelfen. da sollte man sich
so tipps wie perlstyle etc. schon zu herzen nehmen bzw. einfach mal
durchgelesen haben.
ich jedenfalls finde den code ziemlich unleserlich, und die theorie "viel
code auf möglichst wenig platz" halte ich für wesentlich schlechter als
den ansatz "subroutinen möglichst so klein, dass sie in ein standard-
terminalfenster passen", d.h. eine subroutine sollte von der funktionalität
(nicht unbedingt von der anzahl der zeichen) klein gehalten werden.
so hat man jede funktion von anderen abkekapselt und sieht alles,
was man sehen muss, ohne zu scrollen, und doch kann man
leerzeichen zur leserlichkeit einstreuen.
das ist zumindest das, was *ich* von vielen erfahrenen programmierern
höre/lese, mir ist wirklich noch keiner begegnet, der die erste theorie
vertritt.
aber wenn du damit selbst klarkommst und es nicht zu einem öffentlichen projekt machen willst, was schade wäre...


die Philosophie möglichst kleine Funtkionen von der Funktionalität her hab ich auch gemeint. Hab ich glaub ich zum großteil, wenn auch nicht immer, umgesetzt.
Was genau stört dich eigentlich an meinem Code. Also dass ich manchmal zu viele Klammern Benutze, weiss ich. Da könnt ich mich evtl einschränken. Aber ansonsten... Ich nehme immer selbsterklärende Variablen, ich erkläre sogar noch die meisten Variablen mit einem kurzen Kommentar, ich schreibe nur einen Befehl pro Zeile (mit Ausnahme von Funktionsaufrufen innerhalb von If-Bedingungen. Aber das dürfte je nicht das große Problem sein) und ich erkläre auch die meisten Funktionen mit einem kurzen Kommentar. Das ist eigentlich schon mehr, als die meisten Programmierer machen. Das einzige, was jetzt etwas anders ist , das sind meine geschweiften Klammern

anstatt

Code: (dl )
1
2
3
4
if (...)
{
...
}


schreib ich

Code: (dl )
1
2
3
if (...) {
...
}

und ich lass halt Lerrzeilen weg. Aber ich geb dir recht, am Anfang fand ich die Schreibweise, die ich jetzt hab auch verreckt und ich dachte, ich gewöhn mich nie dran. Aber mitlerweile find ich sie ok.

Mal sehen, wenn ich Zeit find, les ich mir das Perlstyle-Teil mal durch. Aber die Dinge, die ich oben beschrieben haben, gehören eigentlich eh schon zum guten Programmierstil und ich find, ich halt mich auch im Großen und Ganzen ganz gut daran. Aber irgendwie scheint dir mein Code-Bild wohl überhaupt nicht zuzusagen, frag mich nur, worans liegt...

Quote
darauf weist dich die warnung hin. an $msg wird nur der erste string
angehängt, nicht der zweite.
warnungen sind häufig der hinweis auf fehler, die du sonst nicht
finden würdest bzw. die du erst durch einen fehler im laufenden
betrieb bemerkst und dann auch noch suchen musst.
benutze warnings, du wirst später dankbar sein =)


ok, überredet. Ich arbeite ab jetzt mit warnings :-D

View full thread MP3Tagger: Ein MP3-Tag-Editierungsprogramm