填充对象的简单方法

Question

我有以下课程：

    public class SectionViewModel
    {
        private static EFModels.VTSEntities _db = new EFModels.VTSEntities();
        public int ID { get; set; }
        public string SectionName { get; set; }
        public bool Active { get; set; }
        public string SiteName { get; set; }
   }

我想从 _db.Sections 中选择一个元素并填充这个类的对象。我可以这样做：

    public SectionViewModel(int ID)
    {
        var s = (from i in _db.Sections
                 where i.ID == ID
                 select new SectionViewModel()
                 {
                     ID = i.ID,
                     SectionName = i.SectionName,
                     Active = i.Active,
                     SiteName = i.Site.SiteName
                 }).FirstOrDefault();
        ID = s.ID;
        SectionName = s.SectionName;
        Active = s.Active;
    }

它可以工作，但是当字段数为 10 时，代码量很大。我想写一些类似的东西

    // IT DOES NOT WORK, ONLY EXAMPLE
    public SectionViewModel(int ID)
    {
        this = (from i in _db.Sections
                 where i.ID == ID
                 select new SectionViewModel()
                 {
                     ID = i.ID,
                     SectionName = i.SectionName,
                     Active = i.Active,
                     SiteName = i.Site.SiteName
                 }).FirstOrDefault();
    }

添加：

创建 SectionViewModel 对象（它是一个视图模型类）：

    public ActionResult SectionForm(int? id)
    {
        SectionViewModel model = new SectionViewModel(id);
        return View(model);
    }

但是，当然，这是不可能的，因为“this”只能用于读取。有什么办法吗？

Answer 1

编辑

好的，所以你从Section 构造一个SectionViewModel，而不是Section。这会有所不同，但我的原始答案中的许多评论仍然适用。

更好的方法是

public ActionResult SectionForm(int id)
{
    Section section = this._context.Sections.Find(id);
    SectionViewModel model = .... // Mapping code
    return View(model);
}

这部分// Mapping code 可以是将属性从section 复制到model 的任何内容。你可以使用 AutoMapper。

在SectionViewModel 的构造函数中不这样做的原因是首先不应该有静态上下文。您可以改为创建和处置上下文。但是谁说SectionViewModels 总是单独构建的？也许在另一种方法中，您将返回它们的列表。单独创建每个模型效率非常低。 AutoMapper's Project().To 方法在那里是合适的。

---

原文

在构造函数中构造一个对象（这是var s = (...).FirstOrDefault()部分发生的事情）然后将其属性复制到构造函数的所有者，这是同一类型，Section，这是荒谬的。更荒谬的是，在查询中您也从一个部分构造一个Section。所以运行语句后...

Section model = new Section(id);

...你已经构建了三个相同的Sections，其中最后一个被最终使用：

第 1 部分：from i in _db.Sections where i.ID == ID select i.
第二部分：select new Section() {...}
第三节：Section(int ID)

而且 EF 甚至不允许像

这样的语句

from i in _db.Sections
select new Section() {...}

它会告诉你不能在 LINQ-to-Entities 查询中构造实体。

但是删除这个select new Section() {...} 甚至不是声音重构的开始。整个结构很荒谬。

另一个不好的做法是使用静态上下文。上下文的设计寿命很短，因为它们缓存了从数据库中获取的每个实体。静态上下文是内存泄漏。

实现你想要的方法很简单......

public ActionResult SectionForm(int id)
{
    Section model = this._context.Sections.Find(id);
    return View(model);
}

...其中this._context 是您的VTSEntities 的一个实例，它是按控制器实例创建的（或由控制反转容器注入其中）。

您的编程风格隐约让人想起 Active Record，这种模式与 EF 的存储库/工作单元模式（其中DbSet 是一个 repo，DbContext 是一个 UoW）不能很好地混合。它还打破了构建 EF 的persistance ignorance 原则。它破坏了查询的可组合性，并且很容易导致n + 1 查询。