comment on

Others have given you most of the answer, but it also looks as though you're pulling the record number out by some incredibly convoluted process. What is wrong with something like:

/Record(\d+):/ && do {
  $recnum = $1;
  s/$str1/$str2/g;
  s/$str3/$str4/g;
  ..
}
[download]

This checks for matching lines and pulls the record number out at the same time...

Note that the regex in the above may not be quite right as your original seems to back up 6 characters from the ':', but the regex can be amended to do that too.

A Monk aims to give answers to those who have none, and to learn from those who know more.

In reply to Re: Need help to fine tune perl script to make it faster( currently taking more than 30 minutes) by space_monk
in thread Need help to fine tune perl script to make it faster( currently taking more than 30 minutes) by anujajoseph

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.


Come for the quick hacks, stay for the epiphanies.
	PerlMonks