NullReferenceException dans le fichier cshtml Edit

Version imprimable

Bonjour,
Je suis entrain de suivre le tutoriel: http://www.asp.net/mvc/tutorials/get...vc-application
et la je me suis trouvé face à un problème dont je n'ai pas su résoudre, pourtant ça me parait que j'ai écris exactement comme le tutoriel.

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
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
@model association.Models.reunion
 
@{
    ViewBag.Title = "Edit";
}
 
<h2>Edit</h2>
 
<script src="@Url.Content("~/Scripts/jquery.validate.min.js")" type="text/javascript"></script>
<script src="@Url.Content("~/Scripts/jquery.validate.unobtrusive.min.js")" type="text/javascript"></script>
 
@using (Html.BeginForm())
{
    @Html.ValidationSummary(true)
    <fieldset>
        <legend>reunion</legend>
 
        @Html.HiddenFor(model => model.reunionID)
 
        <div class="editor-label">
            @Html.LabelFor(model => model.sujet)
        </div>
        <div class="editor-field">
            @Html.EditorFor(model => model.sujet)
            @Html.ValidationMessageFor(model => model.sujet)
        </div>
 
        <div class="editor-label">
            @Html.LabelFor(model => model.pv)
        </div>
        <div class="editor-field">
            @Html.EditorFor(model => model.pv)
            @Html.ValidationMessageFor(model => model.pv)
        </div>
 
        <div class="editor-label">
            @Html.LabelFor(model => model.datereunion)
        </div>
        <div class="editor-field">
            @Html.EditorFor(model => model.datereunion)
            @Html.ValidationMessageFor(model => model.datereunion)
        </div>
            <div class="editor-field">
         <table>
        <tr>
            @{
                int cnt = 0;
              List<association.ViewModels.CreateAssignedMemberData> membres = ViewBag.membres;
 
                foreach (var membre in membres) {
                    if (cnt++ % 3 == 0) {
                        @:  </tr><tr> 
                    }
                    @: <td> 
                        <input type="checkbox" 
                               name="selectedCourses" 
                               value="@membre.PersonneID" 
                               @(Html.Raw(membre.Assigned ? "checked=\"checked\"" : "")) /> 
                        @membre.PersonneID @:  @membre.Nom
                    @:</td>
                }
                @: </tr>
            }
    </table>
</div>
 
        <p>
            <input type="submit" value="Save" />
        </p>
    </fieldset>
}
 
<div>
    @Html.ActionLink("Back to List", "Index")
</div>

L'erreur NullReferenceException est signalé au niveau du "foreach (var membre in membres)".
Nb: la classe "membre" hérite de la classe "personne"

La classe CreateAssignedMemberData se trouve sous un dossier appelé "ViewModels" et contient ce code:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
using System;
using System.Collections.Generic;
using System.Linq;
using System.Web;
using System.ComponentModel.DataAnnotations;
 
 
namespace association.ViewModels
{
    public class CreateAssignedMemberData
    {
        public int PersonneID { get; set; }
        public string Nom { get; set; }
        public string Prenom { get; set; }
        public bool Assigned { get; set; }
    }
}

classe reunion:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
using System;
using System.Collections.Generic;
using System.Linq;
using System.Web;
using System.ComponentModel.DataAnnotations;
 
namespace association.Models
{
    public class reunion
    {
        public int reunionID { set; get; }
        public string sujet { set; get; }
        public string pv { set; get; }
        [DisplayFormat(DataFormatString = "{0:d}")]
        public DateTime datereunion { set; get; }
        public virtual ICollection<membre> membres { get; set; }
 
    }
}

Classe membre:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
using System;
using System.Collections.Generic;
using System.Linq;
using System.Web;
using association.Models;
using System.ComponentModel.DataAnnotations;
 
namespace association.Models
{
    public class membre : personne
    {
 
        public DateTime dateinscription { get; set; }
 
        public virtual profil profil { get; set; }
        public virtual ICollection<reunion> reunions { get; set; }
 
    }
 
}

Pouvez vous m'aidez? merci

25/04/2013, 18h36
g.Arnaud

Dans ta méthode Edit, tu récupère bien les membres de ta réunion?
Ne pas le faire peut te poser problème si la lazyloading est désactiver.

Au niveau de la récupération tu aurais ceci :

Code:

var reunion = db.Reunions.Include(r=>r.membres ).Single(r=>r.reunionID == id);

En faisant comme ça, tu récupère ta réunion et ses membres en une seul requête SQL.

Citation:

Envoyé par g.Arnaud

Dans ta méthode Edit, tu récupère bien les membres de ta réunion?
Ne pas le faire peut te poser problème si la lazyloading est désactiver.

Au niveau de la récupération tu aurais ceci :

Code:

var reunion = db.Reunions.Include(r=>r.membres ).Single(r=>r.reunionID == id);

En faisant comme ça, tu récupère ta réunion et ses membres en une seul requête SQL.

