3

最近我看了“Cruddy by Design” - Laracon US 2017,现在我试图摆脱控制器中的任何额外方法(所以我只有索引、创建、存储、显示、编辑、更新销毁

Adam Wathan 谈到了创建新的控制器来放入这些方法,例如:

PodcastsController@publish

会成为

PublishedPodcastsController@store

所以现在我的应用程序有这些路线:

Route::delete('tags', [TagsController::class, 'bulk_delete']);
Route::resource('tags', TagsController::class);

而且我不确定如何重构该bulk_delete方法。

我想出的一个想法是将TagsController控制器重命名为TagController(单数),创建一个新的TagsController(复数),然后将bulk_delete方法移到那里,所以我会有这些路线:

Route::resource('tag', TagController::class); // index, create, store, show, edit, update, destroy
Route::resource('tags', TagsController::class)->only([
    'delete' // delete => bulk_delete
]);

但我不确定我是否喜欢它。

有没有其他方法可以bulk_delete在 ResourceController 中使用该方法?

4

2 回答 2

1

老实说,在我看来,使用 TagsController 之类的东西是一个很大的问题,因为 Laravel 在他自己的逻辑中使用复数或不使用复数。

您可以向 destroy 方法添加一个请求并检查请求值(如 items),但您仍然必须处理 Object,因为默认情况下它会尝试创建一个对象。

你可以做的是发布一个删除,而不是 /{idofdeleteobject} 传递一个像 /bulk 这样的字符串,如果标签为空,因为它不是请求中对象数组的 ID 检查,

public function(Tag $tag, Request $request) {
    if(!is_null($tag)) {
       return $tag->delete();
    })
   
    if($request->has('bulk') {
       // logic for deleting all bulk
    })
}
于 2021-05-10T13:40:36.470 回答
0

不,这基本上不是为同一资源创建尽可能多的控制器的好方法。

如果你想使用资源以外的一些功能也可以。就像你有

Route::resource('tags', TagsController::class);

因此,如果您需要使用批量删除方法,那么您可以:

Route::get('tags/delete', [TagsController::class, 'bulk_delete']);

您可以在查询参数中传递 id 并通过控制器中的 Request::class 访问它们。

于 2021-05-09T23:39:22.710 回答