Beefy Boxes and Bandwidth Generously Provided by pair Networks
go ahead... be a heretic
 
PerlMonks  

Re^3: perl substitue till found pattern in aline

by rjt (Deacon)
on Dec 03, 2012 at 08:57 UTC ( #1006822=note: print w/ replies, xml ) Need Help??


in reply to Re^2: perl substitue till found pattern in aline
in thread perl substitue till found pattern in aline

Hi!

Your code is slower by about a factor of 3:

use Benchmark qw/:all :hireswallclock/; my $DATA = '01 fines name 2222 -P sws -1 reee.tee rrt'; cmpthese(-10, { 'rjt' => q{ $_ = $DATA; s/ (?=.*\s?-1 )/,/g; }, 'pushtaev' => q{ $_ = $DATA; my ( $left, $right ) = split /-1/, $_, 2; $left =~ s/ /,/g; join '-1', $left, $right; }, });
Rate pushtaev rjt pushtaev 1875006/s -- -75% rjt 7582716/s 304% --

Regarding readability, it will depend who you ask in this case. In my experience, I tend to find a simple regex easier to read than tearing apart and piecing a solution back together with split, s///, and join.

Lastly, in your code, replacing this:

    print join '-1', $left, $right;

... with this:

    print $left, '-1', $right;

... resulted in a 32% speed improvement before I/O on your solution (still ~2.2x slower than the straight regex) and puts the terms in their natural order, which is probably more readable than the strange use of join with a delimiter to concatenate three strings.

I hope this is useful feedback to your feedback!


Comment on Re^3: perl substitue till found pattern in aline
Select or Download Code

Log In?
Username:
Password:

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

How do I use this? | Other CB clients
Other Users?
Others browsing the Monastery: (8)
As of 2015-07-31 06:02 GMT
Sections?
Information?
Find Nodes?
Leftovers?
    Voting Booth?

    The top three priorities of my open tasks are (in descending order of likelihood to be worked on) ...









    Results (274 votes), past polls