CacheDecorator

Complexity

Total Complexity

8

Size/Duplication

Total Lines	43
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	0

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	6	1
A	getHash()	0	4	1
B	find()	0	24	6

<?php

class CacheDecorator

PSR1 recommends that each class must be in a namespace of at least one level to avoid collisions.

{
    private $repository;
    private $cacheRepository;
    private $allColumns;

    public function __construct(IRepository $repository, IRepository $cacheRepository)
    {
        $this->repository = $repository;
        $this->cacheRepository = $cacheRepository;
        $this->allColumns = new Collection();
    }

    private function getHash(string $id, string $class): string
    {
        return $class . '@' . $id;
    }

    public function find(string $id, string $class, array $columns = []): ?Storable
    {
        $hash = $this->getHash($class, $id);
        $model = $this->cacheRepository->find($id, $class, $columns);
        if ($model) {
            if ($columns) {

The expression $columns of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using ! empty($expr) instead to make it clear that you intend to check for an array without elements.

                // If you wanna specified columns...
                if (!array_diff($columns, array_keys($model->toArray()))) {
                    return $model;
                }
            } else {
                // If you wanna all columns...
                if ($this->allColumns->get($hash, false)) {
                    return $model;
                }
            }
        }
        $model = $this->repository->find($id, $class, $columns);
        $this->cacheRepository->update($model);
        if (!$columns) {

The expression $columns of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using empty($expr) instead to make it clear that you intend to check for an array without elements.

            $this->allColumns->put($hash, true);
        }
        return $model;
    }
}