begin process at 2012 02 11 12:01:48
  Trouver un code source :
 
dans
 

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

Déposé sur Gestion d'une bibliothèque

Salut,

Source intéressante, code bien commentée, etc. et puis ça fait du bien de revoir de l'ASP3 après toutes ces années ! :p
Quelques remarques quand même :
- Dans certains fichiers, "/emprunt.asp" par exemple, tu as du code VBScript : Il est important de préciser que ceux-ci ne fonctionneront que sous Internet Explorer. Du javascript aurait été mieux choisi, mais si tu n'utilises l'appli web qu'en intranet avec IE partout, pourquoi pas...
- Pourquoi des morceaux de flash partout là où un usage avisé des CSS aurait tout à fait fait l'affaire ? (surtout juste pour des boutons... une simple image suffit)
- Toutes les images du genre "/images/archiv_enseign.gif" ont toute le même fond : pourquoi pas une seule image, et une ligne de texte "normale" correctement placée dessus ?
- Arrgg, un "frameset" dans "/index.asp" ! Et en plus juste pour afficher un menu à gauche...

En résumé, le code ASP est très clean et bien commenté, mais l'HTML lui-même aurait bien besoin d'une remise à jour. Même toute l'interface en général. D'ailleurs je pense que tu pourrais diviser par 4 ou plus le poids du site (et la bande passante consommée) rien qu'en re-designant l'interface...

J'espère ne pas t'assommer sous les critiques, j'essaye juste de t'encourager à améliorer encore ta source.
Bonne continuation,
Nurgle (Antoine)
Posté le : 30/01/2010 00:01:36

Déposé sur Captcha asp.net et javascript

L'idée est bonne en effet, attention cependant à bien disposer tes objets graphiques à la fin de ta fonction captcha() :
gr.Dispose()
hatchBrush.Dispose()
b.Dispose()
(ou alors utilise un bloc "using")

Sinon, le <%= cap %> est à éviter, l'ASP3 c'est fini depuis un moment... mais bon. Utilise par exemple Page.ClientScript.RegisterStarterScript pour ajouter le contenu de cap.

Enfin, un autre "petit" problème : ton site est pensé pour un seul utilisateur en simultané ! (ce qui n'est pas vraiment le cas d'un site web)
Si tu as assez de requêtes par secondes, ton image 'images/capcha.jpg" sera modifiée en permanence, ce qui risque de provoquer des conflits ! Des pages ne pourront pas la modifier par ce qu'elle est ouverte autre part, et d'autres afficheront la mauvaise image car entre le moment où la page l'a créée et celui où elle s'affiche, l'image à déjà changé, et l'utilisateur voit la mauvaise image !! (Sans compter, dernier détail, la mise en cache de l'image par le navigateur, comportement par défaut qu'il faut lui dire de ne pas faire)
Bref, il faut revoir juste cette partie là : passer par exemple par un HttpHandler, avec un peu de boulot à faire derrière (je suis exprès vague, c'est en essayant qu'on apprend :p)

Bon code,
Nurgle
Posté le : 27/09/2009 16:30:15

Déposé sur Récupération d'infos de film

Bonsoir,
une petite précision : les EXE sont retirés automatiquement des ZIPs, pour lutter contre les virus et co.

Bonne soirée
Posté le : 01/11/2008 21:15:48

Déposé sur Authentification d'utilisateurs avec base de données

Avec un include server : <!-- #include virtual="/comtest.asp" -->
Si de plus tu veux interdire ta page aux visiteurs anonymes, rajoute (juste en dessous de l'include) le code
<%
if Session("USER_login")="" then
Response.Redirect("/logon.asp?go="& Request.ServerVariables("SCRIPT_NAME"))
end if
%>
comme indiqué plus haut :)
Posté le : 10/07/2008 16:37:16

Déposé sur 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/

Posté le : 01/11/2007 12:48:46

Déposé sur 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
Posté le : 01/11/2007 12:46:54

Déposé sur 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,
@++ :)
Posté le : 04/06/2007 22:03:22

Déposé sur [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 ;)
Posté le : 05/05/2007 15:59:49

Déposé sur [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 :)
Posté le : 02/04/2007 22:22:08

Déposé sur Conversion d'une date/heure standard vers date/heure conforme...

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
Posté le : 31/03/2007 12:56:03

1 2 3 4 5 6 7 8


Nos sponsors


Sondage...

Comparez les prix

CalendriCode

Février 2012
LMMJVSD
  12345
6789101112
13141516171819
20212223242526
272829    

Consulter la suite du CalendriCode

 
Développement réalisé par Nicolas SOREL (Nix) avec l'aide de : Cyril DURAND et Emmanuel (EBArtSoft), Merci à Vincent pour ses précieux conseils.
CodeS-SourceS.com© Toute reproduction même partielle est interdite sauf accord écrit du Webmaster
CodeS-SourceS.com© est une marque déposée tous droits réservés

Google Coop CodeS-SourceS Google Coop CodeS-SourceS
Temps d'éxécution de la page : 2,075 sec (3)

Nous contacter | Annoncer sur CodeS-SourceS | Mentions légales