小明是一名萌新php程序员，受雇在一家传统的企业crm开发。
一天老板提出一个新的需求

业务部的业务员下了一张订单的时候，需要钉钉消息提醒物流部的同事去处理

小明收到需求，然后打开了下单的model代码如下

<?php
class Order {
    public function create($params)
    {
        if (user == '业务员') {
            //todo 查询商品
            //todo 计算价格
            //todo 插入数据库
          	......
        }
    }
}

然后小明很快就完成了老板的需求

<?php
class Order {
    public function create($params)
    {
        if (user == '业务员') {
            //todo 查询商品
            //todo 计算价格
            //todo 插入数据库
            ......
            //todo 发送钉钉消息给物流部同事
        }
    }
}

这一段代码就这样，每次新增需求都不挺的网里面加逻辑，最后变成了一堆屎山。完全部分能复用
​

现在我们来分析这段代码的问题

一. 违反了单一原则，再一个方法里做了n件事情

这个方法里做了几件事情

  1. 权限检查
  2. 发送消息
  3. 插入数据库
  4. …….

这样做的坏处就是以后如果我要给运营权限去添加订单时候，这里就需要添加if else 然后越来越多的if else 代码就更难维护了

二. 把权限验证放在model层

权限验证我们应该放在路由访问前后或者控制器里，这样当我们复用该代码时候就不用担心权限问题了

三. 把所有业务逻辑全放在model层

model层应该是一栋建筑的地基，功能应该尽量简单且一旦完成了就不轻易改动的，如果有新的需求我们应该新增方法，而不提倡修改原来的代码。这样既不破坏原来的代码，也能更方便的拓展新功能。

以上是本人在写代码过程中感悟，我仅一家之言，有不对的地方欢迎指正。

版权声明：本文为echoou原创文章，遵循 CC 4.0 BY-SA 版权协议，转载请附上原文出处链接和本声明。
本文链接：https://www.cnblogs.com/echoou/p/15216535.html