Spielerstatistik schliesst sich beim speichern

[Scooby]

Hallo zusammen
Nachdem ich die alten Statistiken importieren konnte, wollte ich jetzt eine neue eröffnen.
Aber jedes Mal, wenn ich die Spieler eintrage, und dann Speichern (Statistik Updaten) drücke, fliege ich aus dem LMO raus.

lmo3.png

Statistikspalten die ich verwende.

lmo4.png

Mit Spieler, vor dem klick auf Statistik update.

lmo2.png

Nach dem klick auf [Statistik updaten]

Wenn ich dann die Statistik wieder öffne ist sie wieder leer.

[Henshingly]

Kann ich nicht nachvollziehen.
Test LMO hier
Standardzugang
Statistik in der 1. Fussball Bundesliga 2023/24 vorbereitet
Kannst ja mal testen

Lass mal die Leerzeichen bei Deiner Formel weg
Tore+Assists-Strafen ist besser

[Scooby]

Habe bei dir das selbe Verhalten. Versuchs in der Testliga

[Henshingly]

Hab einen Spieler angelegt. Keine Probleme

[Scooby]

Hä?
Habe nur auf Statistik updaten geklickt, dann ist es gleich wieder passiert und der Spieler ist wieder gelöscht.
Nutze Chrome, am Handy und am PC, selbes verhalten.

Edit: mache später eine Bildschirmaufnahme (Video)

[Scooby]

Hier noch das Video

[DwB]

Kann ich bestätigen (mit Edge).

Es fliegen sogar alle Spieler weg, wenn ich einen neuen Spieler ergänze (was auch nicht funktioniert).

Ich gucke mal bei meinem LMO, was er da macht...

Edith sagt: in meinem LMO ist es kein Problem, jedenfalls auch mit Edge. Muss das irgendwann nochmal mit Chrome versuchen.

[Henshingly]

Auch ich konnte den Fehler reproduzieren.
Habe noch keine Lösungen.
Verschiebe den Thread deswegen in anderes Forum (LMO Fehler).

Werd ich mich drum kümmern. Wird aber kein Schnellschuss

[Henshingly]

Hab nun die erste Aussage zum Fehler
DivisionByZeroError

Code: Alles auswählen

[08-Mar-2024 13:04:43 Europe/Berlin] PHP Fatal error:  Uncaught DivisionByZeroError: Division by zero in /PFAD_ZUM_LMO/www/vest-sport.de/lmo8/addon/spieler/lmo-statadmin.php(683) : eval()'d code:1
Stack trace:
#0 /PFAD_ZUM_LMO/www/vest-sport.de/lmo8/addon/spieler/lmo-statadmin.php(683): eval()
#1 /PFAD_ZUM_LMO/www/vest-sport.de/lmo8/addon/spieler/lmo-statadmin.php(317): formel_berechnen(Array, Array, Array)
#2 /PFAD_ZUM_LMO/www/vest-sport.de/lmo8/lmo-adminmain.php(184): require('/PFAD_ZUM_LMO/ww...') // ??? Hier zeigt er mir den weiteren Pfad nicht mehr an
#3 /PFAD_ZUM_LMO/www/vest-sport.de/lmo8/lmoadmin.php(58): require('/PFAD_ZUM_LMO/ww...') // ??? Hier zeigt er mir den weiteren Pfad nicht mehr an
#4 {main}
  thrown in /PFAD_ZUM_LMO/www/vest-sport.de/lmo8/addon/spieler/lmo-statadmin.php(683) : eval()'d code on line 1

Momentan mag er es also nicht wenn der Spieler kein Spiel gespielt hat und er trotzdem Tore durch Spiele dividieren soll.
Sobald aber wenigstens ein Spiel vorhanden ist kann man weitere Spieler hinzufügen.
Mal sehen wie ich das abfange.

[Scooby]

OK, danke. Dann habe ich ja scho mal einen Workarround.
Soweit ich weiss ging das aber noch in der alten Version. Wie wurde es da denn abgefangen?

[Henshingly]

Was für eine PHP Version lief denn bisher auf Deinem Webspace?
Bis PHP5 wurde bei einer Division durch 0 ein Fehler der Stufe E_WARNING ausgegeben und false zurückgegeben. Nun gibt der Divisionsoperator, wie in IEEE 754 festgelegt, eine Gleitkommazahl entweder als +INF, -INF oder NAN zurück. Der Modulusoperator gibt nun kein E_WARNING mehr aus, sondern löst ab PHP7 stattdessen eine DivisionByZeroError-Exception aus.

