Thread Variabler Variablenname
(21 answers)
Opened by Gast at 2008-08-22 14:46
also dein code sieht ziemlich chaotisch aus. eigentlich räumt man code erstmal auf, bevor man ihn postet. du hast da ziemlich viele überflüssige sachen drin. ausserdem benutzt du anscheinend kein strict und warnings, deshalb lies bitte
![]() Quote hier benutzt du $i, eine variable, die du nirgendwo mehr benutzt. hast du zuviele variablen übrig? Quote das schreibt sich üblicherweise so: Code (perl): (dl
)
next if /^#/; Quote nur ne kleinigkeit, aber das schreibt man so: Code (perl): (dl
)
s/\s+//g; warum? na weil man mit \s* beliebig viele \s ersetzt, also auch null. das ist im zweifel mehr arbeit für den interpreter. du willst ja nur ersetzen, wenn mindestens ein space vorkommt. Quote sorry, aber $xyz ist kein hilfreicher variablenname. desweiteren brauchst du die zeile nur an einer einzigen stelle, und kurz bevor du sie brauchst, weist du sie einer anderen variable zu. warum benutzt du nicht $_? oder im gegensatz dazu, wenn du "richtige" variablen lieber hast, warum weist du die zeile nicht so einer variablen schon im schleifenkopf zu? Quote du machst zweimal dasselbe split, und die variable @zeile lasst du völlig ungenutzt? ich weiss, du wirst vermutlich sagen, das kommt vom rumtesten. aber bevor man code postet, entfernt man sowas. das hat nix mit schikane zu tun. Quote hast du zuviel CPU übrig? ++$var inkrementiert eine variable eh schon, man braucht das ergebnis nicht nochmal derselben variable zuzuweisen. Quote mist? aha. könntest du das ein wenig näher erläutern? Always code as if the guy who ends up maintaining your code will be a violent psychopath who knows where you live. -- Damian Conway in "Perl Best Practices"
lesen: ![]() ![]() |