Beefy Boxes and Bandwidth Generously Provided by pair Networks
There's more than one way to do things
 
PerlMonks  

Re^3: Changing every subroutine in many perl scripts

by nitin1704 (Sexton)
on Jul 25, 2012 at 04:17 UTC ( #983542=note: print w/ replies, xml ) Need Help??


in reply to Re^2: Changing every subroutine in many perl scripts
in thread Changing every subroutine in many perl scripts

Okay, I got it now, after going through the CPAN documentation a bit more.

use strict; use warnings; use PPI; my $file = '/path/to/perl/script.pl'; my $Document = PPI::Document->new($file) or die "oops"; for my $sub ( @{ $Document->find('PPI::Statement::Sub') || [] } ) { unless ( $sub->forward ) { my @elements = $sub->children; for ( my $i = 0 ; $i < @elements ; $i++ ) { if ( ref $elements[$i] eq "PPI::Structure::Block" ) { $elements[$i]->start->add_content("my mars code"); } } } } $Document->save($file.".new");


Comment on Re^3: Changing every subroutine in many perl scripts
Download Code
Re^4: Changing every subroutine in many perl scripts
by GrandFather (Cardinal) on Jul 25, 2012 at 22:24 UTC

    The loop would be better written:

    for my $sub (@{$Document->find('PPI::Statement::Sub') || []}) { next if $sub->forward; for my $child ($sub->children) { if (ref $child eq "PPI::Structure::Block") { $child->start->add_content("my mars code"); } } }

    The next if ... saves a level of indentation and is easier to understand (at least for me) than an unless.

    Using a Perl style for loop in place of the C style for loop is easier to read, easier to understand, more compact and much less likely to get wrong.

    True laziness is hard work

Log In?
Username:
Password:

What's my password?
Create A New User
Node Status?
node history
Node Type: note [id://983542]
help
Chatterbox?
and the web crawler heard nothing...

How do I use this? | Other CB clients
Other Users?
Others making s'mores by the fire in the courtyard of the Monastery: (12)
As of 2014-09-22 11:46 GMT
Sections?
Information?
Find Nodes?
Leftovers?
    Voting Booth?

    How do you remember the number of days in each month?











    Results (189 votes), past polls