《L02 Laravel 教程 - Web 开发实战进阶 ( Laravel 5.5 )》 6.5. 删除回复

课程问答

使用 can 标签判断是否有删除回复的权限会导致 N+1 问题，有什么更好的办法吗？

@can('destroy', $reply)
    <span class="meta pull-right">
    <form action="{{ route('replies.destroy', $reply->id) }}" method="post">
        {{ csrf_field() }}
        {{ method_field('DELETE') }}
        <button type="submit" class="btn btn-default btn-xs pull-left">
            <i class="glyphicon glyphicon-trash"></i>
        </button>
    </form>
</span>
@endcan

这样写会导致每一条回复都去查找一次数据库
file
有什么更好的办法吗

GerBawn

课程读者 35 声望

暂无个人描述~

5 人点赞

从小程序个人账户申请开始，带你一步步进行开发一个微信小程序，直到提交微信控制台上线发布。

你将学到如 RESTFul 设计风格、PostMan 的使用、OAuth 流程，JWT 概念及使用和 API 开发相关的进阶知识。

smithadam

课程读者 21 声望

最佳答案

传递$topic 就不用关联查询了，但是删除按钮的表单是注入模型 ID 给路由的，要改路由很麻烦，所以新增一个授权方法，跟destroy分开。

# show.blade.php
@include('topics._reply_list', ['replies' => $topic->replies()->with('user')->get(), 'topic' => $topic])

# _reply_list.blade.php 把@can改为 
@can('showdel', [$reply, $topic])
....
@endcan

最后是 ReplyPolicy.php

use App\Models\Topic;
class ReplyPolicy extends Policy
{
.....
    public function showdel(User $user, Reply $reply, Topic $topic)
    {
        return $user->isAuthorOf($reply) || $user->isAuthorOf($topic);
    }
}

6年前评论

讨论数量: 4

xinhuo

课程读者 95 声望

不需要@smithadam 这么麻烦的方法，只需要在传入$replies前加上topic预加载即可，具体如下：
文件resources/views/topics/show.blade.php大概第70行左右的：

@include('topics._reply_list', ['replies' => $topic->replies()->with('user')->get()])

改成：

@include('topics._reply_list', ['replies' => $topic->replies()->with('user', 'topic')->get()])

如图：
file

6年前评论

Athos

多谢答疑

smithadam

课程读者 21 声望

传递$topic 就不用关联查询了，但是删除按钮的表单是注入模型 ID 给路由的，要改路由很麻烦，所以新增一个授权方法，跟destroy分开。

# show.blade.php
@include('topics._reply_list', ['replies' => $topic->replies()->with('user')->get(), 'topic' => $topic])

# _reply_list.blade.php 把@can改为 
@can('showdel', [$reply, $topic])
....
@endcan

最后是 ReplyPolicy.php

use App\Models\Topic;
class ReplyPolicy extends Policy
{
.....
    public function showdel(User $user, Reply $reply, Topic $topic)
    {
        return $user->isAuthorOf($reply) || $user->isAuthorOf($topic);
    }
}

6年前评论

doderic

课程读者 453 声望

还可以在 xinhuo 所写的基础上加上两个方法，实现按回复时间倒序排列以及分页。

@include('topics._reply_list', ['replies' => $topic->replies()->with('user', 'topic')->recent()->paginate(10)])

然后在 resources/views/topics/_reply_list.blade.php 的末尾加上

    ...
    {!! $replies->appends(Request::except('page'))->render() !!}
</div>

6年前评论

smithadam

课程读者 21 声望

传递$topic 就不用关联查询了，但是删除按钮的表单是注入模型 ID 给路由的，要改路由很麻烦，所以新增一个授权方法，跟destroy分开。

# show.blade.php
@include('topics._reply_list', ['replies' => $topic->replies()->with('user')->get(), 'topic' => $topic])

# _reply_list.blade.php 把@can改为 
@can('showdel', [$reply, $topic])
....
@endcan

最后是 ReplyPolicy.php

use App\Models\Topic;
class ReplyPolicy extends Policy
{
.....
    public function showdel(User $user, Reply $reply, Topic $topic)
    {
        return $user->isAuthorOf($reply) || $user->isAuthorOf($topic);
    }
}

6年前评论

xinhuo

课程读者 95 声望

不需要@smithadam 这么麻烦的方法，只需要在传入$replies前加上topic预加载即可，具体如下：
文件resources/views/topics/show.blade.php大概第70行左右的：

@include('topics._reply_list', ['replies' => $topic->replies()->with('user')->get()])

改成：

@include('topics._reply_list', ['replies' => $topic->replies()->with('user', 'topic')->get()])

如图：
file

6年前评论

Athos

多谢答疑

ruodee

课程读者 441 声望 / 物业负责人 @ 天马集团

解决N+1的问题，首先考虑预加载。:+1:

6年前评论

doderic

课程读者 453 声望

还可以在 xinhuo 所写的基础上加上两个方法，实现按回复时间倒序排列以及分页。

@include('topics._reply_list', ['replies' => $topic->replies()->with('user', 'topic')->recent()->paginate(10)])

然后在 resources/views/topics/_reply_list.blade.php 的末尾加上

    ...
    {!! $replies->appends(Request::except('page'))->render() !!}
</div>

6年前评论

讨论应以学习和精进为目的。请勿发布不友善或者负能量的内容，与人为善，比聪明更重要！

帮助

使用 can 标签判断是否有删除回复的权限会导致 N+1 问题，有什么更好的办法吗？

高认可度评论：

社区赞助商

关于 LearnKu

资源推荐

服务提供商

其他信息

使用 can 标签判断是否有删除回复的权限会导致 N+1 问题，有什么更好的办法吗？

高认可度评论：

社区赞助商

关于 LearnKu

资源推荐

服务提供商

其他信息

请登录