c# - 继承/泛型基础知识。如何实现 DRY 的代码结构

Question

为了今天让我的代码变干，我想做以下事情。 (我不知道这是否是最好的方法，但它似乎比拥有一个不断增加的代码库要好，如果我想在整个站点上更改某些内容，我需要不断更新多种方法)

我对继承的了解是可怕的。因为我从来没有质疑过我使用的任何代码/库，而且我以前从来没有真正尝试过写过这样的东西，但我想学习......希望这将是我启蒙的一天 :P

我的问题:

假设我有一个像这样的添加方法(在我所有的 Controller 中):

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 模板/EF 生成。

    ViewModels, Repositories, (Standard) EF Models

我认为每个页面都需要一个 ModelSpecfic 基础 Controller (可以使用 T4 完成)，它继承自包含基本 CRUD 功能的自定义 ControllerBase 类。这样我就可以为每个 Controller 定制代码，而且我的代码库会更干净、更小，而且如果我需要重新生成基本文件也不会受到影响

我不太明白如何按照我的需要来实现某些东西。到目前为止我所了解的是我需要拥有我的存储库，并且 View 模型也从一个基础继承并且以某种方式在 [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);
            }       
        }
    }

Answer 1

这是对您的要求的简单通用解释:

// 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)创建接口(interface)，并将它们用作通用约束。
2. 您可能需要一个通用的存储库接口(interface)和基本实现(这样您就不必为每个存储库单独编写代码，也不必将类似的逻辑从一个存储库复制到另一个存储库)。
3. 考虑使用 Inversion of Control container and dependency injection .例如，与其让 Controller 处理存储库实例的创建，不如将其设为属性并从构造函数中设置它。然后，您可以使用您选择的 IoC(如 Ninject 或 Autofac)并注册具体实现，它将管理依赖项和 Controller 本身的创建和生命周期。