simplify code for ladder sorting

[sc2-widget] / getsc2clan

diff --git a/getsc2clan b/getsc2clan
index b23d2ac4f2f17443db86924791e78c3266333181..6f486f9e6d827ef1085342bb5073aca8abc76f57 100755 (executable)
--- a/getsc2clan
+++ b/getsc2clan
@@ -6,43 +6,48 @@ use utf8;
 use Data::Dump qw( pp );
 use LWP::Authen::OAuth2;
 use JSON qw( decode_json );
-use List::Util qw( all );
+use List::MoreUtils qw( all part nsort_by );
 
-my ($profileid, $clanmatch) = @ARGV;  # clan host and name
-$profileid and $profileid =~ /\A\d+\z/
-       or die "Usage: $0 <profile id> [<clan name>]\n";
+my ($profiles, $clanmatches) = part { /\D/ } @ARGV;  # separate numbers
+@{$profiles}
+       or die "Usage: $0 <profile id>... [<clan name>...]\n";
+my ($clanmatch) = map { $_ && qr/\A(?:$_)\z/i } join '|', @{$clanmatches};
 
-my $bliz = LWP::Authen::OAuth2->new(
-       client_id               => '7f0f95ac9529474f854ee8d68a12c3e0',
-       client_secret           => 'Kfa8n98UAaDo4brOeqxe9C2kJE9pqpSd',
-       token_endpoint          => 'https://us.battle.net/oauth/token',
+my %auth = do './.blizzard.passwd.pl' or die "no auth setup: $!\n";
+my $bliz = LWP::Authen::OAuth2->new(%auth,
+       token_endpoint          => 'https://eu.battle.net/oauth/token',
        request_required_params => [qw( client_id client_secret grant_type )],
 );
 $bliz->request_tokens(grant_type => 'client_credentials');
 
 sub blizget {
        my $args = join('/', @_);
-       my $res = $bliz->get("https://us.api.blizzard.com/sc2/$args");
+       my $res = $bliz->get("https://eu.api.blizzard.com/sc2/$args");
        $res->is_success or die $res->status_line;
        my $json = $res->decoded_content;
        return decode_json($json);
 }
 
-# find largest group consisting entirely of clan members
 # prefer deprecated interface to prevent costly ladder search
-my $ladderdata = blizget(legacy => profile => 2 => 1 => $profileid => 'ladders');
+my @ladderdata = map {
+       blizget(legacy => profile => 2 => 1 => $_ => 'ladders')
+} @{$profiles};
+my %ladders = (
+       map { $_->{ladder}->[0]->{ladderId} => $_ } # unique
+       grep { $_->{ladder}->[0]->{division} }
+       map { $_->{currentSeason}->@* } @ladderdata
+);
 my @ladders = (
-       sort {
-               $b->{ladder}->[0]->{wins}+$b->{ladder}->[0]->{losses} <=>
-               $a->{ladder}->[0]->{wins}+$a->{ladder}->[0]->{losses}
+       nsort_by {
+               -($_->{ladder}->[0]->{wins} + $_->{ladder}->[0]->{losses})
        } # activity desc
        grep {
                !$clanmatch or
-               all { fc $_->{clanName} eq fc $clanmatch } $_->{characters}->@*
+               all { $_->{clanName} =~ $clanmatch } $_->{characters}->@*
        } # members
-       grep { $_->{ladder}->[0]->{division} }
-       $ladderdata->{currentSeason}->@*
+       values %ladders
 ) or die "No matching groups found\n";
+
 my (@members, %memberidx);
 $memberidx{ $_->{id} } //= push(@members, $_) && $#members
        for map { $_->{characters}->@* } @ladders;