Валидност на ModelState с помощта на ViewModel

Имам таблица Product, таблица ProductType и накрая таблица ProductCompanies. Ето тяхното отношение:

Връзка на таблици с продукти

Имам ProductViewModel, който е като този:

public class ProductViewModel
    {
        public Product Product { get; set; }

        public List<SelectListItem> ProductTypes { get; set; }
        public List<SelectListItem> ProductCompanies { get; set; }

        [RegularExpression(@"^[a-zA-Zàéèêçñ\s][a-zA-Zàéèêçñ\s-]+$", ErrorMessage = "Invalid name !")]
        public string ModelName { get; set; }

        [RegularExpression(@"^[a-zA-Zàéèêçñ\s][a-zA-Zàéèêçñ\s-]+$", ErrorMessage = "Invalid name !")]        
        public string CompanyName { get; set; }
    }

Имам персонализиран Създаване на изглед, който работи по следния начин: има падащ списък, който съдържа всички съществуващи типове продукти. Ако потребителят иска да създаде нов, това е възможно чрез щракване върху връзка, която кара скрита секция да се показва. В този раздел можете да добавите ново име на модела и да уточните фирмата:

@model BuSIMaterial.Models.ProductViewModel

@{
    ViewBag.Title = "Create";
}

<h2>Create</h2>

@using (Html.BeginForm()) {
    @Html.ValidationSummary(true)

    <fieldset>
        <legend>Create a material</legend>
        <div class="editor-label">
            Product type : <a class="product_type" id="product_type_link">Using a new model</a>
        </div>
        <div id= "existing_product_type" class="editor-field">
            @Html.DropDownListFor(p => p.Product.ProductType.Id_ProductType, Model.ProductTypes)
            @Html.ValidationMessageFor(model => model.Product.Id_ProductType)
        </div>
        <div id="new_product_type">
            <div class="editor-label">
                Model : 
            </div>
            <div class="editor-field">
                @Html.TextBoxFor(model => model.ModelName, new { maxlength = 50, id = "model"})
                @Html.ValidationMessageFor(model => model.ModelName)
            </div>

            <div class="editor-label">
                Company : <a class="company" id="company_link">Using a new company name</a>
            </div>
            <div id="existing_company" class="editor-field">
                @Html.DropDownListFor(p => p.Product.ProductType.Id_ProductCompany, Model.ProductCompanies)
                @Html.ValidationMessageFor(model => model.Product.ProductType.Id_ProductCompany)
            </div>
            <div id="new_company">
                <div class="editor-label">
                    Name : 
                </div>
                <div class="editor-field">
                    @Html.TextBoxFor(model => model.CompanyName, new { id = "company_name"})
                    @Html.ValidationMessageFor(model => model.CompanyName)
                </div>       
                </div>
        </div>

        <div class="editor-label">
            Catalog price : 
        </div>
        <div class="editor-field">
            @Html.TextBoxFor(model => model.Product.CatalogPrice)
            @Html.ValidationMessageFor(model => model.Product.CatalogPrice)
        </div>
        <div class="editor-label">
            Serial number : 
        </div>
        <div class="editor-field">
            @Html.TextBoxFor(model => model.Product.SerialNumber)
            @Html.ValidationMessageFor(model => model.Product.SerialNumber)
        </div>
        <div class="editor-label">
            Purchase date : 
        </div>
        <div class="editor-field">
            @Html.TextBoxFor(model => model.Product.PurchaseDate, new { id = "datepicker"})
            @Html.ValidationMessageFor(model => model.Product.PurchaseDate)
        </div>

        <div class="form-actions">
          <button type="submit" class="btn btn-primary">Create</button>
        </div>
    </fieldset>
}

<div>
    @Html.ActionLink("Back to List", "Index")
</div>

