BrightComponents\Services\Traits\CachedService

Complexity

Total Complexity

3

Size/Duplication

Total Lines	43
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	1

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	call()	0	6	1
A	resolveBaseService()	0	8	1
A	translateBaseService()	0	7	1

<?php

namespace BrightComponents\Services\Traits;

use Illuminate\Support\Facades\Config;

trait CachedService
{
    /**
     * Respond to service call.
     *
     * @param  array  $parameters
     *
     * @return mixed
     */
    public static function call(...$parameters)
    {
        $decorator = new static(static::resolveBaseService());

The call to CachedService::__construct() has too many arguments starting with static::resolveBaseService().

        return $decorator->cache(...$parameters);

It seems like cache() must be provided by classes using this trait. How about adding it as abstract method to this trait?

    }

    /**
     * Resolve the base service for the cached service.
     *
     * @return string
     */
    private static function resolveBaseService()
    {
        $cachedServices = Config::get('service-classes.cached_services.classes', null);

        $baseService = $cachedServices[static::class] ?? static::translateBaseService();

        return resolve($baseService);
    }

    /**
     * Translate the cached service name to the base service name.
     *
     * @return string
     */
    private static function translateBaseService()
    {
        $cachedServicePrefix = Config::get('service-classes.cached_services.prefix');
        $cachedServiceNamespace = '\\'.Config::get('service-classes.cached_services.namespace');

        return str_replace([$cachedServicePrefix, $cachedServiceNamespace], ['', ''], static::class);
    }
}