APPLICATION SIMPLE DE GESTION DE PRODUCTION
cs_Nurgle
Messages postés
1642
Date d'inscription
samedi 6 novembre 2004
Statut
Membre
Dernière intervention
28 avril 2011
-
4 juin 2007 à 22:03
cs_keke_ben Messages postés 1 Date d'inscription samedi 3 mars 2007 Statut Membre Dernière intervention 19 novembre 2009 - 19 nov. 2009 à 18:29
cs_keke_ben Messages postés 1 Date d'inscription samedi 3 mars 2007 Statut Membre Dernière intervention 19 novembre 2009 - 19 nov. 2009 à 18:29
Cette discussion concerne un article du site. Pour la consulter dans son contexte d'origine, cliquez sur le lien ci-dessous.
https://codes-sources.commentcamarche.net/source/42983-application-simple-de-gestion-de-production
https://codes-sources.commentcamarche.net/source/42983-application-simple-de-gestion-de-production
19 nov. 2009 à 18:29
8 avril 2008 à 04:19
j'aimerais bien si tu as la solution de cette variante partie VB net.
allah ihafdak khouya.
merci
3 août 2007 à 16:48
5 juin 2007 à 10:33
mêmes remarques que Nurgle concernant l'utilisation "abusée" des postback !!! ..
le code est mal commenté aussi :( ce qui le rend illisible meme pour une personne initiée
bonne chance pour les prochaines sources ;)
4 juin 2007 à 22:03
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 : Mon Lien
- Même chose pour ton UserControl qui ne contient qu'un ImageButton qui ne fait qu'un Response.Redirect()... ! Un [mapage.aspx ] 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,
@++ :)