ZONE MEMBRE EN ASP "E-MEMBRES1.0"

Commentaire de Nurgle le 01/11/2007 12:46:54 administrateur CS

Pas mal du tout (ça faisait longtemps qu'on avait pas vu une source ASP3 par ici :D).
J'ai pas encore tout parcouru, mais voilà ce que je vois déjà :

Dans les points positifs, il y a une séparation plutôt claire entre l'UI et les méthodes "métier", la configuration, avec des includes serveurs, etc... et un code bien commenté :)

Après dans les points un peu moins bien :
1- le HTML ! Arg !! Je sais que ASP3 est une vieille technologie, mais ce n'est pas une raison pour utiliser de l'HTML ancestral :p ! Tes pages sont lourdes car il y a trop de balises, presque tous tes <table /> imbriqués peuvent être simplifiés avec un peu de CSS et quelques <div />.

2- Attention à l'injection SQL ! Il y a pas mal de petit endroits où tu fais des requêtes sans toujours valider ce que tu mets dedans. Exemple si dans "contact/ajout.asp" tu ajoutes un contact qui s'appelle :
' DELETE * FROM User --
Eh bien... c'est fini, tu n'as plus d'utilisateurs :p (cf "/contact/contact.asp" ; de même dans la partie Messagerie par exemple)

3- Précises bien (par exemple dans le "A lire.txt") que tu utilises le File System Object (qui par défaut n'est pas activé, ou qui est limité je sais plus...), il faut donc qu'il soit activé au niveau de IIS.
De même, tu utilises CDONTS pour envoyer des mails, il faut également qu'il soit installé et que IIS ait les droits pour l'utiliser... sinon il y en a pleins qui vont tester et se dire que c'est nul ça marche pas :D

4- Encore un truc, le "On Error Resume Next" ! (failli avoir une crise cardiaque :D)
Comme normalement tes pages fonctionnent et ne renvoient pas d'erreur, il ne sert à rien... et s'il y a bel et bien des erreurs, ça serait mieux de les corriger plutôt que les cacher (car c'est bien à ça qu'il sert en pratique :p)

Sinon au niveau des fonctionnalités c'est pas mal du tout :) mais bon, je préfère commenter le code lol

Commentaire de cmaelc le 01/11/2007 14:27:17

Merci "Nurgle" pouvez vous s'il vous plait corriger les erreurs que j'ai malheuresement comises et de me les envoyes ensuite par e-mail s'il vous plait comme cela je pourrais en faire profiter les membres, merci et cordialement. PS: mon adresse msn est mael.camerlynck@hotmail.fr.

Commentaire de mbokoaxel le 19/11/2007 09:09:24

Trop génial ta source Merci

Commentaire de tazenzaret le 22/09/2008 18:27:22

Merci beaucoup cmaelc,ça faisait longtemps que je cherchait le code asp3 comme celui la, toutes les pages ça marche sans ereurs.
merci