3

我编辑了我的整个问题，所以不要怀疑:)

好吧，我想要一个ActionResult接受域模型数据和一些附加参数，即用于分页列表的页面索引和页面大小。它根据 Web 请求的类型（是否为 ajax 请求）自行决定是否返回 PartialViewResult 或 ViewResult。

引用的数据应使用 IMappingService 自动映射，该服务负责将任何域模型数据转换为视图模型。为简单起见，MappingService 使用 AutoMapper。

映射动作结果：

public abstract class MappingActionResult : ActionResult
{
    public static IMappingService MappingService;
}

BaseHybridViewResult：

public abstract class BaseHybridViewResult : MappingActionResult
{
    public const string defaultViewName = "Grid";

    public string ViewNameForAjaxRequest { get; set; }
    public object ViewModel { get; set; }

    public override void ExecuteResult(ControllerContext context)
    {
        if (context == null) throw new ArgumentNullException("context");
        var usePartial = ShouldUsePartial(context);
        ActionResult res = GetInnerViewResult(usePartial);

        res.ExecuteResult(context);
    }

    private ActionResult GetInnerViewResult(bool usePartial)
    {
        ViewDataDictionary viewDataDictionary = new ViewDataDictionary(ViewModel);
        if (String.IsNullOrEmpty(ViewNameForAjaxRequest))
        {
            ViewNameForAjaxRequest = defaultViewName;
        }

        if (usePartial)
        {
            return new PartialViewResult { ViewData = viewDataDictionary, ViewName = ViewNameForAjaxRequest };
        }

        return new ViewResult { ViewData = viewDataDictionary };
    }

    private static bool ShouldUsePartial(ControllerContext context)
    {
        return context.HttpContext.Request.IsAjaxRequest();
    }
}

AutoMappedHybridViewResult:

public class AutoMappedHybridViewResult<TSourceElement, TDestinationElement> : BaseHybridViewResult
{
    public AutoMappedHybridViewResult(PagedList<TSourceElement> pagedList)
    {
        ViewModel = MappingService.MapToViewModelPagedList<TSourceElement, TDestinationElement>(pagedList);
    }

    public AutoMappedHybridViewResult(PagedList<TSourceElement> pagedList, string viewNameForAjaxRequest)
    {
        ViewNameForAjaxRequest = viewNameForAjaxRequest;
        ViewModel = MappingService.MapToViewModelPagedList<TSourceElement, TDestinationElement>(pagedList);
    }

    public AutoMappedHybridViewResult(TSourceElement model)
    {
        ViewModel = MappingService.Map<TSourceElement, TDestinationElement>(model);
    }

    public AutoMappedHybridViewResult(TSourceElement model, string viewNameForAjaxRequest)
    {
        ViewNameForAjaxRequest = viewNameForAjaxRequest;
        ViewModel = MappingService.Map<TSourceElement, TDestinationElement>(model);
    }
}

在控制器中的用法：

public ActionResult Index(int page = 1)
{
    return new AutoMappedHybridViewResult<TeamEmployee, TeamEmployeeForm>(_teamEmployeeRepository.GetPagedEmployees(page, PageSize));
}

所以你可以看到它IMappingService是隐藏的。控制器不应该知道关于IMappingService接口的任何信息，什么时候AutoMappedHybridViewResult使用。

是适当MappingActionResult的static IMappingServer还是我违反了DI原则？

4

2 回答 2

1

我认为更好的设计是拥有一个依赖于 IMappingService 的 ViewResultFactory，然后您可以将其注入到您的控制器中。然后你这样称呼它：

public class MyController : Controller
{
    IViewResultFactory _viewResultFactory;
    ITeamEmployeeRepository _teamEmployeeRepository;

    public MyController(IViewResultFactory viewResultFactory)
    {
        _viewResultFactory = viewResultFactory;
    }

    public ActionResult MyAction(int page, int pageSize)
    {
        return
            _viewResultFactory.GetResult<TeamEmployee, TeamEmployeeForm>(
                _teamEmployeeRepository.GetPagedEmployees(page, pageSize));
    }
}

实现应该是这样的（您需要为每个 HybridViewResult 构造函数创建重载）：

public HybridViewResult<TSourceElement, TDestinationElement> GetResult<TSourceElement, TDestinationElement>(PagedList<TSourceElement> pagedList)
{
    return new HybridViewResult<TSourceElement, TDestinationElement>(_mappingService, pagedList);
}

这样你就可以从你的控制器中隐藏实现，并且你不必依赖容器。

于 2010-09-30T22:29:57.553 回答
0

有几个不同的点可以注入 IMappingService。 http://codeclimber.net.nz/archive/2009/04/08/13-asp.net-mvc-extensibility-points-you-have-to-know.aspx是一个很好的站点，可以帮助您选择适当的可扩展性.NET MVC 的要点。

如果您想坚持将此功能作为派生的 ActionResult，那么我认为您可以根据需要将依赖项放在 ActionInvoker 中，但 Controller 对我来说更有意义。如果您不想在 Controller 中使用 IMappingService，则始终可以将其包装在 HybridViewResultFactory 中，并在 Controller 中访问该对象。在这种情况下，您的快捷方法如下所示：

public HybridViewResult<TSourceElement, TDestinationElement> AutoMappedHybridView<TSourceElement,TDestinationElement>(PagedList<TSourceElement> pagedList, string viewNameForAjaxRequest)
{
    HybridViewResultFactory.Create<TSourceElement, TDestinationElement>(pagedList, viewNameForAjaxRequest);
 }

等等

我不确定为什么需要使用 ActionResult，但如果没有明确需要的原因，您可以创建一个 HybridViewModel 类和一个注入映射服务依赖项的 HybridViewModelBinder 类。

我假设您想使用构造函数注入，但如果您的 UI 程序集中有 StructureMap 依赖项，则可以访问静态依赖项解析器类（如 Clowers 所说）。

如果我理解你为什么使用 ActionResult，这个问题会更容易给出明确的答案。

似乎您正在使用操作结果来处理两个不一定总是一起使用的功能，并且可以单独使用。此外，没有明确的迹象表明它需要在 ActionResult 中。

据推测，您可以 (a) 将 Automapper 功能用于除 html (ViewResult) 输出之外的结果，并且 (b) 您可以利用自动检测 ajax 请求的功能而无需自动映射模型。

在我看来，视图模型的自动映射可用于将视图模型直接注入控制器操作，从而消除控制器对 IMappingService 的依赖。您需要的是一个与您的 IMappingService 一起注入的 ModelBinder 类（我假设它的实现包含存储库或数据存储类型依赖项）。

这是一篇很好的文章，解释了如何利用模型绑定器：http: //odetocode.com/blogs/scott/archive/2009/04/27/6-tips-for-asp-net-mvc-model-binding.aspx。

然后可以覆盖需要Automapped的类中的DefaultModelBinder如下：

   public ActionResult DoItLikeThis([AutoMap(typeof(MyDomainModelClass))]MyViewModelClass viewModel){
               //controller action logic
   }

现在，关于 HybridViewResult，我建议您改用操作过滤器来处理这个问题。因此，您可以只使用 ActionResult 或 ViewResultBase 作为您的操作方法的 Result 类型，并用操作过滤器装饰它，即：

   [AutoSelectViewResult]
   public ViewResultBase AndDoThisLikeSo(){
               //controller action logic
   }

我认为总体而言，这将是一个比将这两个功能耦合到 ActionResult 更好的解决方案。

于 2010-10-03T06:39:11.343 回答