Patch: Sortierung in diversen Listen anpassen

Wiimm · 11. Juni 2016

Hier ist ein Patch, der dafür sorgt, dass die Tabelle der Mitspieler nicht mehr alphabetisch, sondern gemäß Punkte sortiert werden.
Dazu musste der Punkte-Stand mittels LEFT-JOIN in die Hauptabfrage einfließen, um nach ihr sortieren zu können. Die Einzelabfrage des Punktestandes konnte dann entfallen.

Das komische Konstrukt mit "<<< __EOT__" und "__EOT__;" ist ein sogenanntes "here document", wobei der frei wählbare Marker EOT = End-Of-Text bedeutet.. Alles dazwischen wird als mehrzeilige Zeichenkette betrachtet. Ich empfinde es viel übersichtlicher als die vielen Zeichenketten-Operationen. Das ganze birgt aber eine Falle: PHP verlangt, dass der Ende-Marker (hier "__EOT__;" ganz alleine in einer Zeile steht und *NICHT* eingerückt ist.

Diff

--- teilnehmer.ansicht.php.orig 2016-05-01 15:48:08.000000000 +0200
+++ teilnehmer.ansicht.php 2016-06-11 08:59:32.097820024 +0200
@@ -67,15 +67,20 @@
$StartPage = ((isset($RUNTIME['PATH'][2]) ? $RUNTIME['PATH'][2] : 1) -1)*$CONFIG['RANGLISTE']['UPP'];
-$query = "SELECT * FROM ". $CONFIG['MYSQL']['PREFIX'] ."users WHERE isUnlocked = 1 ORDER BY usr LIMIT ". $StartPage .",". $CONFIG['RANGLISTE']['UPP'];
+$query = <<< __EOT__
+ SELECT u.*, r.pkt
+ FROM {$CONFIG['MYSQL']['PREFIX']}users u
+ LEFT JOIN {$CONFIG['MYSQL']['PREFIX']}rangliste_boni r
+ ON r.uid = u.id
+ WHERE u.isUnlocked = 1
+ ORDER BY ifnull(r.rang,999999999), u.usr
+ LIMIT $StartPage, {$CONFIG['RANGLISTE']['UPP']}
+__EOT__;
$sql = mysql_query($query);
while ($row = mysql_fetch_array($sql))
{
- $sqlpkt = mysql_query("SELECT pkt FROM ". $CONFIG['MYSQL']['PREFIX'] ."rangliste_boni WHERE uid ='". $row['id'] ."'");
- $rowpkt = mysql_fetch_array($sqlpkt);
-
- $pkt = $rowpkt['pkt'];
+ $pkt = $row['pkt'];
$rank = (int)getRanglistenplatz("boni", $row['id']);
$name = $row['usr'];
$id = $row['id'];

Alles anzeigen

Nachtrag:
Beim Stöbern in diesem Forum habe ich gerade "Rangliste: Auf einen Blick ..." entdeckt. Dieses macht den Patch evtl unnötig. Ich werde ihn aber bei mir belassen. Dann haben wir halt ne 2.Übersicht, die nach Punkten sortiert ist und ein wenig optimierter arbeitet.

Wiimm · 11. Juni 2016

Diesmal geht es um die Tipp-Übersicht. Ich finde es interessanter (werde mir aber noch die Rückmeldung der Mitspieler einholen), wenn die Tipps nach dem Tipp-Ergebnis sortiert sind anstatt alphabetisch nach dem Namen des Spielers. So kann man viel schneller Extrem-Tipps und andere Ausreißer ausmachen und auch sehen, wer dasselbe getippt hat.

Daher wird die Tabelle nach Tor-Differenz sortiert und bei Gleichheit nach der Anzahl der Tore.
Der Patch hierzu:

Diff

--- statistik.spiel.php.orig 2016-06-04 20:05:08.000000000 +0200
+++ statistik.spiel.php 2016-06-11 09:33:15.878899892 +0200
@@ -686,7 +686,7 @@
LEFT JOIN ". $CONFIG['MYSQL']['PREFIX'] ."spiele AS spiele ON spiele.id = tipps.spiel
WHERE tipps.spiel = '".mysql_real_escape_string($RUNTIME['PATH'][2])."'
AND users.isUnlocked = 1
- ORDER BY name ASC
+ ORDER BY tipps.gast - tipps.heim,
+ tipps.gast * tipps.gast - tipps.heim * tipps.heim,
+ tipps.heim DESC,
+ name
LIMIT ".$begin.",".$length;
}

Alles anzeigen

Horst · 11. Juni 2016

@Wiimm

Ich hab mal den Titel angepasst - hoffe ist in Ordnung

Wiimm · 11. Juni 2016

Zitat von Horst

Ich hab mal den Titel angepasst - hoffe ist in Ordnung

Ok!
Und mein 2. Betrag kann gelöscht werden, da ich den Inhalt als Nachtrag im ersten ergänzt habe.

floriber · 11. Juni 2016

finde ich super. bei dem spiel gestern kam dann auch die frage auf, ob man nicht auf der seite auch darstellen könnte, wie oft 1:0, 2:1 etc getippt wurde, also die häufigkeit der tipps. könnte bei den abgegeben tipps bei dem spiel stehen.

Wiimm · 11. Juni 2016

Ich habe den Patch für die Tipp-Übersicht ergänzt:

Code

ORDER BY tipps.gast - tipps.heim,
tipps.gast * tipps.gast - tipps.heim * tipps.heim,
tipps.heim DESC,
name

Die erste Zeile (höchste Priorität) sorgt dafür, dass höhere Differenzen ganz nach oben bzw. ganz nach unten rutschen. Die zweite Zeile sortiert 4:1 vor 3:0, aber 1:4 nach 0:3. Das macht die Tabelle symmetrisch. Die dritte Zeile ist relevant für Unentschieden (1:1 vor 0:0) und die letzte Zeile sortiert dann bei identischen Tipp nach Namen.

floriber · 11. Juni 2016

Danke nochmals hierfür. was man natürlich nun vor den spielen schon sieht ist wer quasi für welche mannschaft getippt hat.
steht ein spieler auf ersten seite ganz oben dann hat er wohl für die heimmannschaft getippt und jemand der auf der letzten seite steht eben für die gastmannschaft...

SandDiego · 11. Juni 2016

Super Tipps!

Aber bekomme ich noch eine Sortierung in sieger.php hin, wenn die Leute die gleiche Punktzahl haben? Dann ist es irgendwie völlig zufällig gewürfelt. Kann man Leute mit gleicher Punktzahl nicht nach Alphabet des Usernamens sortieren?
oder zur Not noch Reihenfolge der Registrierung?

kung666 · 11. Juni 2016

Hallo,

ja, die Rangliste sollte nach diesen Kriterien sortiert sein:

* Punkteanzahl
* Anzahl richtiger Tipps
* Anzahl Tipps mit richtiger Tordifferenz
* Anzahl Tipps mit richtiger Tendenz
* dann kann man immer noch alphabetisch sortieren

Das wäre m.M. nach super
Danke!

Wiimm · 11. Juni 2016

Sollte machbar sein. Kann mir jemand sagen, welche Felder was bedeuten:

Code

+-------+---------+------+-----+---------+-------+
| Field | Type | Null | Key | Default | Extra |
+-------+---------+------+-----+---------+-------+
| platz | int(10) | NO | | 0 | |
| rang | int(10) | NO | | 0 | |
| uid | int(10) | NO | | 0 | |
| tipps | int(10) | NO | | 0 | |
| e | int(10) | NO | | 0 | |
| d | int(10) | NO | | 0 | |
| t | int(10) | NO | | 0 | |
| b | int(10) | NO | | 0 | |
| pkt | int(10) | NO | | 0 | |
| lsp | int(10) | NO | | 0 | |
| ast | int(10) | NO | | 0 | |
| lst | int(10) | NO | | 0 | |
+-------+---------+------+-----+---------+-------+

Alles anzeigen

kung666 · 11. Juni 2016

e = Ergebnis richtig getippt
d = Tordifferenz richtig getippt
t = Tendenz richtig getippt
b = Bonuspunkte
pkt = Punkte
lsp = letztes Spiel
ast = aktueller Spieltag
lst = letzter Spieltag

Wiimm · 11. Juni 2016

Hätte ich auch selbst drauf kommen könne, Danke Kurt!

Wir reden über Rangliste / Auf einen Blick?
Eine erste Recherche ergab, dass ich hierfür die globale Funktion returnRanglisteKompakt() anfassen muss. Dieses hat dann Auswirkungen auf diverse Tabellen. Evtl.muss ich auch die Rangberechnung anpassen.

Allerdings warte ich erstmal den Bug Fix mit Spiel 14 ab, da ich sonst keine vernünftige Tabelle habe, an der ich die Sortierung verifizieren kann (Fast jeder Mitspieler hat einen eigenen Rang).

SandDiego · 12. Juni 2016

Zitat von Wiimm

Fix mit Spiel 14

Was ist das los? Man muss hier wirklich überall mitlesen glaube ich.

Hat sich erledigt. habe es gefunden.

Wiimm · 12. Juni 2016

Die Tabelle der Mitspieler wird jetzt anders sortiert, da der an anderer Stelle berechnet Rang berücksichtigt wird. Der ifnull()-Teil wird benötigt, um Spieler ohne Tipp ans Tabellenende zu bannen. Der erste Post ist entsprechend aktualisiert.

Code

ORDER BY ifnull(r.rang,999999999), u.usr

Und da sind wir schon beim noch offenen Thema. Die Ränge werden durch diverse Funktionen vorberechnet. Eine einfache Modifikation der SQL-Abfrage ist daher nicht möglich. Es müssten die Funktionen angepasst werden, was zusätzliche Einarbeitung in den PHP-Code benötigt. Und dieses scheue ich.

Außerdem berücksichtigen dieses Funktionen doch den Wunsch nach ...
* Punkteanzahl
* Anzahl richtiger Tipps
* Anzahl Tipps mit richtiger Tordifferenz
* Anzahl Tipps mit richtiger Tendenz
* dann kann man immer noch alphabetisch sortieren
..., oder irre ich mich?

Evtl. reden wirr auch aneinander vorbei und meinen unterschiedliche Statistiken.
Daher meine Frage: Welche Statistik meint ihr?

SandDiego · 12. Juni 2016

Ich rede immer von der sieger.php, die auf HOME als erstes angezeigt wird.

Aber mach dir keine Arbeit! Wenn man es nicht alleine mit dem Quellcode dieser Datei ändern kann ist das alles zuviel. Und nachher bedenkt man was nicht und et läuft nichts mehr.

Wiimm · 12. Juni 2016

Da geht doch der berechnete Rang mit ein. Und wenn ich Rangliste / Auf einen Blick richtig deute, dann gehen Dinge wie "Anzahl richtiger Tipps" in die Sortierung als Sortierfaktor mit ein. Ich kann zumindest bei unserem kleinen 15-Personen-Tipp im Moment nichts gegenteiliges feststellen.

Aber wie gesagt: Änderungen an diesen vorgerechneten Rängen bedeuten für mich extra Einarbeitungszeit, die ich eher nicht aufbringen möchte.

kung666 · 12. Juni 2016

Guten Morgen

sorry für die späte Antwort - musste mich erst vom England-Match gestern erholen

Also, soweit ich das heute überblicke, stimmt die "Rangliste - Auf einen Blick" so wie oben gewünscht mit den Kriterien. Meine Tipprunde besteht aus 128 Leuten - und da passt das.
Da ist also nichts mehr zu tun.

Die sieger.php unterscheidet nicht zwischen ex-aequo-Plätzen - aber damit kann ich leben.

Danke für die Unterstützung... für mich passt das so...

compoundbow83 · 12. Juni 2016

Hallo Wiimm,
hast du hierfür vielleicht eine Tipp für mich?:
Ranglisten von Tippgruppen

Und ist es beim Treppchen noch kurzfristig möglich, dass wenn mehrere die gleiche Punktezahl haben, dass sie dann auch den gleichen Platz haben?

Compoundbow83

Wiimm · 12. Juni 2016

Zitat von compoundbow83

Und ist es beim Treppchen noch kurzfristig möglich, dass wenn mehrere die gleiche Punktezahl haben, dass sie dann auch den gleichen Platz haben?

Den Gedanken hatte ich auch schon. Ich bin aber der falsche Ansprechpartner, da ich nicht zum Entwickler-Team gehöre und mich erst in die Tiefen einarbeiten müsste.

oldie · 12. Juni 2016

@ Wiimm
mal wieder
wartet auf einen fix für Ergebnisse spiel 14

wenn die Ergebnisse für die spiele geladen werden werden als erstes die alten Ergebnisse gelöscht und nicht nur die alles was relevant ist wird gelöscht also auch wenn ein Ergebnis im spiel 14 schon drin steht (das war dann noch vom testen ) sollte aber schon lange weg sein da die Tabellen schon vor Spielbeginn einige male geleert wurde
im dump steht als erstes dieses

TRUNCATE TABLE `emtipp_ergebnisse`;
TRUNCATE TABLE `emtipp_tore`;
TRUNCATE TABLE `emtipp_spielerstat`;

auserdem verführt mir Wimme die User viel zu viel zum herum Murksen im code
das Tippspiel läuft ohne dieses herumgefummel eigentlich richtig und ohne Fehler

Tags