[PHP][class] Prosty container Opcje

[untorched]

Witam.
Chciałem przedstawić klasę prostego kontenera.
Wzorowałem się na kontenerze z laravela, lecz nie "kopiowałem", a pisałem samodzielnie, co na pewno widać (IMG:style_emoticons/default/smile.gif)
Chciałbym prosić o ocenę kodu, w miarę możliwości tego jak będzie wydajny oraz wytknięcie błędów.

Z góry dziękuję za waszą pomoc (IMG:style_emoticons/default/smile.gif)

[PHP] pobierz, plaintext 
class Container implements ArrayAccess {
 
	protected $bindings = array();
 
	protected $instances = array();
 
	private $maxdepth = 10;
 
	// ================================================== //
 
	public function make($key, $params = array())
	{
		if($bind = $this->getBinding($key))
		{
			if(isset($bind['closure']))
			{
				if(isset($bind['singleton']) and $bind['singleton'] === true)
				{
					$bind['instance'] = $bind['closure']();
					$this->instances[$key] = $bind['instance'];
 
					unset($this->bindings[$key]['closure'], $this->bindings[$key]['singleton']);
				}
				else
				{
					$bind['instance'] = $bind['closure']();
				}
			}
 
			return $bind['instance'];
		}
		else
		{
			$ret = $this->makeClassWithParams($key, (array)$params);
 
			if($ret != false)
			{
				$this->bind($key, $ret);
			}
 
			return $ret;
		}
	}
 
	public function bind($key, $value)
	{
		return $this->makeBinding($key, $value, false);
	}
 
	public function singleton($key, $value)
	{
		return $this->makeBinding($key, $value, true);
	}
 
	// ================================================== //
 
	protected function makeClassWithParams($name, array $params, $depth = 1)
	{
		if($depth >= $this->maxdepth)
		{
			throw new OutOfBoundsException('Maximum depth of makeClassWithParams is '.$this->maxdepth);
			return false;
		}
 
		if(class_exists($name))
		{
			$class = new ReflectionClass($name);
 
			$constructor = $class->getConstructor()->getParameters();
 
 
			if(!empty($constructor) and empty($params))
			{
				$constructor = $class->getConstructor()->getParameters();
 
				foreach($constructor as $classparam)
				{
					$classparam = $classparam->getClass();
 
					if(is_null($classparam))
					{
						$params[] = null;
					}
					else
					{
						$params[] = $this->makeClassWithParams($classparam->name, array(), $depth+1);
					}
				}
 
				$class = $class->newInstanceArgs($params);
			}
			else
			{
				$class = $class->newInstanceArgs($params);
			}
 
			return $class;
		}
		else
		{
			echo $name;
			return false;
		}
	}
 
	protected function makeBinding($key, $value, $singleton)
	{
		$this->clearBinding($key);
 
		$bind = array();
 
		$bind['instance'] = $key;
 
		if($value instanceof Closure)
		{
			$bind['closure'] = $value;
			$bind['singleton'] = (bool)$singleton;
		}
		else
		{
			$bind['instance'] = $value;
		}
 
		if(isset($bind['instance']))
		{
			$this->instances[$key] = $bind['instance'];
		}
		else
		{
			$this->instances[$key] = null;
		}
 
		$bind['instance'] = $key;
 
		$this->bindings[$key] = $bind;
	}
 
	protected function getBinding($key)
	{
		if(isset($this->bindings[$key]))
		{
			$bind = $this->bindings[$key];
 
			if(! is_null($this->instances[$key]))
			{
				$bind['instance'] = $this->instances[$key];
			}
 
			return $bind;
		}
 
		return false;
	}
 
	protected function clearBinding($key)
	{
		unset($this->bindings[$key], $this->instances[$key]);
	}
 
	// ================================================== //
 
	public function offsetGet($key)
	{
		return $this->make($key);
	}
 
	public function offsetSet($key, $value)
	{
		$function = function() use ($value) {
			return $value;
		};
 
		$this->bind($key, $function);
	}
 
	public function offsetExists($key)
	{
		return isset($this->bindings['key']);
	}
 
	public function offsetUnset($key)
	{
		$this->clearBinding($key);
	}
 
	public function __set($key, $value)
	{
		$this[$key] = $value;
	}
 
	public function __get($key)
	{
		return $this[$key];
	}
 
}
[PHP] pobierz, plaintext 

Ten post edytował untorched 5.10.2014, 00:50:05

[!*!]

Byłoby prościej jakbyś napisał o co w tym chodzi, jak tego użyć i stworzył dokumentacje ;)

[Daimos]

Kurcze też Wam zahashowało komentarze na // ================================================== //
?
(IMG:style_emoticons/default/wink.gif)

[untorched]

Nie ma opcji edytuj więc muszę dorzucić to tutaj.

Przepraszam za brak komentarzy i opisu działania, nadrobiłem, są komentarze(błędy stylistyczne, błędy składni językowej lub inne dziwne rzeczy mogą się tam znaleźć niestety). Poniżej zamieszczam kod oraz użycie

http://pastebin.com/2H0FsKsB

[PHP] pobierz, plaintext 
// Tworzenie instancji kontrolera, można również rozszerzyć jakąś klasę
$container = new Container();
 
// Dodanie tekstu do kontenera
$container->bind('tekst', 'Dowolna zawartość do przechowania');
 
// Dodawanie stworzonej instancji do kontenera
$container->bind('instancja', $container);
 
// Dodanie funkcji anonimowej do kontenera
$container->bind('funkcja_anonimowa', function() {
	return rand();
});
 
// Dodanie funkcji anonimowej jako singleton do kontenera
$container->bind('funkcja_anonimowa_singleton', function() {
	return rand();
}, true);
 
// Tworzenie klasy przy użyciu kontenera wraz z wymaganymi parametrami konstruktora
class Test1 {}
class Test2 {}
class Test3 {}
 
class Example {
	public function __construct(Test1 $test1, Test2 $test2, Test3 $test3) {}
}
 
$container->make('Example');
 
// ================================================================= //
 
/**
 * Testowanie przechowywanych rzeczy.
 * Można użyć $container->nazwa_bindu;
 * Można użyć $container->make('nazwa_bindu');
 * Można użyć $container['nazwa_bindu'];
 */
 
var_dump($container->tekst);
 
var_dump($container->make('instancja'));
 
var_dump($container['funkcja_anonimowa']);
 
var_dump($container['funkcja_anonimowa_singleton']);
 
var_dump($container['Example']);
[PHP] pobierz, plaintext 

Powinno wszystko działać. Aczkolwiek liczę się z tym, że możliwie popełniłem tu kilka błędów.
Proszę o opinie na temat kodu, wygody użycia, braków itd.

[!*!]

[PHP] pobierz, plaintext 
class Demo
{
    public $text;
 
    public function __construct($text = 'yyyyyyy')
    {
        $this->text = $text;
    }
    public function test()
    {
        return rand();
    }
 
    public function add($t)
    {
        $this->text = $t;
    }
}
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
$di->make('Demo');
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
var_dump($di->Demo->text);
[PHP] pobierz, plaintext 

Zwróci null.

[untorched]

http://pastebin.com/PUfFshcY

Poprawka.

Faktycznie nie uwzględniłem, że parametr konstruktora może posiadać wartość domyślną.

Dziękuje serdecznie !*! za znalezienie tego buga. Jeśli jest coś jeszcze to chętnie przyjmę to do informacji (IMG:style_emoticons/default/smile.gif)

[!*!]

Grunt, że działa. Nie wiem jak to się sprawdza w praktyce przy czymś większym. Osobiście mam wrażenie, że trochę przebajerowałeś z makeClassWithParameters i samym ReflectionClass. Choć może to tylko subiektywne odczucie.