Dienstag, 29. Oktober 2019, 09:56
php-iban und Laden der Daten ("Registry") bei Bedarf
Da wir die Bibliothek php-iban gerne nutzen, haben ich gerade einen Pull-Request erstellt:
https://github.com/globalcitizen/php-iban/pull/92/
Das Laden der Daten ("Registry") erfolgt in den bisherigen Versionen immer, auch wenn keine php-iban-Funktion genutzt wird. Gerade in Verbindung mit Composer unnötig.
Mit dem Patch werden die Daten erstmal geladen, wenn sie gebraucht werden. Mit ganzen 3 Zeilen Code.
https://github.com/globalcitizen/php-iban/pull/92/
Das Laden der Daten ("Registry") erfolgt in den bisherigen Versionen immer, auch wenn keine php-iban-Funktion genutzt wird. Gerade in Verbindung mit Composer unnötig.
Mit dem Patch werden die Daten erstmal geladen, wenn sie gebraucht werden. Mit ganzen 3 Zeilen Code.
Kommentare
Ansicht der Kommentare: Linear | Verschachtelt
Rene
Sven
Die Bibliothek an sich erscheint mir aber fragwürdig. Das erzwungene Immerladen via Composer ist vor allem dem Funktionsinterface zuzurechnen, objektorientiert wäre vieles einfacher.
Der Scope mit PHP 5.2 bis 7.0 ist outdated. Tests finden ohne PHPUnit statt... angesichts des Versionsbereichs aber vermutlich kein Wunder.
Die stärkste Performanceverbesserung würde man vermutlich bekommen, wenn man die registry.txt in ein PHP-Array wandeln und als PHP-Sourcecode ablegen würde, anstatt die Datei zu parsen. Der Code liegt dann nämlich im Opcache.
Manni
Du schraubst selbst an PHP-Bibliotheken rum und veröffentlichst einen Patch, der ein nicht existentes Performance-Problem löst?
Auf welcher Hardware laufen denn eure Systeme, dass das überhaupt relevant sein kann?
Stefan G.
Auf vorhandener und bezahlter?
Als jemand, der für Traffic bezahlt, würde ich da übrigens durchaus einen Performance-Gewinn sehen...
Manni
Von welchen Traffickosten redest du?
Und du bist stolz (???), dass die scheinbar ältere Hardware für eine Kiste, die PHP-Anwendungen betreibt, bezahlt und nicht geleast ist?
Stefan G.
Finde den Widerspruch...
"bezahlt und nicht geleast"
Ich sehe Manuel nicht als Jemanden, der essentielle Betriebsmittel least...
Und selbst wenn, sollte man "sparen", wo es geht. Denn ob es Sinn gemacht hätte, weiss man immer erst, wenn es zu spät ist...
Bernhard
Siehe php-iban.php Zeile 479
Jens
Zu Mannis Neidtirade: Es soll tatsächlich Menschen geben, die leidenschaftlich in dem sind, was sie tun. Da ist es ziemlich egal, was für ein Titel auf dem Schild des Büros steht. Bonus: Als Inhaber des Unternehmens kann man sich den Titel sogar selbst aussuchen - heißt einem selbst gegenüber ist „er“ völlig egal - wurde einem schliesslich nicht verliehen und dient nur für die Kommunikation nach außen.