Флажки в шаблоне редактора не привязаны к модели в пост-действии

Question

Я использую шаблон редактора для отображения флажка для каждой роли, которой может быть назначен пользователь.Модель:

public class UserModel
{
    [Required]
    [Display(Name = "User name")]
    public string UserName { get; set; }

    [Required]
    [DataType(DataType.EmailAddress)]
    [Display(Name = "Email address")]
    public string Email { get; set; }

    [Required]
    [StringLength(100, ErrorMessage = "The {0} must be at least {2} characters long.", MinimumLength = 6)]
    [DataType(DataType.Password)]
    [Display(Name = "Password")]
    public string Password { get; set; }

    [DataType(DataType.Password)]
    [Display(Name = "Confirm password")]
    [Compare("Password", ErrorMessage = "The password and confirmation password do not match.")]
    public string ConfirmPassword { get; set; }

    public IEnumerable<string> UserRoles { get; set; }
}
public class UserRoleModel
{
    public IEnumerable<RoleViewModel> AllRoles { get; set; }
    public UserModel user { get; set; }

    public UserRoleModel()
    {
        this.AllRoles = Roles.GetAllRoles().Select(r => new RoleViewModel
        {
            Name = r
        });
        this.user = new UserModel();
    }
}
public class RoleViewModel
{
    public string Name { get; set; }
    public bool Selected { get; set; }
}

Контроллер:

    public ActionResult Create()
    {
        return View(new UserRoleModel());
    }

    [HttpPost]
    public ActionResult Create(UserRoleModel model)
    {
        if (ModelState.IsValid)
        {
            MembershipCreateStatus createStatus;
            Membership.CreateUser(model.user.UserName, model.user.Password, model.user.Email, null, null, true, null, out createStatus);

            if (createStatus == MembershipCreateStatus.Success)
            {


                foreach (var r in model.AllRoles)
                {
                    if (r.Selected)
                    {
                        Roles.AddUserToRole(model.user.UserName, r.Name);
                    }
                }

                return RedirectToAction("Index", "Home");
            }
            else
            {
                ModelState.AddModelError("", ErrorCodeToString(createStatus));
            }
        }

        return View(model);
    }

Вид:

@model BBmvc.Areas.Tools.Models.UserRoleModel


@using (Html.BeginForm()) {
@Html.ValidationSummary(true)
<fieldset>
    <legend>UserModel</legend>

    <div class="editor-label">
        @Html.LabelFor(model => model.user.UserName)
    </div>
    <div class="editor-field">
        @Html.EditorFor(model => model.user.UserName)
        @Html.ValidationMessageFor(model => model.user.UserName)
    </div>

    <div class="editor-label">
        @Html.LabelFor(model => model.user.Email)
    </div>
    <div class="editor-field">
        @Html.EditorFor(model => model.user.Email)
        @Html.ValidationMessageFor(model => model.user.Email)
    </div>

    <div class="editor-label">
        @Html.LabelFor(model => model.user.Password)
    </div>
    <div class="editor-field">
        @Html.EditorFor(model => model.user.Password)
        @Html.ValidationMessageFor(model => model.user.Password)
    </div>

    <div class="editor-label">
        @Html.LabelFor(model => model.user.ConfirmPassword)
    </div>
    <div class="editor-field">
        @Html.EditorFor(model => model.user.ConfirmPassword)
        @Html.ValidationMessageFor(model => model.user.ConfirmPassword)
    </div>
    <div class="editor-field">
        @Html.EditorFor(x => x.AllRoles)
    </div>


    <p>
        <input type="submit" value="Create" />
    </p>
</fieldset>

}

и шаблон редактора

    @model BBmvc.Areas.Tools.Models.RoleViewModel
@Html.CheckBoxFor(x => x.Selected)
@Html.LabelFor(x => x.Selected, Model.Name)
@Html.HiddenFor(x => x.Name)
<br />

Проблема в том, что в пост-действии не делается различий независимо от того, установлен флажок или нет.Кажется, что это никак не связано с моделью.

Answer 1

Ваша проблема связана с отложенным выполнением запросов LINQ. Вам нужно с нетерпением инициализировать коллекцию:

public class UserRoleModel
{
    public IEnumerable<RoleViewModel> AllRoles { get; set; }
    public UserModel user { get; set; }

    public UserRoleModel()
    {
        this.AllRoles = Roles.GetAllRoles().Select(r => new RoleViewModel
        {
            Name = r
        }).ToList();
        this.user = new UserModel();
    }
}

Обратите внимание на .ToList() вызов:

this.AllRoles = Roles.GetAllRoles().Select(r => new RoleViewModel
{
    Name = r
}).ToList();

А вот и объяснение. Когда вы пишете:

this.AllRoles = Roles.GetAllRoles().Select(r => new RoleViewModel
{
    Name = r
});

В данный момент запрос не выполняется. Строится только дерево выражений, но фактический запрос выполняется только тогда, когда что-то начинает перебирать коллекцию. И что начинает повторяться? Сначала это вид. Внутри представления вы используете шаблон редактора для этой коллекции:

@Html.EditorFor(x => x.AllRoles) 

Поскольку AllRoles является свойством коллекции, ASP.NET MVC автоматически выполнит итерацию и отобразит шаблон редактора для каждого элемента коллекции. Так что это работает для правильной визуализации представления.

Теперь давайте посмотрим, что происходит, когда форма размещена. Вы отправляете сообщение в действие Create, и связыватель модели по умолчанию запускается. Конструктор вызывается, но поскольку нечего перебирать свойство AllRoles, на этот раз запрос не выполняется. Фактически он выполняется позже внутри действия и значения теряются.

По этой причине я бы рекомендовал вам не инициализировать ваши модели представления внутри конструкторов. Было бы лучше сделать это внутри соответствующих действий контроллера:

public class UserRoleModel
{
    public IEnumerable<RoleViewModel> AllRoles { get; set; }
    public UserModel user { get; set; }
}

и затем:

public ActionResult Create()
{
    var model = new UserRoleModel
    {
        AllRoles = Roles.GetAllRoles().Select(r => new RoleViewModel
        {
            Name = r
        }).ToList(),
        user = new UserModel()
    };
    return View(model);
}