Commentaires de Nurgle sur tout CodeS-SourceS

74 commentaire(s) de Nurgle sur des sources sur tout CodeS-SourceS

Le : 01/11/2007 12:48:46
Source : ZONE MEMBRE EN ASP "E-MEMBRES1.0"

Sinon pour répondre à ta question, pour passer à une autre base de données (SQL Server, MySQL, ...), Comme tu n'utilises que des instructions vraiment de base (SELECT, INSERT, ...) a peu de chose près ce n'est que la ConnectionString qui change, dans "connect.asp"... cf . http://www.connectionstrings.com/

Le : 01/11/2007 12:46:54
Source : ZONE MEMBRE EN ASP "E-MEMBRES1.0"

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

Le : 04/06/2007 22:03:22
Source : APPLICATION SIMPLE DE GESTION DE PRODUCTION

Salut,
J'ai pas parcouru tout le code en détail, mais il y a cependant quelques points... :
- Dans ta fonction "message" présente dans différentes pages, tu utilises un Response.Write pour renvoyer un script javascript. Le Response.Write est plus ou moins dépréciée en ASP.NET, essaye plutôt, soit d'ajouter à ta page un Literal, et tu remplis sa propriété Text avec ton script, soit d'utiliser la méthode (en ASP.NET 1.x) prévue pour ça :
If Not Page.IsClientScriptBlockRegistered("clientScript")) Then
Page.RegisterClientScriptBlock("clientScript", "<script ... > ... </script>")
End If
- Un truc frappant aussi : tu fais pleins de PostBacks pour rien ! Tu as des LinkButton, où tu t'abonnes à l'évènement click uniquement pour faire un Response.Redirect() (par exemple dans AjoutEmploye.aspx et Mains.aspx)! N'oublie pas que le web c'est de l'HTML, donc un lien normal suffit : <a href="/mapage.aspx">Mon Lien</a>
- Même chose pour ton UserControl qui ne contient qu'un ImageButton qui ne fait qu'un Response.Redirect()... ! Un <a href="mapage.aspx"><img src="/monimage.gif" style="border: none;" /></a> fait la même chose, mais sans nécessiter un UserControl séparé et sans PostBack sur la page elle-même.
- Sinon, à quelques endroits tu fais des SELECT en concaténant directement des chaines de caractères pour passer les paramètres... Il faut utiliser des requêtes paramétrées, sinon attention aux SQL Injections !
- Tu pourrais également étoffer un peu la description du code, qui est... pas très compréhensible ;)
Et aussi enlever du ZIP les dossiers "_vti_cnf" et "bin", ce ne sont que des dossiers temporaires qui ne sont pas nécessaire pour faire tourner le site...

Bonne journée,
@++ :)

Le : 05/05/2007 15:59:49
Source : [VS2005] VISUALISEURS PERSONNALISÉS

Pas mal du tout, les Visualizers sont très pratiques en effet :)
Indiquons tout de même qu'il faut ajouter la référence à Microsoft.VisualStudio.DebuggerVisualizers.dll pour avoir accès aux classes nécessaires.
(la dll est dans "Program Files\Microsoft Visual Studio 8\Common7\IDE\PublicAssemblies")

Tu devrais aussi enlever du ZIP les répertoires /bin et /obj de chaque projet, ils l'alourdissent pour rien ;)

Le : 02/04/2007 22:22:08
Source : [C#2] MESSAGE BOX D'ERREUR AVEC DÉTAILS

Saphir54321 > as-tu ajouté la référence à l'assembly System.Windows.Forms ? (clique droit sur le projet, "Add Reference", puis dans la liste ajoute System.Windows.Forms) car par défaut il n'est pas référencé dans certains projets (Class Library par exemple)

Bonne soirée :)

Le : 31/03/2007 12:56:03
Source : CONVERSION D'UNE DATE/HEURE STANDARD VERS DATE/HEURE CONFORME À LA NORME RFC822

Salut,
effectivement je ne crois pas me souvenir de fonction ASP/VBScript qui fasse exactement ça, mais est ce que une date du genre FormatDateTime(Date(),1) (qui renverrai un truc du genre "Thursday, February 22, 2007") ne fonctionnerais pas également (dans un RSS) ??

Sinon, pas grand chose à redire, sauf peut être... :
met le If ... Then Exit Function avant les Dim, histoire de ne pas déclarer des tableaux ou d'autres variables qui ne serviront à rien si on rentre dans ton If.
Quant aux Erase, j'utiliserais plutôt des Set ... = Nothing, mais bon :p

Le : 10/01/2007 18:57:09
Source : BANNIR DES IP AVEC GLOBAL.ASA

Il n'y a rien à télécharger :
Tu crées à la racine de ton site un fichier global.asa, tu sélectionnes les 10 lignes du code, plus haut, et tu les mets dans le fichier, et c'est tout :-)

A++

Le : 05/01/2007 17:35:34
Source : MINI MESSAGERIE

Astuce_Jeux > effectivement c'est de l'ASP. La catégorie est fausse, c'est corrigé :-)

Quant à ton erreur, c'est normal : pour faire tourner des pages .asp, tu as besoin de IIS, et pas de Apache !!

Le : 13/11/2006 16:53:40
Source : THREAD ET PROGRESSBAR - EXEMPLE SIMPLE

TMONOD > j'avais déjà postée une source là dessus : http://www.vbfrance.com/code.aspx?ID=38076
(d'accord, c'est du VB... mais bon :p)

Le : 12/11/2006 22:42:06
Source : THREAD ET PROGRESSBAR - EXEMPLE SIMPLE

jolie petite source,
Au moins ça apprend à manipuler correctement les threads :)

Personnellement, j'aurais utilisé un BackgroundWorker, et hop, un petit .ReportProgress avec en paramètre le pourcentage d'avancement de la progressBar et c'était fini. Mais bon, c'est la solution pour flemmard :o)
(et c'est plus limité, c'est vrai...)

par contre, pas de classes partielles... bahhh, ça me rappelles de mauvais souvenirs (.NET 1.x par exemple... :p)