Bonjour, Je veux faire un planning comme ceci : A chaque changement de semaine (clique sur l'un des boutons "semaine précédente" et "semaine suivante"), il faut mettre à jour les combobox contenues dans une groupbox, elle-meme contenue dans un TabPage. L'application est reliée à une base de données de type SqlServer qui contient les horaires et les personnes (guibert,dupont etc..) et les activités (fitness,séance kiné etc).. Comment faire ??

C# / .NET : Comment mettre à jour un onglet (TabPage) ? [Résolu]

Réponse 1 / 7

fcaruso Messages postés 18 Date d'inscription vendredi 15 avril 2005 Statut Membre Dernière intervention 10 février 2009
3 févr. 2009 à 23:32

Pour répondre a ta question :
Si tu veux faire un truc rapide, tu peux travailler en direct sur les DataTable ou un Reader, MS n'a pas inventé ca pour rien ;-)
Si tu veux faire un truc propre, lisible, maintenable et evolutif, sincerement : Oui. On parle souvent de la DAL ( data access layer ) et de la BLL ( business access layer ).

Là tout dépends de ton objectif, je n'ai fait que te dire ce que je ferais moi, apres tu en fais ce que tu veux...

Tu va surtout gagner en lisibilité dans un 1er temps : Manipuler des objets et des collections "Metier" est tellement plus evident et clair quand on en a fait l'experience.

En fait l'idée de base à retenir c'est juste de bien séparer chaque chose :
- La création des combo à un endroit ( dans la Form ou un UserControl )
- Le chargement à un autre endroit (que tu pourra reutiliser sans avoir a recreer tes combo, si tu change de semaine)
- Si possible se faciliter la representation finale ( class Activite ) par quelquechose de tres lisible ( des objets metiers ) et tres manipulables ( un DataReader ne va que dans un sens par exemple (while Read()) et on ne peut rechercher simplement dedans sans relancer une requete SQL... alors que dans une List... )

Bref
A+
Fab

Réponse 2 / 7

cs_vinok Messages postés 36 Date d'inscription jeudi 12 octobre 2006 Statut Membre Dernière intervention 29 mars 2013
2 févr. 2009 à 18:53

Réponse 3 / 7

fcaruso Messages postés 18 Date d'inscription vendredi 15 avril 2005 Statut Membre Dernière intervention 10 février 2009
2 févr. 2009 à 21:20

Commence par totalement séparer le code de création des combobox et le code qui va les remplir. Tu verra tout deviendra plus clair.

Puis attention, la double boucle execute quand meme 90 requetes SQL.
L'idée serait d'avoir une methode de remplissage qui n'execute qu'une seule requete.

Aussi en C# on commence les tableau à 0, pas à 1 : for (int i = 1; i <= 15; i++) -> for (int i = 0; i < 15; i++)

Fabrice

Réponse 4 / 7

cs_vinok Messages postés 36 Date d'inscription jeudi 12 octobre 2006 Statut Membre Dernière intervention 29 mars 2013
3 févr. 2009 à 19:43

Wè j'avoue qu'en terme de complexité, ça pourrait être mieux..
J'ai pris note de ce que tu m'as dit, mais maintenant ça marche encore moins bien, et le temps d'execution est plus long !
J'ai séparé la conception de la grille de Comboboxs de son remplissage et jme retrouve forcément avec 2 fois + de boucles FOR.
Ensuite, pour les 90 requêtes sql, jvoi pa trop comment faire autrement : ce sont les coordonnées [ligne,colonne] que j'utilise pr le jour et l'horaire, paramètres de ma requête SQL..

Réponse 5 / 7

fcaruso Messages postés 18 Date d'inscription vendredi 15 avril 2005 Statut Membre Dernière intervention 10 février 2009
3 févr. 2009 à 21:02

Evite HEURE=@dh dans ta requete. c'est à cause de ca que tu en fais des tonnes.

En fait meix est de faire une classe "Activite" avec des properties publiques qui correspondent à NUMSALARIÉ NOMSALARIÉ PRENSALARIÉ HEURE NUMACTIVITÉ.

Ensuite tu fais une methode "List GetActivite(int numSalarie)" qui fabrique un List a partir du reader. Apres, la liste est en mémoire et beaucoup simple à utiliser qu'un DataReader et tres performante dans la proc de remplissage, avec soit des "foreach(Activite ain activites) if (a.Heure==1) " soit "activite.Where(a=>a.Heure==1)" si tu as System.Linq.

Quelques conseils : evite au maximum de travailler en direct sur les objets de données comme un Reader ou une DataTable, surtout dans le code des forms ( a bannir absolument ), essaye de centraliser et minimiser le nombre de methodes qui utilisent l'acces à la base dans tes objets, et privilegie des classes "metier" comme "Activite" et des collections pour travailler.

Bon courage

Fabrice

Réponse 6 / 7

cs_vinok Messages postés 36 Date d'inscription jeudi 12 octobre 2006 Statut Membre Dernière intervention 29 mars 2013
3 févr. 2009 à 21:32

alors selon toi jdevrai faire des classes-métiers pour tous mes objets?
J'ai un dataset qui est relié à la base, on ne peut pas l'exploiter?
shéma du dataset:

Jte remercie avoir pris le temps de me répondre, jv voir pour encaspuler ça dans des classes

Réponse 7 / 7

cs_vinok Messages postés 36 Date d'inscription jeudi 12 octobre 2006 Statut Membre Dernière intervention 29 mars 2013
4 févr. 2009 à 00:53

wè c clair..
okay...
bin va falloir faire plein de classes avc ttes les propriétés et tt..
Jte remercie pour les infos, c sympa.
A bientot !

Comment mettre à jour un onglet (TabPage) ?

7 réponses

Votre réponse

Discussions similaires