Re: Re: file case change

Okay:
change


for (readdir DIR) {
print "$_<br>";
    rename("$path/$_", lc("$path/$_")) or print "Couldn't rename $file
+: $!\n";
}
[download]

to:


for (readdir DIR) {
next if (/^\.{1,2}$/);
print "$_<br>";
    rename("$path/$_", lc("$path/$_")) or print "Couldn't rename $_: $
+!\n";
}
[download]

GreetZ!,

ChOas
print "profeth still\n" if /bird|devil/;

Comment on Re: Re: file case change Select or Download Code

Replies are listed 'Best First'.
Re: Re: Re: file case change by tachyon (Chancellor) on Mar 15, 2002 at 20:34 UTC
This is really bad code. First you lowercase the whole file path ensuring that any uppercase path will break this code (as when you lowercase it you will then try to write along a non existant path). Next you seem to have forgotten that '.' and '..' are not the only directories you might find. Finally you should never print unescaped arbitrary strings in HTML. cheers tachyon s&&rsenoyhcatreve&&&s&n.+t&"$'$`$\"$\&"&ee&&y&srve&&d&&print	[reply]

Replies are listed 'Best First'.

Re: Re: Re: file case change
by tachyon (Chancellor) on Mar 15, 2002 at 20:34 UTC

This is really bad code. First you lowercase the whole file path ensuring that any uppercase path will break this code (as when you lowercase it you will then try to write along a non existant path). Next you seem to have forgotten that '.' and '..' are not the only directories you might find. Finally you should never print unescaped arbitrary strings in HTML.

cheers

tachyon

s&&rsenoyhcatreve&&&s&n.+t&"$'$`$\"$\&"&ee&&y&srve&&d&&print

[reply]