在服务层或模型中将插入和获取查询逻辑放在哪里?

时间:2018-12-12 18:43:01

标签: php laravel model-view-controller design-patterns

我想使用服务层。但是有一些问题。我将提供示例并进行讨论。

假设在控制器存储功能中,我编写了错误的代码,例如在其中进行验证,在其中也有模型,并为该模型用户的输入提供属性并将其存储。基本上,我在控制器中拥有一切,从而导致胖控制器。现在,我们可以做两件事来解决这个问题。

1)将验证逻辑移至验证类(简单),然后在其中创建新的模型实例并为其设置属性,并将其存储在数据库中(该逻辑现在位于控制器中,但将其带入模型中)。这样,只要我们需要在另一地方使用相同的东西,就可以调用此模型的函数。如果我们没有这样做,我们将一遍又一遍地重复此逻辑。那业务逻辑呢?让我们在服务层中采用业务逻辑。之后,我们的控制器将如下所示:

public function store(Request $request)
    {

        DB::beginTransaction();
        try{
            $results = $this->transport_service->doLogic($request); //makes logic according to user data.
            if($results) {
                $storeData = Transport::storeTransportThisWay($results, $request); //then we use model's function..
            }
            DB::commit();

            return response()->json(['success'=>'Transport type  has been binded to column and its values successfully'], 200);

        }catch(\Exception $e){
            DB::rollback();
            return response()->json(['error'=>'Something went wrong, please try later.'], 500);
        }
    }

所以基本上,我在模型中编写任何类型的查询。以及服务中的任何类型的业务逻辑。

第一个问题是:这是一种好方法吗?

2)我们可以做的是在控制器中,让我们调用服务的方法,在其中编写查询以及业务逻辑。这就是每个人使用的方式。但是问题是我是否遵循这种方法,比方说,在服务的一种方法中,我编写了查询和业务逻辑。一切正常。但换句话说,我也需要具有相同的查询,但是我不需要该业务逻辑。问题是我不能使用该服务的方法,因为它既具有业务逻辑又具有查询功能,而我只需要查询即可。所以问题是我还必须将查询放在其他地方,而不是放在服务的方法中,这给了我们另一个问题,那就是在哪里放置该查询。 我可以吗?如果是,我的问题是为什么人们喜欢这种方法?为什么不喜欢我提到的第一个方法?

很好的例子。

1 个答案:

答案 0 :(得分:0)

在您的情况下,我建议以下概念:

数据流:Request > Controller Action > Service > Repository > and back to the response

控制器(动作)只是“委托”到Service类,获取结果并创建ViewData来呈现响应。

示例

public function __invoke(Request $request, Response $response, array $args): ResponseInterface
{
    $id = (int)$args['id'];

    $user = $this->userService->getUserById($id);

    $viewData = [
        'user' => $user,
    ];

    return $this->render($response, 'User/user-edit.twig', $viewData);
}

(域)服务负责所谓的“业务逻辑”和事务处理。在这里,您还可以使用自定义验证类来检查请求的表单或API数据。

示例

namespace App\Domain\User;

use App\Domain\User\UserData;
use App\Domain\User\UserRepository;
use App\Domain\ApplicationService;

/**
 * Class.
 */
class UserService extends ApplicationService
{
    private $userRepository;

    public function __construct(UserRepository $userRepository)
    {
        $this->userRepository = $userRepository;
    }

    public function getUserById(int $userId): UserData
    {
        // Add some fancy business logic here, validation, transaction handling etc.

        return $this->userRepository->getById($userId);
    }
}

(面向持久性的)存储库仅负责“数据访问逻辑(与数据库通信),例如数据库查询(例如,选择,插入,更新,删除)。

示例

<?php

namespace App\Domain\User;

use App\Domain\User\UserData;
use App\Domain\ApplicationService;
use DomainException;

class UserRepository extends ApplicationRepository
{

    public function getById(int $id): UserData
    {
        // perform sql query here
        $user = ...

        if (!$user) {
            throw new DomainException(__('User not found: %s', $id));
        }

        return $user;
    }
}