FORMULAIRES DYNAMIQUES
alfrai
Messages postés
69
Date d'inscription
mardi 11 avril 2006
Statut
Membre
Dernière intervention
23 juillet 2013
-
4 avril 2010 à 17:19
l0r3nz1 Messages postés 218 Date d'inscription mercredi 20 février 2008 Statut Membre Dernière intervention 17 mars 2012 - 6 avril 2010 à 10:27
l0r3nz1 Messages postés 218 Date d'inscription mercredi 20 février 2008 Statut Membre Dernière intervention 17 mars 2012 - 6 avril 2010 à 10:27
Cette discussion concerne un article du site. Pour la consulter dans son contexte d'origine, cliquez sur le lien ci-dessous.
https://codes-sources.commentcamarche.net/source/51553-formulaires-dynamiques
https://codes-sources.commentcamarche.net/source/51553-formulaires-dynamiques
6 avril 2010 à 10:27
donc il vaudrais mieux que le classe outil_mysql ai une fonction qui renvoi un tableau résultat d'une requete. c'est ce tableau qui sera utilisé dans les autres classes et fonctions pour les formulaires dynamiques.
je suis conscient de mes lacunes en sécurité, y a t'il des tutos ou d'autres conseils (j'en cherche déjà dans mon temps de libre).
merci @+.
6 avril 2010 à 10:04
Juste une chose encore :
http://localhost/PHP/validationmail.php?mail='.$this->ml->destinataire
> Ca manque de sécurité. Il aurait été bien de définir un code qu'on ne peut pas connaitre à moins de recevoir l'email. Au pire un genre de truc comme ça : .'&code'.md5($this->ml->destinataire.'salt')
6 avril 2010 à 10:00
Bonne utilisation des classes pour une partie des fichiers, le style est lisible, par contre gros problème d'indentation. Je me permet de te proposer quelques améliorations :
- Commenter le code en phpDoc
- Arrgh :) le faire en php5
- Sortir le html de la classe outil_html (exemple : corps_mail)
- Tout ce qui est mysql doit rester dans la classe mysql (ce que tu as bien fait en partie, sauf oublis de quelques mysql_fetch*)
Perso, j'aurais plus atomisé les classes de génération en définissant des champs types et la possibilité de définir les méta-données style quand tu fait le listing d'une table pouvoir cacher des champs, ou bien définir leur titre...
Bonne continuation,
akh
4 avril 2010 à 17:19
Merci pour la source. Je les test dès que possible.