Sur le fond, l'idée de ce projet a l'air sympa, il y a juste un truc qui cloche : le code est monstrueux.
Je passerai sur le fait que cela ne soit pas du MVC, ce qui en 2013 est quand même assez hallucinant pour une appli web, mais quand en plus on trouve des trous de sécurité béants comme ça je reste pantois :
modules/temple/index.php:80
$id_chimere = $_POST['idc'];
$recherche_chimere = $bdd->query("SELECT * FROM caranille_chimeres
WHERE id_chimere='$id_chimere'");
Je veux bien que ce ne soit pas une appli pro et qu'on ne peut pas demander le même niveau d'exigence mais dans ce cas là il vaut mieux le garder pour soi plutôt que de le publier.
Le problème c'est que ce genre de code accrédite l'idée que PHP est un langage d'amateurs et que ses utilisateurs en sont.
Mon conseil, surtout si tu cherches du boulot en tant que développeur PHP, c'est de tout casser et de refaire ça propre avec un framework ; Symfony / Zend / CakePHP, t'as le choix.
La courbe d'apprentissage va être un peu rude mais au final tu vas gagner du temps et ça sera beaucoup plus clean.
# Et la qualité ?
Posté par Roger.Dudule . En réponse à la dépêche Caranille 3.5 - Éditeur de MMORPG en PHP. Évalué à 7. Dernière modification le 08 mai 2013 à 22:40.
Bonjour,
Sur le fond, l'idée de ce projet a l'air sympa, il y a juste un truc qui cloche : le code est monstrueux.
Je passerai sur le fait que cela ne soit pas du MVC, ce qui en 2013 est quand même assez hallucinant pour une appli web, mais quand en plus on trouve des trous de sécurité béants comme ça je reste pantois :
modules/temple/index.php:80
$id_chimere = $_POST['idc'];
$recherche_chimere = $bdd->query("SELECT * FROM caranille_chimeres
WHERE id_chimere='$id_chimere'");
Je veux bien que ce ne soit pas une appli pro et qu'on ne peut pas demander le même niveau d'exigence mais dans ce cas là il vaut mieux le garder pour soi plutôt que de le publier.
Le problème c'est que ce genre de code accrédite l'idée que PHP est un langage d'amateurs et que ses utilisateurs en sont.
Mon conseil, surtout si tu cherches du boulot en tant que développeur PHP, c'est de tout casser et de refaire ça propre avec un framework ; Symfony / Zend / CakePHP, t'as le choix.
La courbe d'apprentissage va être un peu rude mais au final tu vas gagner du temps et ça sera beaucoup plus clean.