如果是对一张表的操作，有必要在对应的model里面写一个接口吗

Question

比如说我的controller是这样的

use App\Model\User;
class UserController extends BaseController
{
    protected $model;
    public function __construct(User $user)
    {
        $this->model = $user;
    }
    public function updatePhone(Request $request)
    {
        $id = $request->id;
        $phone = $request->phone;
    }
}

函数updatePhone的用途就是更新用户的手机。在这一点上我不知道怎么办好，我可以这样写

$res = $this->model->where('id',$id)->update('phone',$phone)

因为这个逻辑只是更新用户的手机，只是对user表的单表操作，一般的框架里都有现成的对单表curd的函数。
我还可以在controller里这样写

$res = $this->model->updatePhone($id, $phone);

然后在user model里面加一个接口

public function updatePhone($id,$phone)
{
    return $this->where('id',$id)->update('phone',$phone);
}

不知道这两种写法哪一个好一点？

Answer 1

MVC 分层的好处就是隔离，Controller 负责处理请求，Model 负责和数据库进行 CURD 交互，View 负责显示数据。如果以后业务扩展，User 表新增 email、nickname 等也会发生更新的字段，这时候再在 Model 中写 updateEmail 和 uodateNickname，代码的复用性就很低了。所以我觉得放到 Controller 中处理，更新指定的字段为指定的值，会更灵活。

Answer 2

放在model吧 你可以把代码写成可以更新多个字段的 自由控制传参 像2楼上说的 每次都写一个肯定不好 但是写一个可以更新一个或多个字段不是很好嘛 又能满足规范 又不会有重复的工作

Answer 3

如果这段代码经常被其他controller用到，封装到model里；只是当前业务逻辑用到，写到controller里。

Answer 4

建议还是放到Model中，将业务逻辑和数据操作分离开，这样项目大了后调试或者扩展都会很清晰