Offen JTL Shop 3.19 b5: Smarty in "artikel.php" verursacht Performance Probleme - Bug?

[Nippon]

Hallo!

Aufgrund schlechter Ladezeiten haben wir nach Update auf 3.19 (3.20 folgt bald) alle unnötigen Plugins rausgeworfen, viel an unserem Template optimiert und letztendlich auch einen neuen Server bei Estugo mit SSD-Festplatten etc. eingerichtet. Dadurch sind die Ladezeiten der Startseite schon enorm vebessert worden:

- Startseite: 1,1 sek (Erstbesuch) / 700ms (re-visit mit Cache)

- Kategorien: 1 sek (Erstbesuch) / 560ms (re-visit mit Cache)
- Sonstige Inhaltsseiten (z.B. Impressum): 1 sek (Erstbesuch) / 500ms (re-visit mit Cache)

Mies waren hingegen die Produktseiten (besser als vorher, aber immer noch schlecht)



- Produktseite ohne Videos: 4,2 sek (Erstbesuch) / 3 sek (re-visit mit Cache)

- Produktseite mit Videos: 7,1 sek (Erstbesuch) / 5 sek (re-visit mit Cache)

Unser Programmierer hat den Flaschenhals gesucht und nach und nach das Template bis auf eine weiße Seite ohne HTML runtergefahren, die Probleme saßen aber tiefer. Lokalisiert wurden zwei Codestellen in der "artikel.php", deren Ausklammerung die Ladezeiten um den Faktor 4-5 reduziert. Konkret:

// Zeile 299

if(intval($Einstellungen['artikeldetails']['artikeldetails_aehnlicheartikel_anzahl']) > 0) {

	$smarty->assign("oAehnlicheArtikel_arr", holeAehnlicheArtikel($kArtikel));
}



// Zeile 320

$smarty->assign('Xselling',gibArtikelXSelling($AktuellerArtikel->kArtikel));

Ohne diese beiden Zeilen und entsprechend ohne X-Selling, Ähnliche Artikel etc. in der Artikelübersicht, beträgt die Ladezeit nur noch 600 ms - 1,1 sek auf Artikelseiten. Daher die Frage, ob da irgend etwas "falsch läuft" oder ob es unveränderbar so ist, dass es bis 4 - 7 Sekunden dauert diese Codezeilen auszuführen? Jemand eine Idee, wie ohne Deaktivierung hier ein Perfomace-Schub erreicht werden kann?

Gruß,

Nippon

 

[testjo]

AW: JTL Shop 3.19 b5: Smarty in "artikel.php" verursacht Performance Probleme - Bug?

Wieviel Artikel?

 

[Nippon]

AW: JTL Shop 3.19 b5: Smarty in "artikel.php" verursacht Performance Probleme - Bug?

Wieviel Artikel?

Aktiv im Shop sind 6.500 Artikel, aber Crosselling nur bei einer Handvoll bisher eingerichtet. "Ähnliche Produkte" / "Kunden kauften auch" gibt es natürlich.

Ich kann für einen Test gerne mal alle Artikel rauslöschen und nur 100 hochladen, ob die Scripts dann weniger blockieren. Falls die "Echtzeit"-Berechnung aber so lange dauert bei "ähnlichen Artikeln" oder "Kunden kauften auch", hätte man das ggf. besser täglich per Cronjob gemacht und in eine Tabelle eingetragen.

 

[testjo]

AW: JTL Shop 3.19 b5: Smarty in "artikel.php" verursacht Performance Probleme - Bug?

Habe gefragt weil hier mit 80 Artikel. ( UHU shop4. )
http://forum.jtl-software.de/allgem...p/80286-latenz-zeiten-shop4-2.html#post465763

Weiss aber nicht ob die dort den "ähnlichen Artikeln" oder "Kunden kauften auch", geschaltet haben.

 

[Nippon]

AW: JTL Shop 3.19 b5: Smarty in "artikel.php" verursacht Performance Probleme - Bug?

Unser Programmierer ist der beste und hat die Ursache gefunden und bei uns auch behoben! Fortan laden unsere Artikelseiten inkl. X-Selling, Kunden kauften auch / ähnliche Artikel etc. in nur 800 ms und wir haben wohl den schnellsten JTL 3 Shop im WWW 🙂

Übeltäter ist die Zeile 83 in der /includes/artikel_inc.php:

ORDER BY x.nAnzahl DESC, rand()

Unser Programmierer war zugegeben schockiert, dass in einer Select-Abfrage der native rand() Befehl genutzt wurde - in einem Original JTL-Script vom Shop 3, der ja schon etwas länger auf dem Markt ist und hier nie verbessert wurde. Wenn ich mich nicht irre ist das beim Shop 4 sogar immer noch in der artikel_inc.php...

 

[fränki]

AW: JTL Shop 3.19 b5: Smarty in "artikel.php" verursacht Performance Probleme - Bug?

