服务方法参数传递最优方案：传模型还是传ID？

一、参数传递：模型实例 vs ID？ #

传递模型实例的利弊 #

• 优点：类型安全，能直接在服务层使用模型的属性和方法，避免重复查询数据库；同时能提前验证模型是否存在（比如用findOrFail），减少服务层的错误处理逻辑。
• 缺点：直接依赖Eloquent模型类，一旦未来更换ORM（比如从Eloquent换成Doctrine），服务层的方法签名必须修改，耦合度较高。

传递ID的利弊 #

• 优点：完全解耦具体的ORM实现，服务层只依赖基础类型（int），更换ORM时不需要修改方法签名；服务层可以根据自己的逻辑查询模型，灵活度更高。
• 缺点：服务层需要额外处理ID不存在的情况，可能重复查询数据库；丢失了类型安全，无法提前验证参数有效性。

最优折中方案：依赖抽象接口 #

如果既想保留类型安全，又要解耦ORM，可以定义抽象接口，让模型实现这些接口：

// 定义抽象接口
interface UserInterface
{
    public function getId(): int;
    // 定义服务层需要的其他方法
}

interface OrderInterface
{
    public function getId(): int;
}

// Eloquent模型实现接口
class User extends Model implements UserInterface
{
    public function getId(): int
    {
        return $this->id;
    }
}

// 修改Rating接口
interface Rating
{
    function create(UserInterface $user, UserInterface $ratedUser, Rating $rating, OrderInterface $order);
}

这样服务层依赖的是抽象接口而非具体模型，未来更换ORM时，只需要让新的模型实现对应的接口即可，服务层代码无需修改。

二、控制器代码优化 #

当前控制器存在几个问题：

1. 请求中的rating是数组，但代码直接取rating.rateid，无法正确处理批量评分的情况；
2. 获取rateduser时用了$request->id，但请求参数里是userid，参数名不匹配；
3. 直接new Order创建空订单，不符合业务逻辑，应该根据订单ID查询或关联获取。

优化后的控制器代码示例：

public function store(RateRequest $request)
{
    // 处理批量评分请求，转换为RatingData集合
    $ratingCollection = collect($request->input('rating'))->map(fn($item) => RatingData::from($item));

    // 获取当前登录用户
    $user = Auth::user();
    // 确保被评分用户存在，不存在则抛出404
    $ratedUser = User::findOrFail($request->input('userid'));
    // 根据业务逻辑获取订单，这里假设请求包含order_id
    $order = Order::findOrFail($request->input('order_id'));

    // 批量创建评分
    foreach ($ratingCollection as $rating) {
        $this->rating->create($user, $ratedUser, $rating, $order);
    }

    return response()->json(['message' => '评分创建成功']);
}

优化点：

• 用collect处理批量评分数据；
• 用findOrFail替代find，提前抛出404错误，避免后续逻辑处理空模型；
• 修正参数名匹配问题；
• 补充返回响应，符合RESTful规范。

三、关于未来更换ORM的兼容建议 #

除了依赖抽象接口，还可以在服务层内部封装模型查询逻辑，比如创建一个UserRepository接口，服务层依赖仓库接口而非直接查询模型：

interface UserRepositoryInterface
{
    public function findById(int $id): UserInterface;
}

// Eloquent实现
class EloquentUserRepository implements UserRepositoryInterface
{
    public function findById(int $id): UserInterface
    {
        return User::findOrFail($id);
    }
}

这样服务层如果需要查询用户，直接调用仓库接口，更换ORM时只需要实现新的仓库类即可，进一步降低耦合。

内容的提问来源于stack exchange，提问作者Temp