APPLICATION SIMPLE DE GESTION DE PRODUCTION

Commentaire de Nurgle le 04/06/2007 22:03:22 administrateur CS

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,
@++ :)

Commentaire de senior hicham le 03/08/2007 16:48:44

trashed