Demoshop geht nicht

[GTB]

Wie du in dem Test siehst, gibt die neue Funktion korrekterweise immer 0 zurück, wenn man es erwartet, was die alte Funktion aber nicht macht. Wenn wir das nun ändern, dann kann es zu unerwünschten Effekten kommen.

Bevor das nicht ausgiebig getestet ist, wird es nicht in die Version einfliessen.

Gruss Gerhard

[jonnyB]

Ich meine nicht die !empty() sondern h-h-h's hs_not_null()

Also ich seh da jetzt keinen Unterschied in der Ausgabe gegenüber der alten Funktion oder es ist noch zu früh für meine Augen

[noRiddle (revilonetz)]

@GTB
Sowohl die bieherige Funktion xtc_not_null() als auch h-h-hs Funktion hs_not_null() gibt nicht 0 zurück sondern einen Boolean, also true oder false, oder wenn man will, 0 oder 1.

Code: PHP  [Auswählen]

strlen(trim($value)) > 0

jedenfalls, ist ein überflüssiger Schritt zuviel, es reicht h-h-hs

Code: PHP  [Auswählen]

trim($value) != ""

und man kann deshalb an h-h-hs Testcase auch ohne strlen() eine leichte Performance-Verbesserung sehen.

Außerdem hat h-h-h nachgewiesen, daß

Code: PHP  [Auswählen]

is_array($value) ? !empty($value)

deutlich schneller ist als

Code: PHP  [Auswählen]

if (is_array($value)) {
  if (sizeof($value) > 0) {
    return true;
  } else {
    return false;
  }

Das sieht man an den Performance-Verbesserungen wenn $value ein Array ist.

Außerdem kann man rückschließen, daß an vielen Stellen wo xtc_not_null() bislang benutzt wird !empty(),
also nicht hs_not_null(),
deutlich schneller wäre,
natürlich nur da benutzbar, wo es auch möglich ist.
Wo es möglich ist erkennt man im Testcase an den Stellen wo xtc_not_null() und !empty() den gleichen Boolean ausgeben.

Da man kaum das ganze System durchforsten wollen wird um an den möglichen Stellen xtc_not_null() mit !empty() zu ersetzen ist h-h-hs Funktion eine gute Lösung.
Alle Boolean-Ausgaben sind im Testcase bei allen möglichen Werten für $value identisch.
h-h-hs Funktion könnte nach meiner Meinung problemlos übernommen werden.

Wo ist nach deiner Meinung der Haken ?

Gruß,
noRiddle

[h-h-h]

Als ich habe bewusst mit % Angaben gearbeitet, da die Differenz in Sekunden mind. 4 Stellen nach dem Komma ist. Es ging mir bei dem Test hauptsächlich darum, uns bewusst zu machen xtc_not_null wirklich - nur - zu verwenden, wenn !empty und co nicht ausreichen.

Viele Grüße,
h-h-h

[h-h-h]

Hi jonnyB,
entschuldige bitte, dass ich auf deinen Beitrag nicht eingegangen bin.

@ Ronny u. h-h-h:
um Perfomance gings mir nicht, sondern um die Lösung des Memory-Fehlers.

Wie du schon erwähnt hast, liegt dieses Problem an PHP 5.4, genau konnten wir die Problematik in der PHP-Version bisher nicht identifizieren, siehe dazu im Bugtracker das Ticket 106.

@ h-h-h:
deine neue bzw. überarbeitete Funktion finde ich sehr gut, gerade wegen dem schlanken Code.

Was ich jedoch nicht verstehe, ist daß deine Funktion nicht den Weg in den Shop gefunden hat, da sie diesen Memory-Fehler auch behebt und zusätzlich auch performanter arbeitet.

Sprachen irgendwelche Gründe aus sicht der restlichen Entwickler dagegen?

Hatte die Seite erst letzte Woche geschrieben, daher alles noch recht frisch.

Wir werden sehen, da ist bestimmt noch Optimierungspotenzial.

Viele Grüße,
h-h-h

[Dato]

Hallo,
was ist mit dem DemoShop der 2er version los ?

Da kommt bei mir im firefox 26 nur eine Weisse Seite ohne Meldung.

https://demo.modified-shop.org/trunk/

[noRiddle (revilonetz)]

Guckssu hier https://demo.modified-shop.org/tags/modified-1.0x/.
Wurde von GTB hier gepostet (gebe zu, findet nicht jeder).

Gruß,
noRiddle

[Dato]

Danke,
ich bin immer von der Demo-Hauptseite ausgegangen.

https://www.modified-shop.org/demoshops

und da läuft der Link ins lehre
modified eCommerce Shopsoftware SVN Trunk Version