Non je n'arrive pas à récupérer j'ai l'erreur NullReferenceException
voila les méthodes edit dans reunionController

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
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
  public ActionResult Edit(int id)
        {
            reunion reunion = db.reunions
 
         .Include(i => i.membres)
         .Where(i => i.reunionID == id)
         .Single();
            return View(reunion);
        }
        private void PopulateAssignedCourseData(reunion reunion)
        {
            var allMembers = db.membres;
            var reunionMembre = new HashSet<int>(reunion.membres.Select(c => c.PersonneID));
            var viewModel = new List<AssignedMembreData>();
            foreach (var membre in allMembers)
            {
                viewModel.Add(new AssignedMembreData
                {
                    PersonneID = membre.PersonneID,
                    Nom = membre.nom,
                    Prenom = membre.prenom,
                    Assigned = reunionMembre.Contains(membre.PersonneID)
                });
            }
            ViewBag.Courses = viewModel;
        }
 
        //
        // POST: /reunion/Edit/5
 
        [HttpPost]
        public ActionResult Edit(int id, FormCollection formCollection, string[] selectedCourses)
        {
            reunion reunionToUpdate = db.reunions
 
           .Include(i => i.membres)
           .Where(i => i.reunionID == id)
           .Single();
 
            PopulateAssignedCourseData(reunionToUpdate);
            return View(reunionToUpdate);
             }
        private void UpdatereunionMembre(string[] selectedMembre, reunion reunionToUpdate)
{
    if (selectedMembre == null)
    {
        reunionToUpdate.membres = new List<membre>();
        return;
    }
 
    var selectedCoursesHS = new HashSet<string>(selectedMembre);
    var reunionMembres = new HashSet<int>
        (reunionToUpdate.membres.Select(c => c.PersonneID));
    foreach (var membre in db.membres)
    {
        if (selectedCoursesHS.Contains(membre.PersonneID.ToString()))
        {
            if (!reunionMembres.Contains(membre.PersonneID))
            {
                reunionToUpdate.membres.Add(membre);
            }
        }
        else
        {
            if (reunionMembres.Contains(membre.PersonneID))
            {
                reunionToUpdate.membres.Remove(membre);
            }
        }
    }
}

25/04/2013, 20h11
g.Arnaud

En relisant une énième fois, je me rends compte que c n'est pas les membres assignés à la réunion que tu affiches.

dans ta vue edit tu as

Code:

List<association.ViewModels.CreateAssignedMemberData> membres = ViewBag.membres;

Or dans ton contrôleur, dans tes méthodes edit, tu ne renseignes pas ViewBag.membres donc l'exception est bien justifiée.

Merci beaucoup, je l'ai corrigé et j'ai renseigné membres, ainsi que j'ai modifier la méthode edit:

Code:

1
2
3
4
5
6
7
8
9
10
  public ActionResult Edit(int id)
        {
            reunion reunion = db.reunions
 
         .Include(i => i.membres)
         .Where(i => i.reunionID == id)
         .Single();
            PopulateAssignedCourseData(reunion);
            return View(reunion);
        }

Maintenant l'affichage des membres s'effectue convenable à travers l'edit mais lorsque je valide le changement, rien ne change! ça dois me mener vers l'index mais ce n'est pas le cas, ça reste dans la page edit et meme au niveau de la base de données rien n'a changé.

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
  </div>
            <div class="editor-field">
         <table>
        <tr>
            @{
                int cnt = 0;
                List<association.ViewModels.AssignedMembreData> membres = ViewBag.membres;
 
                foreach (var membre in membres) {
                    if (cnt++ % 3 == 0) {
                        @:  </tr> <tr> 
                    }
                    @: <td> 
                        <input type="checkbox" 
                               name="selectedCourses" 
                               value="@membre.PersonneID" 
                               @(Html.Raw(membre.Assigned ? "checked=\"checked\"" : "")) /> 
                        @membre.PersonneID @:  @membre.Nom
                    @:</td>
                }
                @: </tr>
            }
    </table>
</div>
 
        <p>
            <input type="submit" value="Save" />
        </p>
    </fieldset>
}
 
<div>
    @Html.ActionLink("Back to List", "Index")
</div>

Nb: <tr> est souligné en vert indiquant que le texte n'est pas autorisé entre la balise d'ouverture et fermeture tr

Ce que tu as :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 // // POST: /reunion/Edit/5 [HttpPost] public ActionResult Edit(int id, FormCollection formCollection, string[] selectedCourses) { reunion reunionToUpdate = db.reunions .Include(i => i.membres) .Where(i => i.reunionID == id) .Single(); PopulateAssignedCourseData(reunionToUpdate); return View(reunionToUpdate); }
à aucun moment, tu ne fais de redirection vers l'index

Code:

return View(reunionToUpdate);

ré-affiche le formulaire d'édition.

Il faut rediriger vers l'index, si tu n'as pas eu de soucis lors de la mise à jour

Si il n'y a pas d'erreur dans le formulaire et que tu veux bien rediriger vers l'index, tu fais :

Code:

return RedirectToAction("Index");

Si c'est bien vers l'action index du contrôleur courant que tu veut rediriger. Sinon, tu met le nom de contrôleur en second paramètre lors de l'appel à la méthode RedirectToAction.

26/04/2013, 11h51
janyoura

Merci je me suis rendu compte des fautes et j'ai corrigé le problème