This is a slower and less readable version of what I posted 15 hours ago.
- $aoa[$i] (in $x < scalar($aoa[$i])) is buggy. It should be @{ $aoa[$i] }
- Your code (specifically your use of sort) fails when there are more than 10 columns in each row.
- Why do you hardcode "4" in one place but use "scalar(@aoa)" in another? That should tell you something's wrong.
- There's no reason to use a hash here. It's not appropriate, and it just slows things down for nothing.
- "hash" is an awful name for a variable.
- $x = $x + $y; can be written more concisely as $x += $y;.
- scalar(@aoa) can be written more concisely as @aoa if the context is already scalar.
- for (my $i=0; $i<@aoa; $i++) can be written more concisely as for my $i (0..$#aoa).
Finally, your post formatting is buggy. Put code and other preformatted data in <c>...</c> tags.
-
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.
|
