comment on

You could also implement this with an array, instead of a recursive sub:

opendir(ROOT_DIR,"/")|| die "Error:$!\n";
my @todo = grep {$_ ne '..'} readdir(ROOT_DIR);
closedir(ROOT_DIR);
while (my $what = shift @todo) {
  if (-f $what){
    log_sbit($what) if (-u $what || -g $what);
  }
  elsif (-d $what && opendir(SUB_DIR, $what){
    push @todo, map {"$what/$_"} grep {$_ ne '..'} readdir(SUB_DIR);
    closedir(SUB_DIR);
  }
}
[download]

Untested but perhaps you'll like the approach and work things out. Remember to build full paths each time (that's what the map would be doing if this were working code). When you finally run out of @todo elements you've searched everything.

And I can't resist: for more information on recursion, see Recursion.

In reply to Re: Recursion by TheoPetersen
in thread Recursion by Anonymous Monk

Are you posting in the right place? Check out Where do I post X? to know for sure.
Posts may use any of the Perl Monks Approved HTML tags. Currently these include the following:
<code> <a> <b> <big> <blockquote> <br /> <dd> <dl> <dt> <em> <font> <h1> <h2> <h3> <h4> <h5> <h6> <hr /> <i> <li> <nbsp> <ol> <p> <small> <strike> <strong> <sub> <sup> <table> <td> <th> <tr> <tt> <u> <ul>
Snippets of code should be wrapped in <code> tags not <pre> tags. In fact, <pre> tags should generally be avoided. If they must be used, extreme care should be taken to ensure that their contents do not have long lines (<70 chars), in order to prevent horizontal scrolling (and possible janitor intervention).
Want more info? How to link or How to display code and escape characters are good places to start.


Think about Loose Coupling
	PerlMonks