Как избавиться от лишних запросов к БД при создании своих правил URL в Yii2?

Question

[beta-it]

Создал свое правило для формирования URL в Yii2, проблема в том что при выводе ссылки вида:

<a href="<?= Url::to(['site/test', 'country' => 'de'])?>">Test Route</a>
<a href="<?= Url::to(['site/test', 'country' => 'de'])?>">Test Route</a>
<a href="<?= Url::to(['site/test', 'country' => 'fr'])?>">Test Route</a>
<a href="<?= Url::to(['site/test', 'country' => 'ru'])?>">Test Route</a>

При этом создаются дублирующие запросы к БД вида:
SELECT * FROM `countries` WHERE `id`='DE' LIMIT 1
Одних только запросов с id=DE создалось: 6 судя по Yii Debug как то можно от этого избавиться, сократить?

<?php

namespace components;

use common\models\City;
use common\models\Country;
use yii\base\Object;
use yii\web\UrlRuleInterface;

class CountryCityUrlRule extends Object implements UrlRuleInterface
{

    public function createUrl($manager, $route, $params)
    {
        if ($route === 'site/test'){

            if ( isset($params['country']) ){

                $country = Country::find()->select('name')->asArray()->where(['id' => $params['country']])->limit(1)->scalar();

                if ( isset($params['city']) ){
                    $city = City::find()->select('name_en')->asArray()->where(['id' => $params['city'], 'country_id' => $country])->limit(1)->scalar();

                    if ($country !== false && $city !== false){
                        return $country . '/' .  $city;
                    }
                }

                if ($country !== false) {
                    return $country;
                }

            }

        }
        return false;
    }

    public function parseRequest($manager, $request)
    {
        $pathInfo = $request->getPathInfo();

        if (preg_match('%^(\w+)(/(\w+))?$%', $pathInfo, $matches)) {

            if (isset($matches[1])){
                $country_id = Country::find()->select('id')->asArray()->where(['name' => $matches[1]])->limit(1)->scalar();

                if (isset($matches[3])){

                    $city_id = City::find()->select('id')->asArray()->where(['name_en' => $matches[3], 'country_id'=>$country_id])->limit(1)->scalar();

                    if ($country_id !== false && $city_id !== false){
                        return ['site/test', ['country' => $country_id, 'city'=>$city_id]];
                    }
                }

                if ($country_id !== false) {
                    return ['site/test', ['country' => $country_id]];
                }
            }



        }
        return false;
    }

}

Answer 1

[Ne-Lexa]

Прикрутите кэширование результата SQL запроса.
www.yiiframework.com/doc-2.0/guide-caching-data.html

Пример:

$cacheDuration = 60; // 1 min
$country_id = Country::getDb()->cache(function ($db) {
    return Country::find()->select('id')->asArray()->where(['name' => $matches[1]])->limit(1)->scalar();
}, $cacheDuration);

Comments

[beta-it]

<?php

namespace components;

use common\models\City;
use common\models\Country;
use yii\base\Object;
use yii\web\UrlRuleInterface;

class CountryCityUrlRule extends Object implements UrlRuleInterface
{
    public function createUrl($manager, $route, $params)
    {
        if ($route === 'site/index'){

            if ( isset($params['country']) ){

                $country = $this->getCountryName($params['country']);

                if ( isset($params['city']) ){

                    $city = $this->getCitiesName($params['country'], $params['city']);

                    if ($country !== null && $city !== null){
                        return $country . '/' .  $city;
                    }
                }

                if ($country !== null) {

                    return $country;

                }
            }
        }
        return false;
    }

    public function parseRequest($manager, $request)
    {
        $pathInfo = $request->getPathInfo();

        if (preg_match('%^(\w+)(/(\w+))?$%', $pathInfo, $matches)) {

            if (isset($matches[1])){

                $country_id = $this->getCountryId($matches[1]);

                if ($country_id === null){
                    return false;
                }

                if ( isset($matches[3]) ){

                    $city_id = $this->getCitiesId($country_id, $matches[3]);

                    if ($city_id === null){

                        return false;
                    }

                    if ( isset($country_id) && isset($city_id) ){
                        return ['site/index', ['country' => $country_id, 'city' => $city_id]];
                    }
                }

                if ( isset($country_id) ) {
                    return ['site/index', ['country' => $country_id]];
                }
            }
        }
        return false;
    }

    private function getCountryId($countryName = null)
    {
        if (isset($countryName)){

            $country = Country::getDb()->cache(function ($db) {
                return Country::find()->select('id, slug')->asArray()->indexBy('slug')->all();
            });

            if (array_key_exists($countryName, $country))
                return $country[$countryName]['id'];
            else
                return null;

        }
        return null;
    }

    private function getCitiesId($countryId = null, $cityName = null)
    {
        if (isset($countryId) && isset($cityName)){
            $city = City::find()->select('id, slug, country_id')->where(['country_id' => $countryId])->asArray()->indexBy('slug')->all();
            //return $city[$cityName]['id'];

            if (array_key_exists($cityName, $city))
                return $city[$cityName]['id'];
            else
                return null;
        }
        return null;
    }

    private function getCountryName($countryId = null)
    {
        if (isset($countryId)){
            $country = Country::getDb()->cache(function ($db) {
                return Country::find()
                    ->select('id, slug')
                    ->asArray()
                    ->indexBy('id')
                    ->all();
            });
            return $country[$countryId]['slug'];
        }
        return null;
    }

    private function getCitiesName($countryId = null, $cityId = null)
    {
        if ( isset($countryId) && isset($cityId) ){
           // $city = City::getDb()->cache(function ($db) {
                if (isset($countryId)){
                    $city = City::find()
                        ->select('id, slug, country_id')
                        ->where(['country_id' => $countryId])
                        ->asArray()
                        ->indexBy('id')
                        ->all();
                }
                 else{
                     return null;
                 }
            //});
            return $city[$cityId]['slug'];
        }
        return null;
    }
}

Еще один момент не подскажите - пытаюсь кэшировать и города, но не вытаскивает данные...

При чем просто в виде запроса все работает хорошо, а если оборачиваю результат в кэширование то ни чего...

Answer 2

[Илья]

Кешировать? https://github.com/yiisoft/yii2/blob/master/docs/g...

Answer 3

[beta-it]

Решил следующим способом, сначала сократил всю выборку до одного запроса, далее работал по массиву, + еще воспользовался Вашими советами спасибо...

private function getCountry($countryId = null){

        if (isset($countryId)){

            $result = Country::getDb()->cache(function ($db) {

                return Country::find()->select('id, name')->asArray()->indexBy('id')->all();

            });

            return $result[$countryId]['name'];
        }
        return null;

    }