RE: Subroutine question

could you just pass $script $arg and $file as arguments to the sub routine?

ie in the main loop:

$stuff_to_do[0] = "/path/to/script -e;;arguments;;file";
...

...
foreach $line (@stuff_to_do)
{
  ($script,$arg,$file) = split(/;;/,$line);
   &file_processing($script,$arg,$file);
}
...

then your subroutine would be:
sub file_processing {
    my ($script, $arg, $file) = @_;

    ....

}

I think that's what you're trying to do, but not sure..

-brad..
[download]

Comment on RE: Subroutine question Download Code

Replies are listed 'Best First'.
RE: RE: Subroutine question by BlaisePascal (Monk) on Sep 18, 2000 at 22:14 UTC
I think that's what he's trying to do, but he's also modifying the global `@array1`, and he wants to modify the global `@array2` the second time around. Personally, I'd consider that poor style, and rewrite it as... `@array1 = file_processing($script,$arg1,$file1); @array2 = file_proceesing($script,$arg2,$file2); ... sub file_processing { my ($script,$arg,$file) = @_; my @results; ... return @results; }` [download] Actually, I wouldn't... I'd call the sub "processfile", and make several other tweaks if I was writing this myself, but that's just me.	[reply] [d/l]

Replies are listed 'Best First'.

RE: RE: Subroutine question
by BlaisePascal (Monk) on Sep 18, 2000 at 22:14 UTC

@array1

@array2

Personally, I'd consider that poor style, and rewrite it as...

@array1 = file_processing($script,$arg1,$file1);
@array2 = file_proceesing($script,$arg2,$file2);
...
sub file_processing {
  my ($script,$arg,$file) = @_;
  my @results;
  ...
  return @results;
}
[download]

[reply]
[d/l]