There are two problems with your code: - Assuming SORTD is some form of file-handle, the first time you arrive at the while-loop ($i being equal to 0), the while-loop will totally exhaust the filehandle. All following calls to readline(SORTD) (when $i is incremented and the while-loop is entered again, will just return undef and the while-loop will not even start. Re-entering the loop does not reset the filehandle to the beginning of the file.
- The value of $i remains constant throughout each iteration of the for loop. So the while-loop will see the same value of $i for the whole of the file you are reading.
One more comment: it is more customary to write readline(SORTD) as <SORTD>
CountZero A program should be light and agile, its subroutines connected like a string of pearls. The spirit and intent of the program should be retained throughout. There should be neither too little or too much, neither needless loops nor useless variables, neither lack of structure nor overwhelming rigidity." - The Tao of Programming, 4.1 - Geoffrey James
-
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.
|