[英]MVC5 Linq to ViewModel to Razor View code improvement?
我想从两个EF表(菜单和菜单项)中获取数据。 然后,我想将结果放入ViewModel中。 因此,我可以轻松地在Razor视图中循环浏览数据。
我已经工作了。 但是我是.Net MVC的新手,所以我想知道我的代码是否可以改进。 也许查询可以合并并且缩短很多。 而且我正在使用AsEnumerable(),我不确定这是否是最好的方法。
我认为我的尝试还可以。 但我想听听您的想法。 欢迎提出任何改进代码的建议。 谢谢。
控制器类:
public ActionResult Index(int? id)
{
if (id == null) return RedirectToAction("Index", new { controller = "Menu" });
var menu =
(from m in _db.Menus
join mi in _db.MenuItems on m.Id equals mi.MenuId
where m.Id == id
select m).First();
var menuItems =
(from mi in _db.MenuItems.AsEnumerable()
join m in _db.Menus on mi.MenuId equals m.Id
where m.Id == id
select new MenuItem
{
Id = mi.Id,
Name = mi.Name,
Href = mi.Href,
CssClass = mi.CssClass,
CssId = mi.CssId,
Title = mi.Title,
Weight = mi.Weight
});
var model = new MenuModelView
{
Id = menu.Id,
Name = menu.Name,
CssClass = menu.CssClass,
CssId = menu.CssId,
Deleted = menu.Deleted,
MenuItems = menuItems
};
return View(model);
}
ViewModel类:
using System.Collections.Generic;
namespace DesignCrew.Areas.Admin.Models
{
public class MenuModelView
{
public int Id { get; set; }
public string Name { get; set; }
public string CssClass { get; set; }
public string CssId { get; set; }
public bool Deleted { get; set; }
public IEnumerable<MenuItem> MenuItems { get; set; }
}
}
剃刀视图:
@model DesignCrew.Areas.Admin.Models.MenuModelView
@{
ViewBag.Title = "Index";
}
<h2>Menu - @Html.DisplayFor(model => model.Name, new { @class = "control-label col-md-2" })</h2>
<p>
@Html.ActionLink("Create New Item", "Create")
</p>
<table class="table">
<tr>
<th>
@Html.DisplayNameFor(model => model.MenuItems.First().Id)
</th>
<th>
@Html.DisplayNameFor(model => model.MenuItems.First().Name)
</th>
<th>
@Html.DisplayNameFor(model => model.MenuItems.First().Href)
</th>
<th>
@Html.DisplayNameFor(model => model.MenuItems.First().Title)
</th>
<th>
@Html.DisplayNameFor(model => model.MenuItems.First().CssClass)
</th>
<th>
@Html.DisplayNameFor(model => model.MenuItems.First().CssId)
</th>
<th>
@Html.DisplayNameFor(model => model.MenuItems.First().ParentId)
</th>
<th>
@Html.DisplayNameFor(model => model.MenuItems.First().Weight)
</th>
<th>Options</th>
</tr>
@foreach (var item in Model.MenuItems)
{
<tr>
<td>
@Html.DisplayFor(modelItem => item.Name)
</td>
<td>
@Html.DisplayFor(modelItem => item.Name)
</td>
<td>
@Html.DisplayFor(modelItem => item.Href)
</td>
<td>
@Html.DisplayFor(modelItem => item.Title)
</td>
<td>
@Html.DisplayFor(modelItem => item.CssClass)
</td>
<td>
@Html.DisplayFor(modelItem => item.CssId)
</td>
<td>
@Html.DisplayFor(modelItem => item.ParentId)
</td>
<td>
@Html.DisplayFor(modelItem => item.Weight)
</td>
<td>
@Html.ActionLink("Edit", "Edit", new { id = item.Id }, new { @class = "link-menu" }) |
@Html.ActionLink("Delete", "Delete", new { id = item.Id }, new { @class = "link-menu" })
</td>
</tr>
}
</table>
如果您的EF模型在Menu
和MenuItem
之间具有可导航的外键,则无需显式联接两个表,并且您可以在获取父Menu
的同时急于加载子MenuItems
,只需从父Menu
导航到儿童:
var menu = _db.Menus
.Include("MenuItems") // Or, use the typed version on newer EF's
.First(m => m.id == id); // Many LINQ expressions allow predicates
return new MenuModelView
{
Menu = menu.Name,
CssClass = menu.CssClass,
CssId = menu.CssId,
Deleted = menu.Deleted,
MenuItems = menu.MenuItems
}
而且,由于MenuModelView
与Menu
EF实体之间似乎有很多共性,因此您可以考虑使用AutoMapper
。 配置完成后,这将使您可以将手动映射步骤替换为以下内容:
return Mapper.Map<Menu, MenuModelView>(menu);
编辑
这是一个便宜又讨厌的ViewModel
,它包装了EF实体,并用表示层数据进行了扩充。 纯粹主义者可能会注意到,您应该为包装好的EF模型创建新的类,尽管如此,您的EF模型似乎还是对Html
建模:)
public class MenuModelView
{
// Presentation tier stuff
public string PageTitle { get; set; }
public string MetaTagsForSEO { get; set; }
public bool IsThisARegisteredUserSoSkipTheAdverts { get; set; }
// Your EF / Domain Model
public Menu Menu { get; set; }
}
您的剃须刀@Model
是MenuModelView
。
声明:本站的技术帖子网页,遵循CC BY-SA 4.0协议,如果您需要转载,请注明本站网址或者原文地址。任何问题请咨询:yoyou2525@163.com.