如何更优雅的给控制器 “减负”

MVC 是一个非常伟大的概念,但是最近我发现一个现象,包括我自己,我们在最开始接触 MVC 概念时,我们非常严谨地贯彻这种分层思想,Controller 层处理业务逻辑,而 Model 层只是单纯的处理数据 I/O。但是,伴随着我们项目体量的逐渐增大,控制器的负担也越来越大。这样一来会有一个非常明显的弊端,当我们在定位 BUG 时,我们总是需要对照着代码查看许久。除此之外,彼此的业务代码并没有太好的关联,这使得我们想要抽出一个 Service 时就显得极为困难。
因此,是时候给我们的控制器做一些 “减负” 了。这里的减负并不意味着会违背 MVC 的设计思想,而是把我们的控制器层的业务适当的分给其他部分。
有使用过一些主流框架的朋友应该都知道,其实很多框架都给 Controller 层做了一些 “减负” 的工作,比如 KOA 里面的 middleware,抑或是 Laravel 里面的 Event,Policy 等。
但是事与愿违,即使这些框架提供了这些帮助,但是许多人在实际项目中使用到的却很少,当然,也有可能是我接触到的代码不够多。究其原因,窃以为尚未意识到这种理念的重要性。因此,我在这里总结了自己这些年来 “减负” 的一些经验,同时我也会配合一些代码予以解释。当然,我所写的未必全对,因此希望有幸看到的读者能保持自己的独立性。

Model 分流#

我们在写代码时往往会有这样一种场景,我们需要对从 Model 取出来的数据进行加工,但是,加工数据的部分我们经常会放到控制器,毕竟这属于业务逻辑,确实无可厚非,如下伪代码所示:

// controller
public function userList()
{
    $users = array_map(function ($user){
//        这里会对我们的代码进行业务逻辑的加工
        $user['created_at'] = date('Y-m-d' , $user['created_at']);
        // ...
        return $user;
    } ,$model->availableList());
}

// model
public function availableList()
{
    // 从数据库取数据  
    return $users;
}

但是我们有没有考虑过这样一个问题,当我们同事来接手我们项目或者我们 debug 时,我们需要了解的代码量非常大,特别是涉及到一些数据加工的格式问题,我们并不需要关心。或者换个角度,当我们遇到数据加工的 bug 时,我们能第一联想到这段代码是放在 Model 层时,是不是更加快捷呢?

// controller
public function userList()
{
    $users = $model->availableList();
//    处理其他逻辑
}

// model
public function availableList()
{
    // 从数据库取数据 $users
    return array_map(function ($user){
//        这里会对我们的代码进行业务逻辑的加工
        $user['created_at'] = date('Y-m-d' , $user['created_at']);
        // ...
        return $user;
    } , $users);
}

如上代码所示,在 Model 层中已经帮我们封装好了我们所需要的数据以及其格式,当我们在浏览他人代码时,我们并不需要关心他的格式是怎么加工的,我们只需要根据他对方法的命名就能知道是获取的怎样的数据。

分离 Controller#

在写具体的方法之前,我想要阐述的一点是,我们在写代码的时候需要保持一定的前瞻性。什么意思?虽然我们的大部分工作都是跟具体的业务逻辑打交道,但是我们经常会发现总会有重复的工作,那么有的人会直接把这段代码复制。但是,在我们复制之前,我们是不是可以问自己这样一个问题:如果接下来还有类似的业务,我们还是复制吗?我们是不是可以把这段基于我们项目的代码抽象出一个 Service 呢?
我举个例子,比如一个网站,可能会有打赏功能,可能也有付费阅读功能,我们不难发现,这两种付费有着相似的地方,比如创建本平台订单系统的业务逻辑,再比如回掉时可能存在的相同业务逻辑,所以这段代码我们是不是可以以一个 trait 的形式做一个 Service

trait PayService
{
    private $_callback = null;

    public function createOrder()
    {
//        处理你的业务逻辑,配置调用三方支付接口的参数等
    }

    public function callback()
    {
//        处理共同的回调逻辑

        $this->handler();
    }
}

这里我们保留了一个 handler 方法来处理每个功能独有的业务逻辑,至此,我们就可以非常方便的扩展我们的支付服务了。

