keran - 2020-02-02 03:25:50
Salut / Hi
Quelques améliorations seraient bienvenues avant de concourir car en l'état, bien qu'intéressante, cette classe n'est pas utilisable même avec php 5.
Les shorts tags, ce n'est plus possible pour être portable.
split est obsolète est ce remplace parfaitement par explode même en version 5. Et c'est portable.
Le code est bourré de notice et warning... E_ALL aide à les éliminer.
Pourquoi ne pas écrire plusieurs constructeurs avec des paramètres différents plutôt que ces méthodes statiques?
La variable debug gagnerait à être privé et en get/set. Comme les protected.
Et çà, c'est faux car la méthode n'existe pas : Log::decode($imageSize)
Testé avec 5.6 et 7.3.
Some improvements would be welcome before competing because as it stands, although interesting, this class is not usable even with php 5.
Tag shorts are no longer possible to be portable.
split is obsolete and is perfectly replaced by explode even in version 5. And it's portable.
The code is full of instructions and warning ... E_ALL helps to eliminate them.
Why not write several constructors with different parameters rather than these static methods?
The debug variable would benefit from being private and in get / set. Like the protected.
And this is false because the method does not exist: Log :: decode ($ imageSize)
Tested with 5.6 and 7.3.
Chouchen - 2020-02-03 16:57:47 - In reply to 
