10

我有一个设计精良的架构,其中控制器转到访问与数据库通信的存储库的服务。

因此,控制器中的逻辑保持在最低限度,但我仍然有非常微妙的代码片段可以执行一些任务,例如

  • 验证模型
  • 安排动作方法参数
  • 使用这些参数调用一些服务,如果模型现在无效,可能会验证结果并返回视图
  • 最后从服务的结果中产生一个模型,并返回它。

根据服务返回的“状态”,一些较长的案例会做不同的事情。

这里有几个例子:

[HttpPost]
[AjaxOnly]
[Authorize]
public JsonResult Preview(string input)
{
    LinkResult parsed = linkService.ParseUserInput(input);
    if (parsed.Result == LinkParseResult.Used)
    {
        long? postId = parsed.Link.PostId;
        if (postId.HasValue)
        {
            Post post = postService.GetById(postId.Value, false);
            return Json(new
            {
                faulted = "used",
                link = DetailsRoute(post),
                id = postId
            });
        }
        else
        {
            return Json(new { faulted = "invalid" });
        }
    }
    else if (parsed.Result == LinkParseResult.Invalid)
    {
        return Json(new { faulted = "invalid" });
    }
    else
    {
        Link link = parsed.Link;
        if (link.Description != null && link.Description.Length > 200)
        {
            link.Description = link.Description.Substring(0, 200);
        }
        return AjaxView(link);
    }
}

和(Post来自域,PostModel是视图模型)

private PostModel PostModelConverter(Post post)
{
    Link link = post.Link;
    if (link == null)
    {
        throw new ArgumentException("post.Link can't be null");
    }
    if (link.Type == LinkType.Html)
    {
        return new PostedLinkModel
        {
            Description = link.Description,
            PictureUrl = link.Picture,
            PostId = post.Id,
            PostSlug = postService.GetTitleSlug(post),
            Timestamp = post.Created,
            Title = link.Title,
            UserMessage = post.UserMessage,
            UserDisplayName = post.User.DisplayName
        };
    }
    else if (link.Type == LinkType.Image)
    {
        return new PostedImageModel
        {
            PictureUrl = link.Picture,
            PostId = post.Id,
            PostSlug = postService.GetTitleSlug(post),
            Timestamp = post.Created,
            UserMessage = post.UserMessage,
            UserDisplayName = post.User.DisplayName
        };
    }
    return null;
}

这就提出了一个问题,即视图模型是否真的应该作为规则出现在 Web 项目中,或者它们实际上可能是域或其他项目的一部分。

我不确定我可以对预览操作做很多事情,除了可能使用接收链接并截断描述的 PreviewModel 之外,但这会节省两行。

模型转换器可能应该在其他地方,但我不知道应该在哪里。

想到的另一点是我是否应该使用partial关键字拆分此控制器(将其用于自动生成的类以外的其他东西是一种不好的做法吗?),或者根据请求的操作添加使用不同控制器的路由或正在使用什么http方法,通常的处理方法是什么?

4

3 回答 3

5

这已经被问过好几次了:控制器中的业务逻辑我应该

哪里将控制器业务逻辑放在 MVC3
保持控制器精简

以及其他地方的文章:
ASP MVC 最佳实践 - Skinny Controllers
Keep Controllers Thin

社区似乎达成了很好的共识,即这种逻辑属于控制器之外。一般在模型(或 ViewModel)中,但在业务层的某个地方。

最后一点,partials不鼓励使用非自动生成的代码。如果拆分事物有意义,请这样做。想想你拆分它的原因是什么。这将是一个个案。

于 2012-07-31T20:43:37.747 回答
2
private PostModel PostModelConverter(Post post)
{
    Link link = post.Link;
    if (link == null)
    {
        throw new ArgumentException("post.Link can't be null");
    }
    if (link.Type == LinkType.Html)
    {
        var model = AutoMapper.Map<PostedLinkModel>(post);
        model.PostSlug = postService.GetTitleSlug(post);
        return model;
    }
    else if (link.Type == LinkType.Image)
    {
        var model = AutoMapper.Map<PostedImageModel>(post);
        model.PostSlug = postService.GetTitleSlug(post);
        return model;
    }
    return null;
}

http://www.viddler.com/v/b568679c

于 2012-07-31T20:42:51.030 回答
0

控制器将不包含任何域逻辑

控制器应仅负责:

验证输入

调用模型来准备视图

返回视图或重定向到另一个操作

如果您正在做任何其他事情,那么您是在错误的地方做的,而是您在 Controller 中所做的模型责任。

如果您遵循此规则,您的操作方法将不会超过 20 - 25 行代码。Ian Cooper 有一篇很棒的帖子Skinny Controller Fat Model,请阅读。

于 2012-08-01T09:25:11.980 回答