Insérer une propriété de navigation en base

Version imprimable

Bonjour, je travaille sur une appli MVC3.
Je souhaite enregistrer un objet "Equipe" qui ne contient qu'un ensemble de "Responsable" comme suit:

Code:

1
2
3
4
5
6
7
8
9
 
    public class Team
    {
        public int TeamId { get; set; }
 
        //propriétés de naviagtion
        public virtual ICollection<Responsable> Responsables { get; set; }
    }
}

Ensuite, dans une vue, je demande à l'utilisateur de remplir un textarea grace à JQuery. Je recupere une chaine de caractere qui contient les noms des responsables separés par un espace. Dans mon controler, je split cette chaine de caractere et je fais une recherche dans la table "Responsable" sur chaque element ainsi séparé. Ensuite je parcours le résultat de la recherche puis j'ajoute le responsable à la propriété de navigabilité de ma classe :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
 
[HttpPost]
        public ActionResult CreerTeam( String listeMembres)
        {
            Team team = new Team();
            //je decoupe la string qui est la concaténation des noms des membres de l'equipe séparés chacun par un espace. De cette maniere, je me retrouve avec un espace dans le tableau, dans le dernier indice
            String[] membres = listeMembres.Split(' ');
 
            var responsables = from s in db.Responsables
                          select s;
 
            foreach (string str in membres)
            {
                     responsables = responsables.Where(d => d.Nom.ToUpper().Contains(str.ToUpper()));
 
                    if (responsables != null)
                    {
                        foreach (var resp in responsables)
                        {
                            Response.Write(responsables.Count());
                                Response.Write(" "+resp.Nom );
                                if (resp != null)
                                {
                                    Response.Write("nom du responsable:" + resp.Nom + " mail: " + resp.Mail + " prenom: " + resp.Prenom);
                                    team.Responsables.Add(resp);
                                }
                                db.SaveChanges();
                        }
                    }
            }
 
            ViewBag.Responsables = db.Responsables.ToList();
            return View();
        }

Le Response.Write() avant l'Add en base me renvoi bien les propriétés de l'objet, mais j'obtient une erreur lors du Add(resp):

Citation:

Object reference not set to an instance of an object.

Donc je ne comprends pas...on ne peux pas ajouter un element à une propriété de navigation de cette manière? erreur de conception?

Merci et bonne journée

31/05/2011, 09h43
mermich

Salut,

En premier lieu: n'utilise pas à la fois response.write et une vue (cela peut peut être marcher mais c'est vraiment pas beau).

Enfin ton bug vien d'une erruer simple, tu n'as pas céer ta collection:

db.Responsables est null => tu dois creer ton instance avant d'ajouter des membres à ta listes ex : db.Responsables = new List<Responsable> ();

Salut,
merci pour la reponse rapide,
Pourquoi parles-tu de db.Responsables? Je precise que j'ai instancié db plus haut:
Code:

1 2 private DemandeContext db = new DemandeContext();
je pense que db.Responsables fonctionne puisque ma liste de responsables est bien remplie dans la vue. Le probleme viendrait de:
Code:

1 2 team.Responsables.Add(resp);
Est-ce qu'il faut instancier team.Responsables ? je ne comprends pas trop ta remarque, peux-tu m'expliquer stp?:mrgreen:
Merci et bonne journée

Edit: J'ai instancier mon team.Responsables dans la classe tel que suis:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 public class Team { public int TeamId { get; set; } public string nom { get; set; } //propriétés de navigation public virtual ICollection<Responsable> Responsables { get{ if (Responsables != null) { return Responsables; } else { return new List<Responsable>(); } } protected set{Responsables = value;} } }
J'ai une erreur au niveau de l'accolade du get:

Citation:

{Cannot evaluate expression because the current thread is in a stack overflow state.}

Boucle infinie? si quelqu’un a une idée

Re,

Désolé de na pas avoir été clair: c'est bien ta liste de responsable qui n'est pas instanciée.

Le plus simple et le plus sur est de faire un constructeur pour ta classe Team:

Code:

1
2
3
4
5
 
public Team()
{
  this.Responsables = new List<Responsable>();
}

Et d'enlever ton

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
        public virtual ICollection<Responsable> Responsables
        {
            get{
                if (Responsables != null)
                {
                    return Responsables;
                }
                else
                {
                    return new List<Responsable>();
                }
            }
            protected set{Responsables = value;}
        }

afin de remettre la version initiale de ta propriété:

Code:

public virtual ICollection<Responsable> Responsables { get; set; }

Enfin le problème de stackoverflow dans ton dernier post est tu à ta propriété, en effet le get et le set bouclent sur eux-même, car tu n'as pas de champ privé.

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
public virtual ICollection<Responsable> Responsables
        {
            get{
                if (Responsables != null)
                {
                    return Responsables;
                }
                else
                {
                    return new List<Responsable>();
                }
            }
            protected set{Responsables = value;}
        }

=> je pense que tu as essayé de copier-coller un bout de code sur le net sans regarder la casse, la version correcte est:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
        private List<string> mesString;
        public List<string> MesString
        {
            get
            {
                if (mesString == null)
                    mesString = new List<string>();
                return mesString;
            }
            set
            {
                this.mesString = value;
            }
        }

cf http://msdn.microsoft.com/fr-fr/libr...=vs.80%29.aspx

01/06/2011, 20h04
john85
Salut,

Merci, je comprends bien mieux comme ça. C'était donc un probleme C# de débutant :oops: Ta solution marche nickel. Donc pas de probleme pour ajouter un element à une propriété de navigation,

je précise que pour l'enregistrement, j'ai rajouté la ligne d'ajout au DbSet avant de sauvegarder:
Code:

1 2 3 team.Demandes.Add(demande); db.SaveChanges();
Bonne journée