php - Laravel 5.2 View Composer 被多次执行

Question

我在我正在处理的项目中遇到了一些(主要的)性能问题，在记录了所有执行的查询之后，我意识到其中许多被执行了多次，我无法找到问题的根源。

所有多次执行的查询都在我的 View composer provider 中。

这是我的 View Composer 的样子:

public function boot()
    {
        view()->composer('partials.sidebar', function ($view) {
            $currentCategory = $this->getCurrentCategory();
            $sidebarCategories = SidebarCategory::category($currentCategory)
                ->order()
                ->get();
            $view
                ->with('sidebarCategories', $sidebarCategories);
        });

        view()->composer('partials.footer', function ($view) {
            $footerLinks = FooterCategory::with('links.translations')->order()->get();
            $footerColumn1 = $footerLinks->filter(function ($value, $key) {
                return $value->column == 1;
            });
            $footerColumn2 = $footerLinks->filter(function ($value, $key) {
                return $value->column == 2;
            });
            $footerColumn3 = $footerLinks->filter(function ($value, $key) {
                return $value->column == 3;
            });
            $footerColumn4 = $footerLinks->filter(function ($value, $key) {
                return $value->column == 4;
            });

            $view
                ->with(compact('footerColumn1', 'footerColumn2', 'footerColumn3', 'footerColumn4'));
        });
}

这两个查询(Sidbar 和 Footer 类别)都执行了大约 6 次，即使每个部分只调用一次。它们都在主视图中使用@include('partialname') 调用。

我已经试过了:

if($view->offsetExists('sidebarCategory'))
    return;

但是 offsetExists 总是返回 false(即使在调用 5 次之后)。

知道为什么会发生这种情况以及我做错了什么吗？

编辑:

我已经意识到问题出在哪里了。在我正在访问的页面上(执行多个查询的页面)有一些 404 元素(主要是图像)。每次找不到文件时都会抛出一个新的 404 异常。每次抛出 404 异常时，都会执行 404 View => 意味着页脚/侧边栏查询也会执行(因为它们是 404 View 的一部分)。示例:http://imgur.com/a/RrmOD

所以接下来的问题是如何防止在不需要时渲染 View (例如 404 是未找到的图像)。

这是我的 route 的一段代码，我认为这是发生这种情况的原因:

Route::get('{slug}', ['as' => 'findBySlug', 'uses' => function($slug) {
    if(\App\DynamicCategory::findBySlug($slug)->count() > 0)
        return App::make('App\Http\Controllers\GeneralController')->getDynamicCategoryIndex($slug);
    else if(\App\DynamicPage::noCategory()->findBySlug($slug)->count() > 0)
        return App::make('App\Http\Controllers\GeneralController')->getDynamicPage($slug);
    else
        abort(404);
}]);

PS:我知道这段代码极度未优化(因为它基本上执行相同的查询两次，一次是查看该项目是否存在，另一次实际上是在 Controller 中)。这项工作正在进行中，并且在待办事项列表中。

编辑 2:

我想出了下一个解决方案，我愿意接受改进，因为这是一种有点 hack-ish 的方式(如果我添加更多文件夹，我也需要记住更新它)。我在公用文件夹中只有 3 个直接子文件夹:something、files 和 resources。

解决方案是在呈现异常时检查 url 的第一段(在文件 app/Exceptions/Handler.php 中)，如果它与 3 个文件夹中的一个匹配，则返回不带 View 的 404 响应:

public function render($request, Exception $e)
{
    $firstSegment = $request->segment(1);
    if(starts_with($firstSegment, 'files') || starts_with($firstSegment, 'something') || starts_with($firstSegment, 'resources')) {
        return response('Stran ne obstaja', 404);
    }

    return parent::render($request, $e);
}

提前致谢

Answer 1

不要通过 laravel 路由文件请求并在 404 上提供空白图像

.htaccess 示例:

RewriteEngine On
RewriteCond %{REQUEST_URI} \.(jpg|jpeg|gif|png|ico)$ [NC]
RewriteCond %{REQUEST_FILENAME} !-f
RewriteCond %{REQUEST_FILENAME} !-d
RewriteRule .*$ /no_picture.png [L]

您可以将其发布在重定向到 index.php 的规则之上

这是假设图片是从网站提供的，而不是通过流式数据库内容提供的。

基本上是这样的:

• 后端用户上传图片:
• laravel 将图片存储在/some/path/to/store/data/public/12a/4d8/458/12a4d8458.gif
• laravel 在你的数据库中将图片存储为 12a4d8458.gif
• --------------------时间流逝--------------------
• 访客请求页面。
• 请求与文件不匹配。重定向到 index.php(根据 .htaccess)
• 请求被发送到 index.php
• laravel 构建页面。查找图片
• laravel 编写完整的公共(public) url 路径
• www.awesome.com/data/public/12a/4d8/458/12a4d8458.gif
• 将编写的 html 内容推送给访问者
• ---------------- 毫秒过去------------------------
• 访客请求图片/data/public/12a/4d8/458/12a4d8458.gif
• 请求匹配文件文件由 apache 提供服务
• laravel 仍然幸福地不知道请求
• ---------------- 不到一毫秒 --------------
• 访客请求图片/data/public/4e8/d44/98f/4e8d4498f.gif
• 请求与文件不匹配。重定向到 index.php(根据 .htaccess)
• laravel 构建页面。找不到路线。
• laravel 调用 404 例程
• laravel 构建 404 页面并关联所有触发器
• laravel 为用户提供 404

因此，您在最后一步中要做的是根据 .htaccess 规则提供您自己的图像，从而阻止图像请求甚至到达 Laravel。这样您的网络服务器就可以更快地响应丢失的图像。