in reply to Edit a List of Files
A mixture of, mostly minor, stylistic issues:
use strict; use warnings;
That mantra should come before you do anything else! Always! Regardless of projet size! No exceptions! Comprende?
(condition) or print "fail string";
is cute, but
print "fail string" if condition;
is clearer. In cases where the fail condition is non-zero it is even better to make it explicit:
print "fail string" if 0 != (condition);
It is tempting to comment things that you have just learned, but that can lead to over commenting and make it harder to grok the flow of the code.
$var1 . 'string1' . "string2" is better written as "${var1}string1string2". Note the ${var1} usage to clarify where the variable name ends.
The block for while (<FILE2CHANGE>) { is not indented.
|
|---|
| Replies are listed 'Best First'. | |
|---|---|
|
Re^2: Edit a List of Files
by lev36 (Sexton) on Apr 11, 2006 at 20:54 UTC | |
by GrandFather (Saint) on Apr 11, 2006 at 21:03 UTC |