X-Git-Url: https://git.sesse.net/?a=blobdiff_plain;f=www%2Findex.pl;h=c63ae099bf36d21b3a5b0327300346262ffb8e9c;hb=77b9653b9c3704f8ab67af7b2f1d85a256ac5ca2;hp=33a719f0f43fed5828defb436778e5414df4d282;hpb=908a60f3b8f1e49cffb9eb0fddccc07b6039a6bd;p=wloh

diff --git a/www/index.pl b/www/index.pl
index 33a719f..c63ae09 100755
--- a/www/index.pl
+++ b/www/index.pl
@@ -41,6 +41,27 @@ sub color {
 	return int(255.0 * ($x ** (1.80)));
 }
 
+sub get_max_season {
+	my $dbh = shift;
+	my $ref = $dbh->selectrow_hashref('SELECT MAX(sesong) AS max_sesong FROM fotballserier');
+	return $ref->{'max_sesong'};
+}
+
+sub get_divisions {
+	my ($dbh, $season) = @_;
+
+	my @divisions = ();
+
+	my $q = $dbh->prepare('SELECT DISTINCT(divisjon) FROM fotballserier WHERE sesong=? ORDER BY divisjon');
+	$q->execute($season);
+
+	while (my $ref = $q->fetchrow_hashref) {
+		push @divisions, $ref->{'divisjon'};
+	}
+
+	return @divisions;
+}
+
 sub make_table {
 	my ($lowest_division, $used_ratings, $used_cov) = @_;
 
@@ -184,37 +205,24 @@ printf <<"EOF", $match_stddev;
     <form method="get" action="/">
 EOF
 
-$q = $dbh->prepare('SELECT MAX(sesong) AS max_sesong FROM fotballserier');
-$q->execute;
-my $season;
-while (my $ref = $q->fetchrow_hashref) {
-	$season = $ref->{'max_sesong'};
-}
+my $season = get_max_season($dbh);
+my @divisions = get_divisions($dbh, $season);
+$division = 1 if (!grep { $_ == $division } @divisions);
+my $max_division = $divisions[$#divisions];
 
 print <<"EOF";
      <p>Divisjon:
         <select name="divisjon" onchange="form.submit();">
 EOF
 
-$q = $dbh->prepare('SELECT DISTINCT(divisjon) FROM fotballserier WHERE sesong=? ORDER BY divisjon');
-$q->execute($season);
-
-my $found_division = 0;
-my $max_division;
-
-while (my $ref = $q->fetchrow_hashref) {
-	my $d = $ref->{'divisjon'};
+for my $d (@divisions) {
 	if ($d == $division) {
 		print "        <option value=\"$d\" selected=\"selected\">$d</option>\n";
-		$found_division = 1;
 	} else {
 		print "        <option value=\"$d\">$d</option>\n";
 	}
-	$max_division = $d;
 }
 
-$division = 1 if (!$found_division);
-
 print <<"EOF";
         </select>
         Avdeling: