Ключ "CategoryID" имеет тип "System.Int32", но должен иметь тип "IEnumerable<SelectListItem>"

Question

У меня небольшая проблема с сохранением моего продукта. Dropdownlist заполняется, но когда я пытаюсь сохранить, появляется ошибка, которая указана в заголовке. Я прочитал тонны статей об этом, но любая из них помогла мне. Что может повлиять на мой код? Есть идеи?

контроллер:

// GET: Admin/Products/Create
public ActionResult Create()
{
    string domain = Request.Url.Host;
    int clientid = (from a in db.Client where a.Domain == domain select a.ID).First();

    int maxID = db.Product.Where(c => c.ClientID == clientid).Max(c => (int?)c.ProductID) ?? 0;
    ViewBag.MaxID = maxID + 1;

    List<SelectListItem> categories = new List<SelectListItem>();
    foreach (var cat in db.Category.Where(c => c.ClientID == clientid))
    {
        categories.Add(new SelectListItem() { Text = cat.Name, Value = cat.CategoryID.ToString() });
    }

    ViewBag.Categories = categories;

        return View();
}

// POST: Admin/Products/Create
// To protect from overposting attacks, please enable the specific properties you want to bind to, for 
// more details see http://go.microsoft.com/fwlink/?LinkId=317598.
[HttpPost]
[ValidateAntiForgeryToken]
public ActionResult Create([Bind(Include = "ID,ProductID,Name,CategoryID,Price,Promotion,Image1,Image2,Image3,Image4,Description,ClientID")] Product product, HttpPostedFileBase file1, HttpPostedFileBase file2, HttpPostedFileBase file3, HttpPostedFileBase file4)
{
    if (ModelState.IsValid)
    {
        if (file1 != null)
        {
            var fileName1 = Path.GetFileName(file1.FileName);
            var path1 = Path.Combine(Server.MapPath("~/Images/Products/"), fileName1);
            file1.SaveAs(path1);

            @product.Image1 = Url.Content("~/Images/Products/" + fileName1);
        }

        if (file2 != null)
        {
            var fileName2 = Path.GetFileName(file2.FileName);
            var path2 = Path.Combine(Server.MapPath("~/Images/Products/"), fileName2);
            file2.SaveAs(path2);

            @product.Image2 = Url.Content("~/Images/Products/" + fileName2);
        }

        if (file3 != null)
        {
            var fileName3 = Path.GetFileName(file3.FileName);
            var path3 = Path.Combine(Server.MapPath("~/Images/Products/"), fileName3);
            file3.SaveAs(path3);

            @product.Image3 = Url.Content("~/Images/Products/" + fileName3);
        }

        if (file4 != null)
        {
            var fileName4 = Path.GetFileName(file4.FileName);
            var path4 = Path.Combine(Server.MapPath("~/Images/Products/"), fileName4);
            file4.SaveAs(path4);

            @product.Image4 = Url.Content("~/Images/Products/" + fileName4);
        }

        string domain = Request.Url.Host;
        int clientid = (from a in db.Client where a.Domain == domain select a.ID).First();

        List<SelectListItem> categories = new List<SelectListItem>();
        foreach (var cat in db.Category.Where(c => c.ClientID == clientid))
        {
            categories.Add(new SelectListItem() { Text = cat.Name, Value = cat.CategoryID.ToString() });
        }

        ViewBag.Categories = categories;

        db.Product.Add(product);
        db.SaveChanges();
        return RedirectToAction("Index");
    }

    return View(product);
}

Dropdownlist в представлении:

<div class="col-md-2">
    <div class="form-group">
        <label>Kategoria</label>
        @Html.DropDownListFor(model => model.CategoryID, (IEnumerable<SelectListItem>)ViewBag.Categories, new { @class = "form-control" })
        @Html.ValidationMessageFor(model => model.Name, "", new { @class = "text-danger" })
    </div>
</div>

Модель:

public partial class Product
    {
        public int ID { get; set; }
        public int ProductID { get; set; }
        public string Name { get; set; }
        public int CategoryID { get; set; }
        public decimal Price { get; set; }
        public int Promotion { get; set; }
        public string Image1 { get; set; }
        public string Image2 { get; set; }
        public string Image3 { get; set; }
        public string Image4 { get; set; }
        [DataType(DataType.MultilineText)]
        [AllowHtml]
        public string Description { get; set; }
        public int ClientID { get; set; }
    }

