MVC 3 - 如何实现服务层，我需要存储库吗？

Question

我目前正在使用 EF Code First、SQL CE 和 Ninject 构建我的第一个 MVC 3 应用程序。 我读过很多关于使用存储库、工作单元和服务层的内容。我想我已经理清了基础知识，并且已经做出了自己的实现。

这是我当前的设置：

实体

public class Entity
{
    public DateTime CreatedDate { get; set; }
    public Entity()
    {
        CreatedDate = DateTime.Now;
    }
}

public class Profile : Entity
{
    [Key]
    public Guid UserId { get; set; }
    public string ProfileName { get; set; }

    public virtual ICollection<Photo> Photos { get; set; }

    public Profile()
    {
        Photos = new List<Photo>();
    }

public class Photo : Entity
{
    [Key]
    public int Id { get; set; }
    public Guid FileName { get; set; }
    public string Description { get; set; }

    public virtual Profile Profile { get; set; }
    public Photo()
    {
        FileName = Guid.NewGuid();
    }
}

SiteContext

public class SiteContext : DbContext
{
    public DbSet<Profile> Profiles { get; set; }
    public DbSet<Photo> Photos { get; set; }

    protected override void OnModelCreating(DbModelBuilder modelBuilder)
    {
        modelBuilder.Conventions.Remove<PluralizingTableNameConvention>();
    }
}

接口：IServices

public interface IServices : IDisposable
{
    PhotoService PhotoService { get; }
    ProfileService ProfileService { get; }

    void Save();
}

实现：服务

public class Services : IServices, IDisposable
{
    private SiteContext _context = new SiteContext();

    private PhotoService _photoService;
    private ProfileService _profileService;

    public PhotoService PhotoService
    {
        get
        {
            if (_photoService == null)
                _photoService = new PhotoService(_context);

            return _photoService;
        }
    }

    public ProfileService ProfileService
    {
        get
        {
            if (_profileService == null)
                _profileService = new ProfileService(_context);

            return _profileService;
        }
    }

    public void Save()
    {
        _context.SaveChanges();
    }

    private bool disposed = false;

    protected virtual void Dispose(bool disposing)
    {
        if (!this.disposed)
        {
            if (disposing)
            {
                _context.Dispose();
            }
        }
        this.disposed = true;
    }

    public void Dispose()
    {
        Dispose(true);
        GC.SuppressFinalize(this);
    }
}

接口

public interface IPhotoService
{
    IQueryable<Photo> GetAll { get; }
    Photo GetById(int photoId);
    Guid AddPhoto(Guid profileId);
}

实现

public class PhotoService : IPhotoService
{
    private SiteContext _siteContext;

    public PhotoService(SiteContext siteContext)
    {
        _siteContext = siteContext;
    }

    public IQueryable<Photo> GetAll
    {
        get
        {
            return _siteContext.Photos;
        }
    }

    public Photo GetById(int photoId)
    {
        return _siteContext.Photos.FirstOrDefault(p => p.Id == photoId);
    }

    public Guid AddPhoto(Guid profileId)
    {
        Photo photo = new Photo();

        Profile profile = _siteContext.Profiles.FirstOrDefault(p => p.UserId == profileId);

        photo.Profile = profile;
        _siteContext.Photos.Add(photo);

        return photo.FileName;
    }
}

Global.asax

protected void Application_Start()
    {
        AreaRegistration.RegisterAllAreas();

        RegisterGlobalFilters(GlobalFilters.Filters);
        RegisterRoutes(RouteTable.Routes);

        ControllerBuilder.Current.SetControllerFactory(new NinjectControllerFactory());

        Database.SetInitializer<SiteContext>(new SiteInitializer());
    }

NinjectControllerFactory

public class NinjectControllerFactory : DefaultControllerFactory
{
    private IKernel ninjectKernel;
    public NinjectControllerFactory()
    {
        ninjectKernel = new StandardKernel();
        AddBindings();
    }
    protected override IController GetControllerInstance(RequestContext requestContext, Type controllerType)
    {
        return controllerType == null
            ? null
        : (IController)ninjectKernel.Get(controllerType);
    }

    private void AddBindings()
    {
        ninjectKernel.Bind<IServices>().To<Services>();
    }
}

PhotoController

public class PhotoController : Controller
{
    private IServices _services;

    public PhotoController(IServices services)
    {
        _services = services;
    }