@section Scripts {
    @Scripts.Render("~/bundles/jqueryval")
    @Scripts.Render("~/bundles/jqueryui")
    @Styles.Render("~/Content/themes/base/css")

    <script type="text/javascript">

        $(document).ready(function () {

            $("#product_type_link").click(function () {

                if ($("#new_product_type").css("display") == "block") {

                    $("#new_product_type").css("display", "none");
                    $("#existing_product_type").css("display", "block");
                    $("#product_type_link").text("Using a new model");
                    $("#model").val("");
                    $("#company_name").val("");

                }
                else {
                    $("#new_product_type").css("display", "block");
                    $("#existing_product_type").css("display", "none");
                    $("#product_type_link").text("Using an existing model")
                }
            });

            $("#company_link").click(function () {

                if ($("#new_company").css("display") == "block") {

                    $("#new_company").css("display", "none");
                    $("#existing_company").css("display", "block");
                    $("#company_link").text("Using a new company name");
                    $("#company_name").val("");

                }
                else {
                    $("#new_company").css("display", "block");
                    $("#existing_company").css("display", "none");
                    $("#company_link").text("Using an existing company name")
                }
            });

        });

    </script>
}

В моето действие за публикуване просто проверявам дали потребителят е въвел нещо в скритото поле. Ако стане, върша малко работа, но нищо наистина сложно:

[HttpPost]
public ActionResult Create(ProductViewModel pvm)
{
    var productTypeList = from obj in db.ProductTypes orderby obj.ProductCompany.Name ascending where !((from element in db.VehicleTypes select element.Id_ProductType).Contains(obj.Id_ProductType)) select obj;

    ViewBag.Id_ProductType = new SelectList(productTypeList, "Id_ProductType", "Information", pvm.Product.Id_ProductType);
    pvm.ProductTypes = productTypeList.ToList().Select(p => new SelectListItem { Text = p.Information, Value = p.Id_ProductType.ToString() }).ToList();

    ViewBag.Id_ProductCompany = new SelectList(db.ProductCompanies, "Id_ProductCompany", "Name", pvm.Product.ProductType.Id_ProductCompany);
    pvm.ProductCompanies = db.ProductCompanies.ToList().Select(c => new SelectListItem { Text = c.Name, Value = c.Id_ProductCompany.ToString() }).ToList();

    Product product = new Product()
    {
        PurchaseDate = pvm.Product.PurchaseDate,
        SerialNumber = pvm.Product.SerialNumber,
        CatalogPrice = pvm.Product.CatalogPrice
    };

    ProductType productType = null; 

    if (ModelState.IsValid)
    {
        ModelStateDictionary errors = Validator.isValid(pvm.Product);

        if (errors.Count > 0)
        {
            ModelState.Merge(errors);
            return View(pvm);
        }

        if (!string.IsNullOrWhiteSpace(pvm.ModelName))
        {
            if (!string.IsNullOrWhiteSpace(pvm.CompanyName))
            {
                ProductCompany productCompany = new ProductCompany()
                {
                    Name = pvm.CompanyName
                };

                productType = new ProductType()
                {
                    Model = pvm.ModelName,
                    ProductCompany = productCompany
                };

            }
            else
            {
                productType = new ProductType()
                {
                    Model = pvm.ModelName,
                    Id_ProductCompany = pvm.Product.ProductType.Id_ProductCompany
                };
            }
        }
        else
        {
            productType = new ProductType()
            {
                Id_ProductType = pvm.Product.Id_ProductType,
                Id_ProductCompany = pvm.Product.ProductType.Id_ProductCompany
            };
        }

        product.ProductType = productType;
        db.Products.AddObject(product);
        db.SaveChanges();

        return RedirectToAction("Index");
    }

    return View(pvm);
}

Проблемът ми е, че никога не достигам до кода вътре в if(ModelState.isValid), защото моят ModelState не е валиден... И не знам защо! След отстраняване на грешки няколко пъти открих, че Product.ProducType.Model на моя ViewModel е null (и не трябва, защото ако потребителят избере нещо в падащия списък, това е добре и ако той въведе ново име на модел, трябва да е добре).

Някаква идея какво се случва?

РЕДАКТИРАНЕ: HttpGet на действието Create:

public ActionResult Create()
{
    ProductViewModel pvm = new ProductViewModel();
    var productTypeList = from obj in db.ProductTypes orderby obj.ProductCompany.Name ascending where !((from element in db.VehicleTypes select element.Id_ProductType).Contains(obj.Id_ProductType)) select obj;

    ViewBag.Id_ProductType = new SelectList(productTypeList, "Id_ProductType", "Information");
    pvm.ProductTypes = productTypeList.ToList().Select(p => new SelectListItem { Text = p.Information, Value = p.Id_ProductType.ToString() }).ToList();

    ViewBag.Id_ProductCompany = new SelectList(db.ProductCompanies, "Id_ProductCompany", "Name");
    pvm.ProductCompanies = db.ProductCompanies.ToList().Select(c => new SelectListItem { Text = c.Name, Value = c.Id_ProductCompany.ToString() }).ToList();


    return View(pvm);
}