Answer 1

Вы не устанавливаете ViewBag.Categories в своем сообщении, если ModelState недействителен.

Переместите код выбора списка за пределы блока IsValid.

// POST: Admin/Products/Create
// To protect from overposting attacks, please enable the specific properties you want to bind to, for 
// more details see http://go.microsoft.com/fwlink/?LinkId=317598.
[HttpPost]
[ValidateAntiForgeryToken]
public ActionResult Create([Bind(Include = "ID,ProductID,Name,CategoryID,Price,Promotion,Image1,Image2,Image3,Image4,Description,ClientID")] Product product, HttpPostedFileBase file1, HttpPostedFileBase file2, HttpPostedFileBase file3, HttpPostedFileBase file4)
{
    if (ModelState.IsValid)
    {
        if (file1 != null)
        {
            var fileName1 = Path.GetFileName(file1.FileName);
            var path1 = Path.Combine(Server.MapPath("~/Images/Products/"), fileName1);
            file1.SaveAs(path1);

            @product.Image1 = Url.Content("~/Images/Products/" + fileName1);
        }

        if (file2 != null)
        {
            var fileName2 = Path.GetFileName(file2.FileName);
            var path2 = Path.Combine(Server.MapPath("~/Images/Products/"), fileName2);
            file2.SaveAs(path2);

            @product.Image2 = Url.Content("~/Images/Products/" + fileName2);
        }

        if (file3 != null)
        {
            var fileName3 = Path.GetFileName(file3.FileName);
            var path3 = Path.Combine(Server.MapPath("~/Images/Products/"), fileName3);
            file3.SaveAs(path3);

            @product.Image3 = Url.Content("~/Images/Products/" + fileName3);
        }

        if (file4 != null)
        {
            var fileName4 = Path.GetFileName(file4.FileName);
            var path4 = Path.Combine(Server.MapPath("~/Images/Products/"), fileName4);
            file4.SaveAs(path4);

            @product.Image4 = Url.Content("~/Images/Products/" + fileName4);
        }

        db.Product.Add(product);
        db.SaveChanges();
        return RedirectToAction("Index");
    }
    string domain = Request.Url.Host;
    int clientid = (from a in db.Client where a.Domain == domain select a.ID).First();

    List<SelectListItem> categories = new List<SelectListItem>();
    foreach (var cat in db.Category.Where(c => c.ClientID == clientid))
    {
        categories.Add(new SelectListItem() { Text = cat.Name, Value = cat.CategoryID.ToString() });
        } 
    ViewBag.Categories = categories;
    return View(product);
}

пока вы на нем.. Так как вы также пропускаете значение MaxID, вы можете просто переместить код Viewbag в отдельный метод, так что вам просто нужно вызвать метод, прежде чем вы вернете представление.

private void LoadCreateViewBagData()
{
    string domain = Request.Url.Host;
    int clientid = (from a in db.Client where a.Domain == domain select a.ID).First();
    int maxID = db.Product.Where(c => c.ClientID == clientid).Max(c => (int?)c.ProductID) ?? 0;

    ViewBag.MaxID = maxID + 1;

    List<SelectListItem> categories = new List<SelectListItem>();
    foreach (var cat in db.Category.Where(c => c.ClientID == clientid))
    {
        categories.Add(new SelectListItem() { Text = cat.Name, Value = cat.CategoryID.ToString() });
    }

    ViewBag.Categories = categories;
}

// GET: Admin/Products/Create
public ActionResult Create()
{
    LoadCreateViewBagData();
    return View();
}


[HttpPost]
[ValidateAntiForgeryToken]
public ActionResult Create([Bind(Include = "ID,ProductID,Name,CategoryID,Price,Promotion,Image1,Image2,Image3,Image4,Description,ClientID")] Product product, HttpPostedFileBase file1, HttpPostedFileBase file2, HttpPostedFileBase file3, HttpPostedFileBase file4)
{
    if (ModelState.IsValid)
    {
        //yada yada
    }
    LoadCreateViewBagData();
    return View(product);
}