Laravel控制器中服务注入的正确方式：接口使用困惑

我正在Laravel应用中实现Repository Pattern（仓库模式），对服务与接口的搭配使用存在困惑。我创建了如下CRUD接口，用于规范实现CRUD方法的服务命名：

<?php

namespace App\Interfaces;

interface CRUD
{
  public function create(array $data);
  public function update(int $id, array $data);
  public function delete(string $ids);
};

目前我直接在控制器中注入GroupsService，示例代码如下：

<?php

namespace App\Http\Controllers\Cms;

use App\Http\Controllers\Controller;
use App\Http\Requests\GroupRequest;
use App\Models\Group;
use App\Services\GroupsService;
use Illuminate\Http\Request;

class GroupsController extends Controller
{
  private $service;

  public function __construct(GroupsService $service)
  {
    $this->service = $service;
  }

  public function store(GroupRequest $request)
  {
    $result = $this->service->create($request->all());
    return redirect()->back()->with('response', $result);
  }

  public function update(GroupRequest $request, $id)
  {
    $result = $this->service->update($id, $request->all());
    return redirect()->back()->with('response', $result);
  }

  public function destroy($groups_id)
  {
    $result = $this->service->delete($groups_id);
    return redirect()->back()->with('response', $result);
  }
}

GroupsService的实现代码如下：

<?php

namespace App\Services;

use App\Models\Group;
use App\Interfaces\CRUD;
use Exception;

class GroupsService implements CRUD
{
  public function listAll()
  {
    return Group::all();
  }

  public function create(array $data)
  {
    $modules_id = array_pop($data);

    $group = Group::create($data);
    $group->modules()->attach($modules_id);

    return cms_response(trans('cms.groups.success_create'));
  }

  public function update(int $id, array $data)
  {
    try {
      $modules_ids = $data['modules'];
      unset($data['modules']);
      $group = $this->__findOrFail($id);

      $group->update($data);
      $group->modules()->sync($modules_ids);

      return cms_response(trans('cms.groups.success_update'));
    } catch (\Throwable $th) {
      return cms_response($th->getMessage(), false, 400);
    }
  }


  public function delete(string $ids)
  {
    Group::whereIn('id', json_decode($ids))->delete();
    return cms_response(trans('cms.groups.success_delete'));
  }

  private function __findOrFail(int $id)
  {
    $group = Group::find($id);
    if ($group instanceof Group) {
      return $group;
    }
    throw new Exception(trans('cms.groups.error_not_found'));
  }
}

常见做法是为每个服务单独创建接口并在控制器中注入对应类型，这看起来有些冗余。请问当前直接注入服务的做法是否可行？是否应该注入CRUD接口，或是为每个服务创建专属接口？

1. 直接注入服务的做法完全可行 #

如果项目规模不大、业务逻辑短期内无大幅变动，直接注入具体服务类是最直接高效的选择。代码简单直观，无需额外的接口绑定操作。但缺点也很明确：后续若要替换服务实现（比如更换为兼容的新类），必须修改控制器的依赖声明，违反依赖倒置原则，不利于长期维护。

2. 注入通用CRUD接口的优缺点 #

注入CRUD接口需先在Laravel服务容器中绑定接口与具体实现，比如在AppServiceProvider的register方法中添加：

$this->app->bind(CRUD::class, GroupsService::class);

控制器构造方法改为：

public function __construct(CRUD $service)
{
  $this->service = $service;
}

这种方式的优点是统一了CRUD方法的规范，所有实现该接口的服务都遵循相同方法签名。但局限性很明显：

• 通用接口无法覆盖服务的专属方法（比如GroupsService中的listAll），控制器调用这些方法会失去类型提示，甚至引发错误。
• 若后续其他服务需实现CRUD，但方法签名与现有接口不匹配（比如delete方法参数为int而非string），通用接口要么修改破坏现有实现，要么失去作用。

3. 为每个服务创建专属接口是更推荐的长期方案 #

虽然看似冗余，但为每个服务创建专属接口（比如为GroupsService创建GroupServiceInterface）是更贴合业务的做法：

<?php

namespace App\Interfaces;

interface GroupServiceInterface extends CRUD
{
  public function listAll();
}

让GroupsService实现该专属接口：

class GroupsService implements GroupServiceInterface
{
  // 保持原有实现
}

最后在服务容器绑定并注入：

// AppServiceProvider中绑定
$this->app->bind(GroupServiceInterface::class, GroupsService::class);

// 控制器中注入
public function __construct(GroupServiceInterface $service)
{
  $this->service = $service;
}

这种方式的优势：

• 既继承了通用CRUD的规范，又能扩展专属方法，控制器调用时可获得完整类型提示。
• 遵循依赖倒置原则，后续替换服务实现时，只要新类实现同一接口，控制器无需修改代码，仅调整服务容器绑定即可。
• 便于单元测试，可轻松创建接口的Mock类模拟服务行为，无需依赖真实数据库操作。

总结 #

• 小型项目/短期迭代：直接注入具体服务，快速落地。
• 多个服务有完全一致的CRUD需求：可尝试通用CRUD接口，但需注意专属方法的兼容问题。
• 中大型项目/长期维护：优先为每个服务创建专属接口，兼顾规范与灵活性。

内容的提问来源于stack exchange，提问作者Bernardo Benini Fantin