Hallo Nippon,
bitte entschuldige das ich so dumm Frage (habe keine Ahnung von solchen Dingen)
Muß ich einfach in der "Zeile 83 in der /includes/artikel_inc.php: ORDER BY x.nAnzahl DESC, rand()" das grün markierte () löschen?

 

[fränki]

AW: JTL Shop 3.19 b5: Smarty in "artikel.php" verursacht Performance Probleme - Bug?

Hallo Nippon,
bitte entschuldige das ich so dumm Frage (habe keine Ahnung von solchen Dingen)
Muß ich einfach in der "Zeile 83 in der /includes/artikel_inc.php: ORDER BY x.nAnzahl DESC, rand()" das grün markierte () löschen?

Mein Shop ist der 3.20 (Build: 4)

 

[ag-websolutions.de]

AW: JTL Shop 3.19 b5: Smarty in "artikel.php" verursacht Performance Probleme - Bug?

Nein, warten bis JTL das Problem näher analysiert hat!

 

[Nippon]

AW: JTL Shop 3.19 b5: Smarty in "artikel.php" verursacht Performance Probleme - Bug?

Hallo Nippon,
bitte entschuldige das ich so dumm Frage (habe keine Ahnung von solchen Dingen)
Muß ich einfach in der "Zeile 83 in der /includes/artikel_inc.php: ORDER BY x.nAnzahl DESC, rand()" das grün markierte () löschen?

Mein Shop ist der 3.20 (Build: 4)

Nein, der komplette Aufruf musste neu geschrieben werden. Unsere Lösung funktioniert, aber die ist nicht "offiziell freigegeben" und da wir kein Servicepartner o.ä. sind werde ich das auch nicht veröffentlichen.

Wenn du zum Testen etwas löschen (vorher Backup der Datei anlegen) bzw. ausklammern willst, dann die im ersten Posting erwähnten Zeilen 299 und 320 der artikel.php. Wenn du dann (ohne Cross-Selling etc.) einen Ladezeitenvorteil hast, dann wirst du den mit einem neu geschriebenen Befehl der article_inc.php auch bekommen. Das Problem mit "ORDER BY ... rand()" ist abhängig von der Artikelanzahl (Tabellengröße) und bei unseren 6.500 Artikeln waren das ca. 3 Sekunden mehr als jetzt mit unserem eigenen Script. Und die Funktionalität ist die gleiche geblieben, Cross-Selling, Ähnliche Artikel und Co. funktionieren wie gehabt.

Es kann sein, dass es bei 500 Artikeln kaum einen Unterschied macht und bei 10.000 Artikeln noch viel mehr.

 

[FMoche]

AW: JTL Shop 3.19 b5: Smarty in "artikel.php" verursacht Performance Probleme - Bug?

Gegen rand() an sich ist jetzt ja nichts einzuwenden, aber in Verbindung mit dem Join ist das in der Tat nicht sonderlich elegant.
Eine perfomantere Alternative wäre eventuell das folgende (die IF-Schleife ab ca. Zeile 67 ersetzen):

if ($isParent) {
    $inArray = array($kArtikel);
    $tmps    = holeKinderzuVater($kArtikel);
    foreach ($tmps as $_article) {
        $inArray[] = $_article->kArtikel;
    }
    $xsell = $GLOBALS["DB"]->executeQuery("SELECT *
                                            FROM txsellkauf
                                            WHERE kArtikel IN (" . implode(', ', $inArray) . ")
                                            GROUP BY kXSellArtikel
                                            ORDER BY nAnzahl DESC, rand()
                                            LIMIT {$anzahl}", 2);


    if (count($xsell) > 0 && count($tmps) > 0) {
        $childs = array();
        $count = count($xsell);
        foreach ($tmps as $child) {
            $childs[] = $child->kArtikel;
        }


        for ($i = 0; $i < $count; $i++) {
            if (in_array($xsell[$i]->kArtikel, $childs) && in_array($xsell[$i]->kXSellArtikel, $childs)) {
                unset($xsell[$i]);
            }
        }
    }
}

 

[Nippon]

AW: JTL Shop 3.19 b5: Smarty in "artikel.php" verursacht Performance Probleme - Bug?

Sieht gut aus! Wir haben bei uns jetzt komplett auf rand() verzichtet und das etwas anders gelöst, aber die Grundidee ist soweit ich das als Code-Laie sehe die gleiche.

Danke für das offizielle Feedback! Ich frage mich nur, wieso das niemandem früher aufgefallen ist. Die Produktseiten waren ja schon immer langsam (zumindest bei tausenden Artikeln). Bisher hab ich alles auf den Server geschoben, aber der konnte auch nix dafür.

Unsere Artikelseiten haben jetzt Ladezeiten von 800 ms inkl. Crosselling etc. und das ist super 🙂