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

Nippon

Gut bekanntes Mitglied
1. Juni 2009
741
8
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:

Code:
// 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
 

Nippon

Gut bekanntes Mitglied
1. Juni 2009
741
8
AW: JTL Shop 3.19 b5: Smarty in "artikel.php" verursacht Performance Probleme - Bug?

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.
 

Nippon

Gut bekanntes Mitglied
1. Juni 2009
741
8
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:

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

Aktives Mitglied
28. Januar 2014
28
0
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

Aktives Mitglied
28. Januar 2014
28
0
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)
 

Nippon

Gut bekanntes Mitglied
1. Juni 2009
741
8
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

Moderator
Mitarbeiter
15. Dezember 2014
941
154
Halle (Saale)
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):
Code:
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

Gut bekanntes Mitglied
1. Juni 2009
741
8
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 :)
 
Ähnliche Themen
Titel Forum Antworten Datum
JTL-Shop 5.0.1 mit Wawi 1.5.44.0 verbinden - Shop URL verweist nicht auf gültigen Shop Einrichtung JTL-Shop5 0
Spezialseite "Kontakt" nach Upgrade von JTL Shop 4 auf 5 Einrichtung JTL-Shop5 0
Neu [JTL SHOP 5] Sitemap kann nicht automatisch erstellt werden JTL-Shop - Fehler und Bugs 2
Neu Internet Explorer + JTL Shop 5 = Bestellabschluss nicht möglich JTL-Shop - Fehler und Bugs 0
Neu JTL Software + Shop + Ebay/Amazon lohnt sich das? Starten mit JTL: Projektabwicklung & Migration 3
Neu JTL SHOP 5 NOVA - keine Preisspanne, kein ab Technische Fragen zu Plugins und Templates 0
Neu Upgrade auf ebay-Shop / Verhalten in JTL? eBay-Anbindung - Ideen, Lob und Kritik 1
Neu Automatische Abmeldung aus der JTL Shop Administration verhindern Allgemeine Fragen zu JTL-Shop 2
Neu [JTL-Shop 5] Mediendateien anzeigen unter Artikelbeschreibung - 2 Column Layout bei zwei Mediendatein? JTL-Shop - Ideen, Lob und Kritik 0
Neu [JTL-Shop 5] OPC Portlets auch in Tabs ermöglichen, die in der Wawi als Attribute angelegt sind (tab1 name, tab2 name, etc) JTL-Shop - Ideen, Lob und Kritik 0
Neu Ladezeit Shop - Plesk --> JTL 5 (STRATO) JTL-Shop - Fehler und Bugs 5
Neu PayPal Ratenzahlung Anbindung an JTL-Shop 4 User helfen Usern - Fragen zu JTL-Wawi 1
Neu Transferring the customer account credit from shop to JTL WAWI User helfen Usern - Fragen zu JTL-Wawi 5
Neu JTL Shop 5 - Consentmanager.de Technische Fragen zu Plugins und Templates 1
Neu Passwort Sicherheit im JTL Shop User helfen Usern - Fragen zu JTL-Wawi 3
Neu PayPal Plus wird GAR nicht anzeigt - JTL Shop 4 JTL-Shop - Fehler und Bugs 0
Neu JTL Shop 5.0.1 Nova-Template Slick Slider Bildgröße fehlerhaft JTL-Shop - Fehler und Bugs 1
Neu JTL-Shop 5.0 - Release 5.0.2 Upgrade JTL-Shop4 auf JTL-Shop5 1
Kostenlose Einsteigerschulung: Tag 7 Teil 1: Einführung in JTL-Shop 5 mit OPC Messen, Stammtische und interessante Events 0
Neu JTL Shop 5 Artikelanzeigefilter Betrieb / Pflege von JTL-Shop 0
Neu JTL Shop 5 OPC Akkordeon: Reihenfolge der Gruppen JTL-Shop - Ideen, Lob und Kritik 1
Neu JTL Shop 5 Unterkategorien in der Navileiste Allgemeine Fragen zu JTL-Shop 4
Neu JTL Shop 4.6.17 Fatal Error bei Artikelexport JTL-Shop - Fehler und Bugs 2
Neu Problem: Wie die Versandarten aus dem JTL Shop mit den WaWi Versandarten verknüpfen? User helfen Usern - Fragen zu JTL-Wawi 2
Neu JTL Shop 5 PayPal Express mit aktiviertem Cache JTL-Shop - Fehler und Bugs 1
Neu Freelancer für JTL Shop 5 Template Anpassungen gesucht Dienstleistung, Jobs und Ähnliches 0
Neu Plugin Hinweis Fenster JTL Shop 4 Allgemeine Fragen zu JTL-Shop 1
Neu Linkfarbe im Header und Footer ändern - JTL Shop 5? Allgemeine Fragen zu JTL-Shop 7
Neu JTL Shop 5 NOVA: Child-Datei für Preisanzeige ohne Rabatt gesucht Dienstleistung, Jobs und Ähnliches 0
Neu JTL Shop 5 Warnungen in der Console JTL-Shop - Fehler und Bugs 2
Neu JTL Shop 5 NOVA: Anmeldebutton soll gegen anderes Icon getauscht werden Templates für JTL-Shop 1
Neu JTL Shop 4 - Versandart Abholung nur Barzahlung -> Kunde kann trotzdem mit PayPal zahlen JTL-Shop - Fehler und Bugs 2
Neu JTL Shop 5 Theme Editor Nova Template Farbe Buttons Templates für JTL-Shop 5
Zustimmung der Weitergabe der eMail an Versanddienstleister im JTL Shop 5.0 Einrichtung JTL-Shop5 4
Neu Artikel im JTL-Shop nicht sichtbar... User helfen Usern - Fragen zu JTL-Wawi 3
Neu JTL Shop 5 NOVA Footer Textfarbe Templates für JTL-Shop 3
Neu JTL Shop 5 PayPal Problem JTL-Shop - Fehler und Bugs 0
JTL Shop 5 unter "localhost" Fehler beim Setzen des Cookies Einrichtung JTL-Shop5 0
Neu Umzug JTL shop 5 auf anderen Server, gleiche Domain Installation / Updates von JTL-Shop 9
Neu JTL-Shop 5 Product Stream müsste verbessert werden JTL-Shop - Ideen, Lob und Kritik 1
Neu Jtl Shop Filter Allgemeine Fragen zu JTL-Shop 1
Neue Standard Schriftart dem JTL Shop 5 hinzufügen Einrichtung JTL-Shop5 1
Neu Export Staffelmenge und Staffelpreise aus dem JTL Shop 4 Allgemeine Fragen zu JTL-Shop 0
Neu Wie kann ich <meta name="google-site-verification" content="" /> in Head Bereich einfügen? JTL Shop Version 4.6 Allgemeine Fragen zu JTL-Shop 2
Neu JTL Shop "Auf Lager" Sticker ausblenden Allgemeine Fragen zu JTL-Shop 1
Neu Wawi Datenabgleich mit JTL Shop 5.0.1 nicht möglich - Fehler 8 JTL-Wawi - Fehler und Bugs 0
Neu Jtl Shop Kategorie Seiten Top Angebote/ Unsere Bestseller ausschalten Allgemeine Fragen zu JTL-Shop 1
Neu JTL Shop 5 NOVA-Template Kategorien als Menüpunkt Technische Fragen zu Plugins und Templates 6
Neu Artikelbeschreibung in JTL Shop vereinheitlichen User helfen Usern - Fragen zu JTL-Wawi 5
Kundenimport von JTL Wawi zu JTL Shop 5 Einrichtung JTL-Shop5 0

Ähnliche Themen