给控制器减负的方法还有很多,比如对我们加工数据的部分,其实我们也可以不放到 Model,我们也可以单独开辟一层来处理我们的数据加工。让控制器变得清晰明朗,每个人阅读代码时都能非常快速的了解控制器下的每个方法在处理什么业务逻辑。这便是我们给控制器减负的目的。
我很喜欢「包」的概念和设计思想,当我们在使用包时,不仅仅意味着方便,更重要的是,他做为一个独立的 “组件” 存在于我们的代码逻辑中,与我们项目的代码不存在任何的耦合,同时我们也无需知道他的具体实现。

欢迎关注我的公众号,每周至少一篇比较有深度的原创文章:

如何更优雅的给控制器 “减负”

文章首发地址:我的博客

本作品采用《CC 协议》,转载必须注明作者和本文链接
Nine
本帖由系统于 7年前 自动加精
《L01 基础入门》
我们将带你从零开发一个项目并部署到线上,本课程教授 Web 开发中专业、实用的技能,如 Git 工作流、Laravel Mix 前端工作流等。
《G01 Go 实战入门》
从零开始带你一步步开发一个 Go 博客项目,让你在最短的时间内学会使用 Go 进行编码。项目结构很大程度上参考了 Laravel。
讨论数量: 14

深有体会的是,第三方登录,普通登录,注册登等这些方式都是有相同的后置操作,我当时做的是将这些成功后的操作封装在 Model 中创建了一个 successLogin() 方法。

7年前 评论

我们公司,增加了服务层,和数据库打交道就在服务层中,不用 model, 直接用 DB::table ()->

7年前 评论

@xuanjiang1985 增加服务层不意味着要放弃 model 啊,eloquent 那么强大的功能不用可惜了

7年前 评论

@xuanjiang1985 orm 是很强大的功能,也是使得 laravel 这么慢的原因,如果放弃 orm, 推荐使用 lumen 框架

7年前 评论

在 laravel 开发中可以说不存在重复代码这种概念,任何多余的架构设计都是耍流氓!我不是针对谁,我就是针对的仓库层~
laravel 其实使用的并不是 mvc 架构,而是 MVCRPMMNRELH 层架构。当然原来还有 service 层.

Request (表单验证), Policy (权限验证),Mail (邮件服务), Middleware, notification, resource event 层 尤其是事件层可以稀释大量的控制器的代码,用户不关心的副作用,基本上都可以丢到事件层,监听层继承 queue 可以集快增加响应速度

7年前 评论

我觉得时刻记得请求就是获取资源这个概念,而 controller 只是检查输入并调取资源,所以 controller 里代码会很少的,我觉得 controller 层不需要关注资源结构之类

7年前 评论

楼主的意思如果我没理解错的话应该是 Repository
https://github.com/zhaohehe/z-repository

7年前 评论
Jourdon

写的不错,把 窃以为 改成 妾以为 就完美了!

7年前 评论

@Max
@ykenny
其实我写这篇文章并不是针对 Laravel 来的,而是说不管我们用什么框架和什么语言,当我们在处理业务逻辑时应该保持抽象和解耦的理念。

7年前 评论

可以看一哈这个视频,里面的做法也挺好的 https://www.youtube.com/watch?v=MF0jFKvS4S...

6年前 评论

@young 好的,感谢分享?

6年前 评论

各位.我之前也发现这个问题,所以,我把项目分成了 servces,models,repository,controller, 等层.并且做了几个项目.但是,我发现这么分代码量增加了很多,增加很多不必要的操作.后来我发现国外的 coder 和国内的有很多的分别.他们的代码短小精悍.我发现我虽然写了几个项目,但是对 laravel 并不十分了解.看了 laravel 的文档,我现在框架本身带了很多东西没用到.使用框 架自带的 controller,model,transformer,request,eloqents,可以很好的处理问题了.另外看到很多人说 eloqents 不好用的.我以前也这么认为,现在我改变了看法.分享相关的视频给大家看看.https://www.youtube.com/channel/UC0dhWGm6PI0piTDbhtGlirQ/videos,希望与大家共同进步

5年前 评论

你代码数量总共有这么多,有 bug 时,你不都要去一个个定位?难道你放 comtroller 里要一行行去定位,那你放 model 里就不要去定位吗?

5年前 评论