Hallo

Ich wollte mal fragen wie ich ein Bestenlistenscript verbessern kann?

Bisher mache ich immer so:


In einer Variablen $platz den Platz speichern von dem ab 10 Leute ausgegeben werden sollen.

Dann lese ich aus der Datenbank mittels folgendermaßen aus:

my $i = 1;
my $b = 0;
$statement = "SELECT name,punkte,id FROM user ORDER BY punkte DESC";
$sth = $dbh->prepare( $statement ) || die "Kann Statement nicht vorbereiten: $DBI::errstr\n";
$sth->execute || die "Kann Abfrage1 nicht ausfuehren: $DBI::errstr\n";
while(my @daten = $sth->fetchrow_array and $b < 10) {
if ($i>=$platz and $i < $platz+10) {
Dann wird hier eine Zeile geschrieben ausgegeben.

$b++;
}

$i++;
}
$sth->finish;


Allerdings sind dann z.B. bei einem Platz= 1500 , 1500 Scheifendurchläufe notwendig!!!

Kann man da was verändern?

Wenn du MySQL verwendest, kannst du eventuell die SQL-Statements LIMIT und OFFSET einsetzen, um nicht alle Zeilen abzurufen.

z.B. so:


Aber bitte nicht eins zu eins übernehmen. :)\n\n

<!--EDIT|jan10001|1068066265-->

Kann mich Geewiz nur anschließen, sofern es sich um MySQL dreht oder ein anderes RDBMS das LIMIT oder eine equivalente Funktion kennt, sollte die bei sowas genutzt werden.

Im Falle von MySQL sähe das dann so aus:


Was evtl. auch noch eine Lösung sein könnte, vorrausgesetzt es handelt sich nicht um zuviele Datensätze, könnte man einfach das komplette Ergebnis in einen Array packen und dann per:


Dabei kommts halt drauf an, wieviel Arbeitsspeicher zur verfügung steht...hab grade mal getesten, mein Perl (ActivePerl 5.8.0, Win2k) braucht bei einem Array mit 100.000 Einträgen und je 512Byte Daten immerhin schon 57MB Arbeitsspeicher ;)
Bei 10.000 Einträge sinds aber vertragbare 8MB :)

oder gleich mit Tie::File arbeiten? dann brauchst du nicht alles in den arbeitsspeicher laden.
(Nebenbei: wenn man eine Datei mit while(<FILE>) ... einliest, hat man auch immer nur eine Zeile im Arbeitsspeicher, weil das ein skalarer Kontext ist. bei foreach (<FILE>) oder @liste = <FILE>, bekommt man einen listenkontext, und die ganze datei wird zurueckgegeben)

oder einen Hash auf die Festplatte bannen, wo ein bisschen indiziert wird (z.B. mit DB_File), einmal benutzername -> id, und einmal id -> benutzername & punkte oder was auch immer.
aber wenn es viel wird, koennte da eine datenbank sicher einfacher sein\n\n

<!--EDIT|Strat|1068078507-->

Ich hatte das so verstanden das er die ganze Tabelle in 10er Blöcken ausgeben möchte. Wenn er noch einen zweiten Zähler in die Schleife setzt hätte er auch die jeweiligen Platznummern.