person Traffy    schedule 29.04.2013    source източник
comment
Сигурен ли си, че полетата ти преминават тестовете за регулярен израз?   -  person Dave Alperovich    schedule 29.04.2013
comment
Не става въпрос за регулярния израз. Всъщност, дори да избера нещо в моя падащ списък, ProductType.Model е нула.   -  person Traffy    schedule 29.04.2013
comment
ИНТЕГРАЛНА СХЕМА. моля, споделете вашата get версия на Create, бих искал да видя вашата инстанция...   -  person Dave Alperovich    schedule 29.04.2013
comment
Ето го. Благодаря за отделеното време.   -  person Traffy    schedule 29.04.2013
comment
НП. Не виждам екземпляр на pvm.Product   -  person Dave Alperovich    schedule 29.04.2013
comment
Трябва ли да инстанцирам нещо? Защото създавам нов (продукт).   -  person Traffy    schedule 29.04.2013
comment
p.Product.ProductType.Id_ProductType опитвате се да попълните член на член на клас, който не сте инстанциирали. вие трябва да инстанциирате както p.Product, така и p.Product.ProductType, в противен случай не попълвате нищо.   -  person Dave Alperovich    schedule 29.04.2013
comment
Можете ли да го напишете в отговор, за да мога да го тествам и да приема отговора ви?   -  person Traffy    schedule 30.04.2013
comment
Ако вашето състояние на модела не е валидно, какво ви казва ModelState.Errors при проверка?   -  person Major Byte    schedule 30.04.2013
comment
@MajorByte Всъщност сега имам нова грешка: докато записвам обекта, той връща грешка, която гласи Не мога да вмъкна стойността NULL в колона „Модел“, но избирам нещо от моя падащ списък...   -  person Traffy    schedule 30.04.2013
comment
Тъй като продуктът трябва да има ProductType, modelbinder създава нов обект ProductType при обратно изпращане. Вероятно този ProductType има само зададен свой ID, затова получавате: не може да вмъкне стойността NULL в колона „Модел“. Сега вероятно не искате да създавате нов ProductType всеки път, когато искате да създадете продукт.   -  person Major Byte    schedule 30.04.2013
comment
@MajorByte, според мен е точно както казваш. Точно така, имам само правилния идентификатор на типа продукт... Как трябва да продължа, за да получа относителната информация?   -  person Traffy    schedule 30.04.2013
comment
Какво ще стане, ако промените ` @Html.DropDownListFor(p =› p.Product.ProductType.Id_ProductType, Model.ProductTypes)` във вашия изглед на ` @Html.DropDownListFor(p =› p.Product.Id_ProductType, Model.ProductTypes)` т.е. просто задаване на ID_ProductType на продукта на избраната стойност, а не създаване на (нов) обект ProductType с този идентификатор.   -  person Major Byte    schedule 30.04.2013
comment
нека да продължим тази дискусия в чата   -  person Traffy    schedule 30.04.2013


Отговори (1)


Както от коментарите: Променете @Html.DropDownListFor(p => p.Product.ProductType.Id_ProductType, Model.ProductTypes) във вашия изглед на @Html.DropDownListFor(p => p.Product.Id_ProductType, Model.ProductTypes), т.е. просто задаване на ID_ProductType на продукта на избраната стойност, а не създаване на (нов) обект ProductType с този идентификатор.

След това в метода Create променете

    else
    {
        productType = new ProductType()
        {
            Id_ProductType = pvm.Product.Id_ProductType,
            Id_ProductCompany = pvm.Product.ProductType.Id_ProductCompany
        };
    }

    product.ProductType = productType;
    db.Products.AddObject(product);
    db.SaveChanges();

to

    else
    {
        product.Id_ProductType = pvm.Product.Id_ProductType,
    }

    product.ProductType = productType;
    db.Products.AddObject(product);
    db.SaveChanges();
person Major Byte    schedule 29.04.2013