For the sample data, your y-values are monotonic. If this is generally true, there's no point in including them in the comparison.
my $modified = reduce{
my( $x1, $y1, $x2, $y2 ) = ( @{ @{ $a }[ -1 ] }, @$b );
$x2 < $x1 ? pop @{ $a } : push @{ $a }, $b;
$a;
} [ shift @points ], @points;
Assuming you necessarily hold all data in memory, you could also use
splice in a while, which feels more natural to me:
my $i = 1;
while ($i < @points) {
if ($points[$i-1][0] > $points[$i][0]) {
splice @points,$i-1,2;
--$i or $i = 0;
} else {
$i++;
}
}
or just a streaming
push/
pop, which is algorithmically what you've done, but reads better, I think.
my $modified = [];
for my $point (@points) {
if (@$modified and $modified->[-1][0] > $point->[0]) {
pop @$modified;
} else {
push @$modified, $point;
}
}
I'd like there to be a more streaming solution here, but there's no way to tell if the first point should have been accepted until you are past halfway in the list, and you don't
a priori know how long the list is.
Golfed:
my @m;
@m&&$m[-1][0]>$_->[0]?pop @m:push @m,$_ for @points;
#11929 First ask yourself `How would I do this without a computer?' Then have the computer do it the same way.
-
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.