    public ActionResult Show(int photoId)
    {
        Photo photo = _services.PhotoService.GetById(photoId);

        if (photo != null)
        {
            string currentProfile = "Profile1";

            _services.PhotoService.AddHit(photo, currentProfile);

            _services.Save();

            return View(photo);
        }
        else
        {
            // Add error message to layout
            TempData["message"] = "Photo not found!";
            return RedirectToAction("List");
        }
    }

    protected override void Dispose(bool disposing)
    {
        _services.Dispose();
        base.Dispose(disposing);
    }
}

我可以构建我的解决方案，并且它似乎工作正常。

我的问题是：

1. 我的实施中是否存在我遗漏的明显缺陷？
2. 我可以将其与 TDD 一起使用吗？通常我会看到对存储库的嘲笑，但我在上面没有使用它，这会导致问题吗？
3. 我是否正确且足够地使用 DI (Ninject)？

我是一名业余程序员，因此欢迎对我的代码提出任何意见和/或建议！

Answer 1

您已经有了大概的想法，但需要一段时间才能真正习惯依赖注入。我发现有许多可能需要改进的地方：

1. 您的 IServices 界面似乎没有必要。我更愿意让控制器通过其构造函数指定它需要哪些服务（IPhotoService 等），而不是使用像某种强类型服务定位器那样的 IServices 接口。
2. 我在那里看到了 DateTime.Now 吗？您将如何验证单元测试中的日期设置是否正确？如果您稍后决定支持多个时区怎么办？使用注入的日期服务来生成 CreatedDate 怎么样？
3. 有一个非常好的专门针对 MVC 的 Ninject 扩展。它负责插入 MVC 3 支持注入的各个点。它实现了诸如 NinjectControllerFactory 之类的东西。您所要做的就是使您的 Global 类扩展特定的基于 Ninject 的应用程序。
4. 我建议使用 NinjectModules 来设置绑定，而不是在 ControllerFactory 中设置它们。
5. 考虑使用按约定绑定，这样您就不必将每个服务显式绑定到其实现。

更新

Ninject MVC 扩展可以在此处找到。有关如何扩展 NinjectHttpApplication 的示例，请参阅 README 部分。此示例使用模块，您可以在此处阅读有关模块的更多信息。 （它们基本上只是放置绑定代码的地方，这样您就不会违反单一职责原则。）

关于基于约定的绑定，总体思路是让您的绑定代码扫描适当的程序集并自动绑定诸如根据命名约定将 IPhotoService 更改为 PhotoService。 此处还有另一个扩展可以帮助解决此类问题。有了它，您可以将这样的代码放入模块中：

Kernel.Scan(s =>
                {
                   s.From(assembly);
                   s.BindWithDefaultConventions();
                });

上面的代码将自动将给定程序集中的每个类绑定到它实现的遵循“默认”约定的任何接口（例如 Bind()。到()）。

更新 2

关于对整个请求使用相同的 DbContext，您可以执行以下操作（使用 MVC 扩展所需的 Ninject.Web.Common 库）：

Bind<SiteContext>().ToSelf().InRequestScope();

然后是任何依赖于上下文的服务Ninject 创建的将在请求中共享相同的实例。请注意，我个人使用过较短生命周期的上下文，所以我根本不知道如何强制在请求结束时处理上下文，但我确信它不会太难了。

Answer 2

IServices 和 Services 类型对我来说似乎是多余的。如果您删除它们并将控制器的构造函数更改为，

public PhotoController(IPhotoService photoService, IProfileService profileService)
{
  _photoService = photoService;
  _profileService = profileService;
}

那么它实际上取决于什么将会更加明显。此外，当您创建一个新控制器时，它实际上只需要 IProfileService，您可以只传递 IProfileService 而不是完整的 IService，从而为新控制器提供更轻的依赖关系。

Answer 3

我可以说你的服务看起来非常有存储库。仔细观察界面：

IQueryable<Photo> GetAll { get; }
Photo GetById(int photoId);
Guid AddPhoto(Guid profileId);

对我来说看起来非常像一个存储库。也许是因为这个例子相当简单，但我看到了如果在服务上添加用例逻辑就拥有服务的意义。而不是这些相当简单的 CRUD 操作。

您可能会说 EF DbSet 和 DbContext 是应用程序的存储库和工作单元……此时我们进入一个新区域，这在某种程度上超出了问题的范围。