Question

规则是控制器不应具有业务逻辑，而应将其委托给服务。但是，这样做时，我们无法处理所有可能的情况并返回适当的HTTP响应。

让我们看一个例子。假设我们正在建立某种社交网络，并且需要创建一个端点来对帖子进行评分（喜欢或不喜欢）。

首先让我们看一个将逻辑委托给服务的示例，这是我们的控制器操作：

public IActionResult Rate(long postId, RatingType ratingType)
{
    var user = GetCurrentUser();
    PostRating newPostRating = _postsService.Rate(postId, ratingType, user);
    return Created(newPostRating);
}

您看到这个有问题吗？如果没有指定ID的帖子该怎么办，我们将如何返回未找到的回复？如果用户无权对帖子进行评分，我们将如何返回被禁止的响应？

PostsService.Rate只能返回一个新的PostRating，但是其他情况呢？好吧，我们可以抛出一个异常，我们需要创建很多自定义异常，以便我们可以将它们映射到适当的HTTP响应。我不喜欢为此使用异常，我认为有一种更好的方法来处理这些情况而不是异常。因为我认为发布帖子不存在并且用户没有权限的情况根本不是例外，所以它们只是正常的情况，就像成功地对帖子进行评分一样。

我建议的是改为在控制器中处理该逻辑。因为我认为，无论如何，这应该是控制器的责任，在提交操作之前检查所有权限。所以这就是我要做的：

public IActionResult Rate(long postId, RatingType ratingType)
{
    var user = GetCurrentUser();
    var post = _postsRepository.GetByIdWithRatings(postId);

    if (post == null)
        return NotFound();

    if (!_permissionService.CanRate(user, post))
        return Forbidden();

    PostRating newPostRating = new PostRating 
    {
        Post = post,
        Author = user,
        Type = ratingType
    };

    _postRatingsRepository.Save(newPostRating);

    return Created(newPostRating);
}

我认为这是应该这样做的方式，但是我敢打赌，有人会说这对于控制器来说是太多逻辑了，或者您不应该在其中使用存储库。

如果您不喜欢在控制器中使用存储库，那么您会在哪里放置获取或保存帖子的方法呢？在服役？因此会有PostsService.GetByIdWithRatings和PostsService.Save除了调用PostsRepository.GetByIdWithRatings和PostsRepository.Save之外别无其他。这是不必要的，只会导致样板代码。

更新：也许有人会说使用PostsService检查权限，然后调用PostsService.Rate。这很糟糕，因为它涉及到数据库的更多不必要的旅行。例如，可能是这样的：

public IActionResult Rate(long postId, RatingType ratingType)
{
    var user = GetCurrentUser();

    if(_postsService.Exists(postId))
         return NotFound();

    if(!_postsService.CanUserRate(user, postId))        
         return Forbidden();

    PostRating newPostRating = _postsService.Rate(postId, ratingType, user);
    return Created(newPostRating);
}

我什至需要进一步解释为什么这不好吗？

Answer 1

有很多方法可以解决此问题，但是最接近“最佳实践”方法的方法可能是使用结果类。例如，如果您的服务方法创建了一个等级，然后返回它创建的该等级，则您将返回一个封装该等级以及其他相关信息（例如成功状态，错误消息，如果有的话）的对象。

public class RateResult
{
    public bool Succeeded { get; internal set; }
    public PostRating PostRating { get; internal set; }
    public string[] Errors { get; internal set; }
}

然后，您的控制器代码将变为：

public IActionResult Rate(long postId, RatingType ratingType)
{
    var user = GetCurrentUser();
    var result = _postsService.Rate(postId, ratingType, user);
    if (result.Succeeded)
    {
        return Created(result.PostRating);
    }
    else
    {
        // handle errors
    }
}

将业务逻辑与控制器分离的最佳方法

1 个答案: