3

我在使用 laravel 开发应用程序时使用存储库模式，我的问题是我必须在哪里为火灾事件编写代码、发送电子邮件或发送通知？为什么？

4

1 回答 1

2

这确实是一个广泛的问题，许多人会提出自己的意见。在我看来，在 的上下文中Laravel，我会根据操作定义我的事件类型。

例如，正如您提到的电子邮件/通知事件，我想这样想（这是一个假设的例子）：

class UserController
{
    public function register(Request $request, UserRepository $user)
    {
        if ($user = $user->register($request->all())) {
            Email::send(...);
        }
    }
}

在这种情况下，注册后应该向用户发送一封电子邮件，以便我可以使用事件在控制器中执行相同的操作，例如：

class UserController
{
    public function register(Request $request, UserRepository $user)
    {
        try {

            $user = $user->register($request->all());

            Event::fire('user_registered', $user);

        } catch(RegistrationException $e) {
            // Handle the exception
        }
    }
}

在这种情况下，我认为，事件调度应该在控制器中，因为它是我的应用程序层的一部分来控制应用程序流，因此，应该从控制器调度电子邮件发送事件。UserRepository不应该关心您的应用程序的流程，向用户发送电子邮件不是您的一部分，UserRepository就是这样。

现在，想想另一个假设的例子，假设你有一个delete方法，UserController如下所示：

class UserController
{
    public function delete(UserRepository $user, $id)
    {
        if($user->findOrFail($id)->delete()) {
            Post::where('user_id', $id)->delete();
        }

    }
}

好吧，在这种情况下，删除用户涉及一些与域相关的操作，所以我将重写方法如下：

public function delete(UserRepository $user, $id)
{
    try {

        $user->delete($id);

        return redirect('/users'); // show users index page

    } catch (InvalidOperationException $e) {
        // Handle the custom exception thrown from UserRepository
    }
}

请注意，删除方法中没有发生相关操作，因为我可能会在其中触发一个事件，UserRepository因为此删除操作涉及一些其他领域/业务相关的操作，应用层不应该关心它（在这种情况下），因为删除用户会影响其他一些域对象，因此我将以这种方式处理该事件。

无论如何，这只是我的思维方式，这只是一个意见。也许在现实世界的情况下，我可以想出一个不同的想法，所以这取决于你，这取决于你应该考虑的上下文，最后在 Laravel 中没有推荐的方式，你甚至可以Models用来触发事件，所以保持很简单，根据适合您的上下文做出决定。

于 2018-02-03T20:36:14.603 回答