Stratadox\Hydration\Proxying\Proxying

Complexity

Total Complexity

2

Size/Duplication

Total Lines	16
Duplicated Lines	0 %

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
A	__load()	0	7	2

<?php

declare(strict_types=1);

namespace Stratadox\Hydration\Proxying;

use function is_null;
use Stratadox\Hydration\LoadsProxiedObjects;
use Stratadox\Hydration\Proxy;

/**
 * Lazily loads proxy targets.
 *
 * @author Stratadox
 * @package Stratadox/Hydrate
 */
trait Proxying
{
    /** @var LoadsProxiedObjects */
    private $loader;

    /** @var object|null */
    private $instance;

    /** @return self */
    public function __load()
    {
        if (is_null($this->instance)) {
            /** @var Proxy $this */
            $this->instance = $this->loader->loadTheInstance();

Accessing instance on the interface Stratadox\Hydration\Proxy suggest that you code against a concrete implementation. How about adding an instanceof check?

Accessing loader on the interface Stratadox\Hydration\Proxy suggest that you code against a concrete implementation. How about adding an instanceof check?

        }
        return $this->instance;
    }
}