comment on

Hi Perl Ally,

Some observations and questions:
In readOnly subroutine, why declare the variable first before assignment? i.e

    my @contents;
    my $filePath; my $file;
    my $errMsg;
[download]

From the usage in the subroutine, you can actually use them straight. At least, if not for anything, it shorten the line of codes to read through.
You really don't have to do explicit close $file; The file should close, when the subroutine returns.
Then in your extractData subroutine, what does $pointerProj variable stand for. It must be a variable defined somewhere in the code?
Suggestion (troubleshooting): Why not print out, what you have in "@projects" before the for-loop, to be sure you are getting desired data first?
NOTE: Since one can't have more information than already provided. One can on speculate.

If you tell me, I'll forget.
If you show me, I'll remember.
if you involve me, I'll understand.
--- Author unknown to me

In reply to Re^5: Passing Along Arrays by 2teez
in thread Passing Along Arrays by Perl_Ally

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.


Think about Loose Coupling
	PerlMonks