我是否在这个 MVC 实现中混淆了控制器/模型

Question

MVC 新手，请原谅这个问题。

我创建了两个模型，Observation 和 StaffMember。一名工作人员进行了一次观察，我根据观察记录记录了 StaffMember 的 ID (GUID)。显示结果时，我希望显示 StaffMember.FullName 而不是 StaffMember.StaffMemberId。

我已经通过使用以下模型实现了这一点：

public class Observation
{
    public Guid ObservationId { get; set; }
    public string Observer { get; set; }
    [DisplayFormat(DataFormatString = "{0:dd/MM/yyyy}")]
    public DateTime ObservationDate { get; set; }
    public Guid StaffMemberId { get; set; }
    public int? ObservationGrade { get; set; }
    public virtual IEnumerable<StaffMember> StaffMembers { get; set; }

    public string StaffMemberName
    {
        get
        {
            var db = new ObservationTrackerContext();
            var staffMember = db.StaffMembers.FirstOrDefault(s => s.StaffMemberId == StaffMemberId);

            return staffMember != null ? staffMember.FullName : "Missing entry or error";
        }
    }
}

我有点担心我将模型的定义与数据库交互混淆了。这是对 MVC 模式的正确使用还是我应该以不同的方式这样做？

提前感谢任何 cmets 或建议。

Answer 1

我假设您指的是最后一部分：

public string StaffMemberName
    {
        get
        {  var db = new ObservationTrackerContext();
            var staffMember = db.StaffMembers.FirstOrDefault(s => s.StaffMemberId == StaffMemberId);    
        return staffMember != null ? staffMember.FullName : "Missing entry or error";
        }
     }

我个人认为，虽然这可能有效，但它并不是最干净的做事方式。 我尽可能保持我的视图模型简洁明了，所以基本上只是一堆带有{get;set;} 的属性。

您如何设置此视图模型中的其他属性？

有什么理由不能像设置StaffMemberId 那样设置StaffMemberName 吗？

我个人会将所有逻辑（除了DataAnnotations 等）排除在视图模型之外。 更好的一步是将您的逻辑放在控制器方法中。 更好的是，放置一个额外的层，您可以在其中进行所有数据库检索。

通过这种方式，控制器和视图保持尽可能简单。

在您的情况下，我会做以下事情，我会像这样创建一个 MemberService：

public class MemberService
{
public string GetStaffMemberById(Guid Id)
{
    var db = new ObservationTrackerContext();
    var staffMember = db.StaffMembers.FirstOrDefault(s => s.StaffMemberId == StaffMemberId);return staffMember != null ? staffMember.FullName : "Missing entry or error";    
    return staffMember;
} 
}

然后，在控制器中，设置视图模型：

public ActionResult GetInformation(Guid id)
{
var vm = new StaffMemberViewModel();
vm.StaffMemberName = memberService.GetStaffMemberById(id);
return View(vm);
}

通过这种方式，您的视图模型可以更改为

public string StaffMemberName {get;set;}

编辑：我更关注视图模型中的数据库检索方面，但@Miller Koijam 也提出了一个很好的观点，将这些不同的模型封装到一个复合视图模型中。

编辑号 2：您似乎返回的是纯数据库对象，这在大型项目中并不是真正可维护的。您应该将这些数据库对象“转换”或“投影”到视图模型中。

this link gives a pretty good explanation on how to do this

Answer 2

最好的办法是将每个模型分成不同的类。 如果存在一对一的关系，那么您的模型就可以了。 你可以有一个类似的方法

GetStaffMemberById(Guid Id)
{
     //Add your code to fetch the StaffMember
}

如果存在一对多关系 您需要在更大的模型中显示两个模型的关联，您可以使用名为 StaffMemberObeservation/OberservationStaffMember 的复合模型作为父子的关联性。 如果您的 StaffMember 有多个 Observation 而不是

public class StaffMemberObeservation
{
public Observation { get; set; }
public IList <StaffMember>{ get; set; }
}

如果您的 Observation 的工作人员不止一名

public class ObersavtionStaffMember
{


   public Staff Member{ get; set; }
   public IList <Observation> { get; set; }

}