begin process at 2012 05 27 08:50:54
  Trouver un code source :
 
dans
 

52 commentaire(s) de Nurgle sur des sources sur aspfr

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 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 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

Déposé sur 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++
Posté le : 10/01/2007 18:57:09

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

Posté le : 05/01/2007 17:35:34

Déposé sur Calcul de l'age d'une personne (asp vbscript)

Salut,

Avec pourquoi pas aussi au tout début de la fonction un :
If Not IsDate(dat) Then Return 0
pour éviter les problèmes... :-)

Jette un coup d'oeil aux fonctions sur les Dates, comme dit Tikrimi, autant les utiliser :
http://www.w3schools.com/vbscript/vbscript_ref_functions.asp#date

A++,
Nurgle
Posté le : 12/10/2006 22:49:37

1 2 3 4 5 6


Nos sponsors


Sondage...

Comparez les prix

CalendriCode

Mai 2012
LMMJVSD
 123456
78910111213
14151617181920
21222324252627
28293031   

Consulter la suite du CalendriCode

A découvrir



 
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 : 0,094 sec (4)

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