Controller中重复的代码行，在哪里用它们创建方法？

Question

Aloha，我的 Controller 中有两种方法，一种用于设置头像，另一种用于更新头像。我在这两种方法中都使用了这行代码：

    $user = Auth::user();

    if (Input::file('image')) {
        $image = Image::make(Input::file('image'));
        $fullName = Input::file('image')->getClientOriginalName();
        $extension = Input::file('image')->getClientOriginalExtension();

        $pathToCreate = public_path() .'/images/'. $user->email . '/';
        $fullPath = $pathToCreate . $user->email . '.' . $extension;
        $pathDatabase = 'images/' . $user->email . '/' . $user->email . '.' .$extension;

        // Creating directory if it does not exists
        File::exists($pathToCreate) or File::makeDirectory($pathToCreate);
        $image->resize(null, 145, function ($constraint) { $constraint->aspectRatio(); })
            ->crop(130,130)
            ->save($fullPath);
        $user->picture = $pathDatabase;

我想用这行代码创建一个方法，但我觉得控制器不是一个好地方。我应该把这个方法放在哪里？

Answer 1

一般来说，这取决于代码的用途。如果这是您网站上唯一会使用该代码的地方，那么将代码作为私有方法包含在 Controller 类中是完全可以接受的。

如果此代码将在其他控制器中使用，您可能需要创建一个服务类来处理此问题。在您的控制器中使用时，这将等同于以下内容：

$user->picture = $this->fileUploadService->process( Input::file('image') );

非常简短的回答是：把它放在最有意义的地方。通用解决方案应作为服务广泛使用。一个特定的解决方案应该去任何需要特定性的地方（控制器、存储库、模型等）。

Answer 2

你应该看看 Laravel-Stapler。对于在 Laravel 中处理图像上传非常有用。 https://github.com/CodeSleeve/laravel-stapler

而不是保存图像并检查它是否存在，您可以将图像附加（或“装订”）到模型。

以下是从文档中获取的有关如何设置表单的示例。

 <?= Form::open(['url' => action('UsersController@store'), 'method' => 'POST', 'files' => true]) ?>
    <?= Form::input('first_name') ?>
    <?= Form::input('last_name') ?>
    <?= Form::file('picture') ?>
    <?= Form::submit('save') ?>
 <?= Form::close() ?>

模型在控制器中会有这样的东西：

 $this->hasAttachedFile('picture', [
        'styles' => [
            'thumbnail' => '100x100',
            'large' => '300x300',
            'pictureCropped' => '75x75#'
        ],
        'url' => '/system/:attachment/:id_partition/:style/:filename',
        'default_url' => '/:attachment/:style/missing.jpg'
    ]);

那么你在控制器中所做的就是User::create(Input::all());

检查“附件”是否存在就像if ($user->picture) ... 一样简单

因此，Stapler 已经处理了文件的保存，并且通过'pictureCropped' => '75x75#' 配置自动完成了裁剪。这应该会删除足够多的代码，您不需要创建其他方法。

希望这会有所帮助！