Hi. Your post is useful, thanks. I have gone through points that you have made and added my comments.
retrieve_ESLIM_codes uses the global variable $dbh
Should I be calling the database from within each subroutine, seperately?
retrieve_ESLIM_codes returns @sorted_parameters_A but the return result is not used
I was planning to use this at a future point. Anyway this is a useful reminder thanks.
look_through_file uses and updates $continue_tag, but the result is not used
It is supposed to be used. When the end of file is reached the $continue_tag goes to "FALSE" and this gets passed out of the subroutine from which point the while loop (that makes the subroutine calls in the first place) is broken.
look_through_file populates a local @array_of_lines, but the result is not used
Again. This will be used at a future point. Although I did not correctly build this array (as hinted in your next point).
look_through_file sets a local $entry_no_new, but the result is not used
At one point I did not correctly use this variable. I have removed this incorrect use. However, I don't believe that you are correct in saying that the result is not used. Basically the code is saying that if you get to the next entry number then exit the subroutine.
create_output is passed parameters that have no value
I'm not sure if I understand what you are suggesting here. Is this problem covered in the other posts?
Posts are HTML formatted. Put <p> </p> tags around your paragraphs. Put <code> </code> tags around your code and data!
Titles consisting of a single word are discouraged, and in most cases are disallowed outright.
Read Where should I post X? if you're not absolutely sure you're posting in the right place.
Please read these before you post! —
Posts may use any of the Perl Monks Approved HTML tags:
- a, abbr, b, big, blockquote, br, caption, center, col, colgroup, dd, del, details, div, dl, dt, em, font, h1, h2, h3, h4, h5, h6, hr, i, ins, li, ol, p, pre, readmore, small, span, spoiler, strike, strong, sub, summary, sup, table, tbody, td, tfoot, th, thead, tr, tt, u, ul, wbr
You may need to use entities for some characters, as follows. (Exception: Within code tags, you can put the characters literally.)
| |
For: |
|
Use: |
| & | | & |
| < | | < |
| > | | > |
| [ | | [ |
| ] | | ] |
Link using PerlMonks shortcuts! What shortcuts can I use for linking?
See Writeup Formatting Tips and other pages linked from there for more info.