X-Git-Url: http://git.shiar.net/gitweb.cgi/perl/list-index.git/blobdiff_plain/356b069f60818a6ef5f0f8cf97a78924675c2c5b..fd5749ab518cdf3c68e94fbad168aa2a7725d592:/lib/List/Index.pm?ds=inline
diff --git a/lib/List/Index.pm b/lib/List/Index.pm
index bf4e04d..339bd7d 100644
--- a/lib/List/Index.pm
+++ b/lib/List/Index.pm
@@ -23,14 +23,15 @@ sub ranges {
my $pages = $options->{pages } || 1 + int $#$self / $pagesize;
$pagesize = @$self / $pages;
- my $offset = $pagesize;
- my @links = (['']);
+ my $offset = $pagesize + .5;
+ my $penalty = 0;
+ my @links = ('');
while ($offset < @$self) {
my $link = substr $self->[$offset], 0, $length;
if ($context) {
- {
+ if ($offset > $context - 1 + $penalty) {
# take a value slightly before the current offset
- my $before = $offset > $context ? $self->[$offset - $context] : '.';
+ my $before = $self->[$offset - $context - 1 + $penalty];
# see how much of it matches the current link
my $trim = 1;
for my $match (split //, $before) {
@@ -41,6 +42,7 @@ sub ranges {
substr($link, $trim) = '' unless $trim > length $link;
}
+ $penalty = 0;
if ($offset + $context < $#$self) {
# take a value after the current offset
my $after = $self->[$offset + $context];
@@ -53,21 +55,27 @@ sub ranges {
# use this link if it's shorter
if ($trim < length $link) {
$link = substr $after, 0, $trim;
+ for ($offset .. $#$self) {
+ last if $self->[$offset + $penalty] =~ /^\Q$link/;
+ $penalty++;
+ }
}
}
}
- push @links, [$link];
+ push @links, $link;
$offset += $pagesize;
}
+ use List::MoreUtils 'uniq';
+ @links = uniq @links;
for my $i (0 .. $#links - 1) {
- my ($link, $lastchar) = $links[$i + 1]->[0] =~ /(.*)(.)/;
+ my ($link, $lastchar) = $links[$i + 1] =~ /(.*)(.)/;
$link .= $lastchar le 'a' ? '.' : chr( ord($lastchar) - 1 );
- next if $link eq $links[$i]->[0] and $i;
- $links[$i]->[1] = $link;
+ next if $link eq $links[$i] and $i;
+ $links[$i] .= '-'.$link;
}
- $links[-1]->[1] = '';
+ $links[-1] .= '-';
return \@links;
}
@@ -76,6 +84,7 @@ sub rangematch {
my ($link) = @_;
my ($s1, $s2) = $link =~ /([^-]*) - ([^-]*)/x
or return qr/^\Q$link/i;
+ $s1 =~ s/\.$//;
my @allow;
if (length $s1) {
@@ -86,10 +95,14 @@ sub rangematch {
my $prefix = '';
my $char;
for my $i (0 .. length($s1) - 1) {
+ my $lasti = $i == length($s1) - 1;
$char = substr $s1, $i, 1;
my $next = $char;
- $next = chr( ord($char) + 1 ) if length $s1 > $i + 1;
+ # do not include prefix character in final range
+ $next = chr( ord($char) + 1 ) unless $lasti;
+
my $last = 'z';
+ next if $next gt $last;
if (length $s2 > $i) {
if ($s2 =~ /^\Q$prefix/) {
$last = substr $s2, $i, 1;
@@ -98,10 +111,23 @@ sub rangematch {
next if $next gt $last;
}
}
+
+ if ($char eq '.') {
+ if ($last eq 'z') {
+# push @allow, $prefix if $i and $lasti;
+# next;
+ }
+# if ($last eq 'z') {
+# push @allow, $prefix if $i and $lasti;
+# next;
+# }
+ $next = 'a';
+ }
+
push @allow, $prefix."[$next-$last]";
}
continue {
- $prefix .= $char;
+ $prefix .= $char eq '.' ? '[^a-z]' : $char;
}
}
@@ -117,11 +143,16 @@ sub rangematch {
next if $c1 le $char;
}
}
- push @allow, $prefix."(?![$char-$last])"
+
+ if ($char eq '.') {
+ next if $i < length($s2) - 1;
+ }
+
+ push @allow, $prefix.'(?!['.($char eq '.' ? 'a' : $char)."-$last])"
if $i or $s1 eq '';
}
continue {
- $prefix .= $char;
+ $prefix .= $char eq '.' ? '[^a-z]' : $char;
}
push @allow, $prefix
@@ -139,18 +170,18 @@ __END__
=head1 NAME
-List::Index - Paginate alphabetic entries by finding minimal prefixes
+List::Index - Find and apply prefix ranges to paginate keywords
=head1 SYNOPSIS
use List::Index;
my $index = List::Index->new(\@values);
my @pages = $index->ranges({pagesize => 50});
- printf '%1$s ', @$_ for @pages;
+ say "$_" for @pages;
use List::Index 'rangematch';
- my $limit = rangematch('b-bmq'); # matches prefix like 'baa'..'bmq'
- @results = grep { $limit } @results;
+ my $limit = rangematch('b-bmq'); # ge 'b' && le 'bmq'
+ @request = grep { $limit } @values;
=head1 DESCRIPTION