Gruß

[EDIT]
Um es vielleicht mal allgemein verständlich, mit einem Beispiel aus dem Sport, zu beschreiben:
Vor PHP7 gab es bei einer Division durch Null nur die gelbe Karte und man konnte weiter spielen.
Seit PHP7 bekommt man für das gleiche "Vergehen" gleich die rote Karte und man muss vom Platz.

[DwB]

Mir fehlt dafür eine Idee, wie man die Division durch 0 abfangen soll, die Funktion formel_berechnen() wertet die übergebene Formel ja entsprechend ihrer Logik aus. Heißt die Funktion selber weiß gar nicht, was sie bekommt.

Das einzige, was ich hier nur als Lösung sehe: keine Spieler vorab erfassen, sofern sie nicht irgendeinen Statistikwert > 0 haben, der in einem Divisor verwendet wird.
Heißt in diesem konkreten Fall wenn ein Spieler neu zu erfassen ist, hat er mindestens 1 Spiel absolviert.

[Henshingly]

So geht es natürlich auch.
Ich glaube aber noch eine andere Idee zu haben...
Brauche aber dafür mal ein paar Stunden Zeit zum umsetzen.
Mal sehen wann ich die habe.

Gruß

[Scooby]

Was für eine PHP Version lief denn bisher auf Deinem Webspace?
Bis PHP5 wurde bei einer Division durch 0 ein Fehler der Stufe E_WARNING ausgegeben und false zurückgegeben. Nun gibt der Divisionsoperator, wie in IEEE 754 festgelegt, eine Gleitkommazahl entweder als +INF, -INF oder NAN zurück. Der Modulusoperator gibt nun kein E_WARNING mehr aus, sondern löst ab PHP7 stattdessen eine DivisionByZeroError-Exception aus.

Gruß

Da ich den LMO seit Corona nicht mehr verwendet hatte und vorher noch auf einer Subdomain mit PHP<7 verwendet hatte gehe ich davon aus, dass es genau darum noch funktioniert hatte.

[Henshingly]

Das Problem unter PHP 8 mit

PHP Fatal error: Uncaught DivisionByZeroError: Division by zero in PFAD_ZUM_LMO/lmo/addon/spieler/lmo-statadmin.php : eval()'d code:1

besteht natürlich weiterhin.

Meine bisherigen Vorschläge

• Warten mit der Eingabe von Spielern bis die entsprechenden Daten vorhanden sind

• Aufpassen beim Anlegen neuer Spieler, wenn IIhr Formeln in der Spielerstatistik habt.
  Nach dem Anlegen und vor dem betätigen des Button Statistik updaten Eingabefelder mit NULLEN die in einer Formel der Divisor sein werden mit 0.001 füllen.
  PHP rechnet zwar mit Kommazahlen aber nicht wie im Deutschland erwartet mit dem Zeichen >,< sondern mit einem Punkt hinter der Ganzzahl.
  Die Divisor 0.001 rechnet dann ganz normal nur wird die Ausgabe der Division auf 2 Stellen nach dem Komma begrenzt.
  Möchtet Ihr also zum Beispiel berechnen wieviele Tore ein Spieler im Durchschnit pro Spiel erziehlt. So würde es wie im unteren Bild in der Administratoransicht aussehen.
  Damit auch in der Besucheransicht unter Tore auch eine NULL auftaucht müßt Ihr noch eine Änderung in der /addon/spieler/lmo-statshow.php vornehmen.
  Suchen nach

  Code: Alles auswählen

  if (is_numeric($data[$j1][$j2])) {

  eine neue Zeile darunter einfügen mit folgenden Code

  Code: Alles auswählen

  $data[$j1][$j2] = round($data[$j1][$j2], 2);

  die 3 Zeilen sollten danach so aussehen

  Code: Alles auswählen

                  if (is_numeric($data[$j1][$j2])) {
                      $data[$j1][$j2] = round($data[$j1][$j2], 2);
                      echo "            <td " . $stat_class . " align='center'>";

  die Zahl 2 vor der runden schließenden Klammer bewirkt das die Zahl, die

  $data[$j1][$j2]

  ausgibt, sollte diese Kommastellen haben, nur 2 Kommastellen ausgibt

Screenshot 2025-02-26 130545.png

Screenshot 2025-02-26 132506.png

---

würde ich mit NULL Toren bei EINEM Spiel rechnen hätte ich wieder einen

DivisionByZeroError

damit einen Fehler und meine XXXX.stat Datei wäre zerstört.

Wie gesagt das ganze passiert ab PHP 8.