Question

从控制器类方法包装一些代码是一种好习惯，因为它太长了，无法将该代码放入控制器类下的另一个自定义函数中。

这是控制器方法：

    public function store(UploadRequest $request)
{
    //Provjera duljine imena slike (max 20 znakova)
    if(!is_valid_name($request->file('file'))) {

        return redirect()->back()->withErrors(['File name can\'t be longer than 20 characters.']);
    }

    $user = Auth::user();
    $time = time();
    $image = $request->file('file');

    //Dodaj trenutno vrijeme prije imena slike kako bi se slika mogla identificirati
    $image_name = $time . $image->getClientOriginalName();

    //Ako je slika png spremi ju bez konverzije
    if($image->getClientOriginalExtension() == "png")
    {
        Storage::put('public/images/'.$user->id.'/png/'.$image_name, file_get_contents($image));
        save_image_to_database($image, $user, $time);
    }

Save_image_to_database（）是我的自定义函数，写在控制器类下面：

function save_image_to_database($image, $user, $time){

$db_image = new Image();

if($image->extension() == 'png')
{
    $db_image->path = $time . $image->getClientOriginalName();
    $db_image->png_size = $image->getClientOriginalSize();
}
else
{
    $path = $time . pathinfo($image->getClientOriginalName(), PATHINFO_FILENAME).'.png';
    $png_size = Storage::size('/public/images/'.$user->id.'/png/'.$path);

    $db_image->path = $path;
    $db_image->png_size = $png_size;
}

$db_image->user_id = $user->id;
$db_image->extension = $image->extension();
$db_image->size = $image->getClientSize();   
$db_image->save();

return redirect('/images');

}

问题是自定义函数中的redirect（）方法无法正常工作，它会重定向到空白窗口，但浏览器“ localhost：8000 / images”中的路径是正确的。视图被返回。如果我将redirect（）方法从自定义函数移到控制器store（），那么它将很好用。

Answer 1

您的函数返回了Response对象，但控制器未使用它

最简单的解决方案是返回

这样的辅助函数的响应

return save_image_to_database($image, $user, $time);

但是我至少将它包装在try-catch中，以处理无法保存文件的情况，

try {
    save_image_to_database($image, $user, $time);
} catch (\Exception $e) {
    // return error response
}

// if we got here image was successfully saved
return redirect('/images');

现在回到您的问题，关于将图像作为另一种控制器方法进行处理是否有很好的经验。.我要说不，您想让您的控制器保持非常薄的外观，并且仅使用Router使用的方法

我要么将其移至Trait（Traits in PHP）并在控制器中使用该特征，要么创建一个Image Service并通过dependency injection使用它。

在控制器类下面的控制器中写入函数？

1 个答案: