Lorsque l'on dépose une source sur un site comme celui-ci, c'est pour la soumettre aux avis de la communauté. Comment les membres pourront-ils juger de la qualité de ton code si il est "offusqué" ?? En l'état, cette source devrait être supprimée.

Renseigne toi sur la minification, tu verras que tu es dans le faux. Elle n'a d'importance que client-side.

Il n'y a aucun intérêt à minifier du code PHP.

Un peu complexe.

L'algo d'instanciation de la Factory est trop lourd : utilisation de eval(), func_get_args(). Dans ce cas, l'utilisation de la réflexion est tout à fait adaptée.

Pourquoi utiliser $class et $function dans l'exception alors que l'erreur est identifiable par l'exception qui l'a créé ? Si tu veux pousser la localisation de l'erreur tu peux t'aider d'Exception::getTrace().

Il n'y a pas d'erreurs mais des problèmes de conception et "d'optimisation".

Je rejoins le commentaire de genetAPT152 (pas très simple ton pseudo).

J'irai même plus loin en supprimant tous les tests "trim(!empty($new))" puisque dans tous les cas la connexion va échouer. C'est une vérification qui doit se faire en amont.

L'utilisation des exceptions est primordiale dans ta classe puisqu'il y a des "evenements" critiques (comme le statut de la connexion). Actuellement, le développeur n'a aucun retour mais surtout aucun message pour l'avertir. De plus comme l'a dit "genet...", les méthodes get() , put() ne doivent plus s'exécuter si la connexion a échoué.

Même si elles ne sont pas standardisées, tu respectes les conventions zend (plutôt bien pensées) et le code est propre.

Fixer son environnement de développement est une chose primordiale :

error_reporting(E_ALL|E_STRICT);
ini_set('display_errors', 1);

Les erreurs s'afficheront.

Oui, actuellement la gestion des logs se résume à sa forme la plus basique : ouverture, écriture, fermeture (une MAJ est d'ailleurs prévue pour cette partie).

Ton principe de multi-logging et de sources de données (bdd, fichier) seront intégrés dans les prochaines versions.

Concernant les MAJ il y aura une console de debug, un système de breakpoints et un benchmark.

Je tiens également à rappeler que cette source s'adresse aux débutants et/ou aux personnes n'ayant pas la possibilité d'installer des extensions comme xdebug sur leurs serveurs. Si vous avez des questions, n'hésitez surtout pas.

Merci pour tes suggestions.

A mon sens (bien qu'humain avant tout), un administrateur doit être patient et à l'écoute des utilisateurs.

Je ne remets pas en cause le travail de Reinfield sur codes-sources mais simplement son manque de tact.

Admin ou pas, je doute que tes sarcasmes fassent avancer les choses.

Bonjour,

Sauf erreur de ma part, ce code plante si un prologue XML est présent en entrée.

C'est bien de changer.
Continue à déposer ce genre de sources.