继承/泛型基础。如何实现DRY的代码结构

时间:2012-05-10 08:04:52

标签: c# asp.net-mvc generics inheritance

今天尝试干掉我的代码我想做以下事情。 (我不知道它是否是最好的方式,但似乎比不断增加的代码库更好,我不断需要更新多个方法,如果我想在整个网站上更改某些内容)

我对继承的了解是可怕的。因为我从未质疑过我使用过的任何代码/库,并且我以前从未真正尝试过这样的事情,但我想学习......希望这将成为我启蒙的日子:P

我的问题:

说我有一个添加方法(在我的所有控制器中),如下所示:

public ActionResult Add(VM_Down_Time_Capture viewModel)
        {
            using (Down_Time_CaptureRepository repository = new Down_Time_CaptureRepository())
            {
                if (!ModelState.IsValid)
                    return ReturnValidationFailure(ViewData.ModelState.Values);

                Down_Time_Capture model = new Down_Time_Capture();
                model.InjectFrom(viewModel);

                string mserMsg = repository.Add(model, User.Identity.Name);

                if (!string.IsNullOrEmpty(mserMsg))
                    return ReturnCustomValidationFailure(Server.HtmlEncode(mserMsg));

                repository.Save();

                return Json("Added successfully.", JsonRequestBehavior.AllowGet);
            }
        }

目前我也有以下内容。

由T4 Templates / EF生成。

    ViewModels, Repositories, (Standard) EF Models

我认为我需要每个页面的ModelSpecfic基本控制器(可以使用T4完成),它继承自包含基本CRUD功能的自定义ControllerBase类。这样我就可以为每个控制器提供自定义代码,而我的代码库将更清晰。较小的&我不需要重新生成基础文件

我不太明白如何按照我需要的方式实现某些内容。到目前为止我所理解的是,需要拥有我的存储库,并且查看模型继承自基础,并以某种方式在[B]中指定我正在使用哪些...但至于如何做到这一点我不这样做知道

例如(这是我最好的尝试,而不是我的实际代码,非常hacky,因为我非常困惑:S)

    public class Down_Time_CaptureController : Down_Time_CaptureBase
    {
     //[A]
    }

    //Generated by T4
    public class Down_Time_CaptureBase: ControllerBase
    {
      //[B]
       public override EntityObject CreateNewModel()
       {
          return new Down_Time_Capture();
       }

       public override Base_Repository CreateNewRepository()
       {
          return new Down_Time_CaptureRepository();
       }

       public override Base_ViewModel CreateNewViewModel()
       {
          return new VM_Down_Time_Capture();
       }

      //how would i go about specifying which repository & model & view model to use
      //although i expect it to be something to what i did here above
      //and how would i go about calling the new generic add method (but in context of this controller)?

    }

    //coded once
    public abstract class ControllerBase: Controller
    {
        //[C]
        //make abstract so i have to override it
        public abstract Base_Controller CreateNewModel();
        public abstract Base_Controller CreateNewRepository();
        public abstract Base_Controller CreateNewViewModel();

        //I'm assuming my generified add method would go in here  
        public virtual ActionResult Add(Base_ViewModel viewModel)
        {
           using (Base_Repository repository = CreateRepository())
           {
                   if (!ModelState.IsValid)
                       return ReturnValidationFailure(ViewData.ModelState.Values);

                   EntityObject model = CreateNewModel();
                   model.InjectFrom(viewModel);

                   string mserMsg = repository.Add(model, User.Identity.Name);

                   if (!string.IsNullOrEmpty(mserMsg))
                       return ReturnCustomValidationFailure(Server.HtmlEncode(mserMsg));

                   repository.Save();

                   return Json("Added successfully.", JsonRequestBehavior.AllowGet);
            }       
        }
    }

1 个答案:

答案 0 :(得分:3)

以下是您要求的简单通用解释:

// concrete controller implementation
public class Down_Time_CaptureController: ControllerBase<Down_Time_Capture, VM_Down_Time_Capture, Down_Time_CaptureRepository>
{
}

// generic controller base
public abstract class ControllerBase<TModel, TViewModel, TRepository>: Controller
        where TModel : Base_Model, new()
        where TViewModel : Base_ViewModel, new()
        where TRepository : Base_Repository, new()
{
    protected virtual TModel CreateNewModel()
    {
           return (TModel)Activator.CreateInstance<TModel>();

    }

    protected virtual TRepository CreateNewRepository()
    {
           return (TRepository)Activator.CreateInstance<TRepository>();
    }

    protected virtual TViewModel CreateNewViewModel()
    {
            return (TViewModel)Activator.CreateInstance<TViewModel>();
    }

    //I'm assuming my generified add method would go in here  
    public virtual ActionResult Add(TViewModel viewModel)
    {
       using (var repository = CreateRepository())
       {
               if (!ModelState.IsValid)
                   return ReturnValidationFailure(ViewData.ModelState.Values);

               var model = CreateNewModel();
               model.InjectFrom(viewModel);

               string mserMsg = repository.Add(model, User.Identity.Name);

               if (!string.IsNullOrEmpty(mserMsg))
                   return ReturnCustomValidationFailure(Server.HtmlEncode(mserMsg));

               repository.Save();

               return Json("Added successfully.", JsonRequestBehavior.AllowGet);
        }       
    }
}

一些注意事项:

  1. 您可能希望为三种类型(Model,ViewModel,Repository)创建接口,并将它们用作通用约束。
  2. 您可能需要一个通用的Repository接口和基本实现(因此您不必独立编写每个存储库,并将类似的逻辑从一个复制到另一个)。
  3. 考虑使用Inversion of Control container and dependency injection。例如,不是让控制器处理创建存储库的实例,而是将其作为属性并从构造函数中设置它。然后,您可以使用您选择的IoC(如Ninject或Autofac)并注册具体实现,它将管理依赖项和控制器本身的创建和生命周期。