Discussion :

[PatStan17]

Bonjour à tous

Contexte : ASPNET ET C#

Je voudrai mettre en variable de session une collection d'objets, je dois mal m'y prendre car à chaque fois que je récupére ma collection je n'ai qu'un objet dedans alors que j'en ai mis plusieurs ...voici mon code , il doit y avoir un binz mais je ne le vois pas:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
 
 
 protected void MiseAJourSession(int IdMenu)
    {
      ///Je crée une collection d'objets Menu
        List<Menu> LstMenus = new List<Menu>();
 
 ///je récupére la collection déjà mise en variable de session
        LstMenus = (List<Menu>)Session["LstMenus"];
 
 
 /// Je crée une instance de mon objet Menu avec l'identifiant passé en paramétre de la fonction
        Menu LeMenu = new Menu(IdMenu);
 
 ///j'ajoute mon nouveau menu dans la collection
        LstMenus.Add(LeMenu);
 ///Je supprime ma variable de session
        Session.Remove("LstMenus");
 ///Je recréer ma variable de session avec la collection ou j'ai ajouté un nouvel objet
        Session.Add("LstMenus", LstMenus);
    }

qqn voit'il un problème dans mon code et:ou dans ma logique ?

MERCI D'AVANCE

P.JEAN

[Bluedeep]

Bonjour

Déjà, je vois (sauf erreur de ma part, bien sur)deux choses bizarres et inutiles :

- Quel interêt d'instancier ta liste puis d'écraser le contenu de la variable référençant l'instance avec le contenu de la variable de Session ?

- Ensuite, tu récupére ta liste, qui est en variable de session, tu la mets à jour, mais quel interêt de l'enlever puis la remettre ensuite (puisqu'elle est déjà en variable de session) ?

La lecture de ton code laisse apparaitre (a priori) une mauvaise compréhension de la notion de réferences aux objets. (et/ou de "reference objects" vs "value objects").

List est un objet "réference", pas un objet "value".

[PatStan17]

MERCI Bluedeep POUR TA REPONSE

Je vais surement dire des bêtises mais bon ...c'est le pix à payer pour progresser ...

Pour moi je ne peux pas travailler en direct sur ma liste qui est en variable session ....Donc je la récupére et je "transvase" le contenu sur la liste que j'ai crée dans mon code....à laquelle j'ajoute mon nouveau menu...


Je supprime ma variable de session car pour moi la liste qui est dedans est maintenant obsolète ...puis je la recréer avec la liste mise à jour ....

Si cela n'est pas bon pourrais tu me proposer un solution alternative ?

Voilà ...


EN TOUT CAS ENCORE MERCI
P.

[Bluedeep]

MERCI Bluedeep POUR TA REPONSE

Je vais surement dire des bêtises mais bon ...c'est le pix à payer pour progresser ...

Pour moi je ne peux pas travailler en direct sur ma liste qui est en variable session ....Donc je la récupére et je "transvase" le contenu sur la liste que j'ai crée dans mon code....à laquelle j'ajoute mon nouveau menu...

Mais justement, tu ne transvases rien du tout car tu références la même instance ! Simplement, cette instance tu la vire des variables de sessions pour la remettre derrière, mais c'est la même.

Si tu veux changer de liste, tu en instancie une autre, tu mets ce que tu veux dedans et tu fais avec la nouvelle instance, un Remove de l'ancienne et un Add de la nouvelle.

Or, ici, tu instancies une liste au début, mais tu ne la gardes pas car tu écrases la variable (LstMenus) qui la référence avec celle contenue dans ta variable de Session.

[PatStan17]

Bon ok ....Je vais tester ça de plus près ...et essayer de comprendre ...

Merci

P.JEAN