1 ------------------------------------------------
Préfixer les variable de config, ou les mettre dans un tableau. Quand
on lit le code, c'est par exemple pas très evident que $bypass_act est
défini dans le fichier de config.
2 ------------------------------------------------
N'afficher le lien de déconnection que si on est connecté....
3 ------------------------------------------------
Le code if($_SERVER['HTTP_HOST']=='127.0.0.1'){
est à remplacer par :
if(($_SERVER['HTTP_HOST']=='127.0.0.1') ||
($_SERVER['HTTP_HOST']=='localhost')){
4 ------------------------------------------------
Ca serait pas mal de regrouper les fonctions de connection à la base
dans une classe, Je ne développerai pas plus, je pense que tu comprend
pourquoi.
5 ------------------------------------------------
Personnelement j'évite toujours les history.back(), donc ca serait
souhaitable de mettre une option dans la config pour spécifier une URL
de retour, si on aime pas cette fonction JS
6 ------------------------------------------------
Lorsque La création du compte echoue, ne pas afficher le mess d'erreur
et le lien pour revenir en arrière, mais l'ancien formulaire + le
message d'erreur, ca fait une page de moins à télécharger pour l'utilisateur.
7 ------------------------------------------------
Peut-être ajouter une option pour afficher ces erreurs en JS (alert)
8 ------------------------------------------------
Commenter les champs de la base dans ta doc. Et fait une doc des
fonction genre phpdocumentor, ca fait jamais de mal :p
9 ------------------------------------------------
En faisant un retour dans l'historique, j'ai réussi à créer 2
fois le meme compte :
-> Je m'inscri
-> j'ai le message qui me dit que j'ai 2heures pour activer mon compte
-> je refresh, hop 2 comptes avec le meme pseudo dans la base !
tu devrais aussi rendre le champ login unique
ALTER TABLE `membres` ADD UNIQUE `login` ( `login` )
10 -----------------------------------------------
Dans la génération d'image, on ne distingue pas le Ø du O
11 -----------------------------------------------
Faire une vérif des formulaires de saisie en JS, ca evite de poster
pour rien
12 -----------------------------------------------
Peut-être prévoir utérieurement une classe ou des fonctions pour
d'autres bases, tous le monde n'est pas sous MySql.
++
Partager