在 Laravel 中将类名传递给视图的常见做法

Question

我有一个将模型名称和类名称传递给视图的控制器方法。然后在另一个控制器方法中实例化这些类。在这种情况下，我使用的是 Laravel Excel 包。

public function index()
{
    $exports = [
        'Model name 1' => TestExport::class,
        'Model name 2' => AnotherExport::class
    ];

    return view('export', compact('exports'));
}


public function download(string $collection)
{
    return Excel::download(new $collection(), 'Export.xlsx');
}

我的视图文件然后重定向到具有特定类名的download 控制器方法。

@foreach($exports as $name => $collection)
  <a href="{{ action('ExportController@download', $collection) }}">Download</a>
@endforeach

由于我正在学习设计模式，并注意到它会违反 DRY 规则，所以我不想要另一个控制器方法或我下载的每个不同的 Excel 文件。

这是好的做法还是可以做得更好？

Answer 1

您可以使$exports 对这两种方法通用，并且不接受请求中的类名。

const EXPORTS = [
        'export_name_1' => TestExport::class,
        'export_name_2' => AnotherExport::class,
    ];
public function index()
{
    return view('export', compact(self::EXPORTS));
}


public function download(string $collection)
{
    if (!isset(self::EXPORTS[$collection]) {
        return 'error';
    }
    $className = self::EXPORTS[$collection];
    return Excel::download(new $className(), 'Export.xlsx');
}

永远不要让请求操作破坏您的代码。对于export_name，您可以简单地使用整数或数组简单索引。

查看

@foreach($exports as $name => $collection)
  <a href="{{ action('ExportController@download', $name)}}">Download</a>
@endforeach

Answer 2

对我来说，感觉您的解决方案是将视图逻辑与控制器逻辑混合在一起。

• 控制器应处理传入请求，获取正确的数据，然后形成响应。
• 视图应根据给定数据处理任何逻辑，以呈现输出页面。

在您的情况下，您传递给视图的数据不会被检索，它只是一个静态列表，也可以保留在视图中。更好的解决方案是：

1. 在刀片模板中包含路由列表（而不是类列表）并对其进行迭代。
2. 如果路线列表不是那么长，请对整个列表进行硬编码，而不是使其动态化。

这样做的主要原因是为了防止出现大量“魔法”代码。例如，在您的刀片模板中，您希望 exports 是一个数组，其中包含一个类列表，这些类必须是控制器，并且必须具有 download 函数，否则代码会中断。

解决方案 1 的示例如下所示：

<?php
 // You could consider moving $routes to the controller
$routes = [
    action('ExportController@download', TestExport::class),
    action('ExportController@download', AnotherExport::class),
];

@foreach($routes as $route)
    <a href="{{ $route }}">Download</a>
@endforeach