artoodetoo/container

Yet another service container

container's Issues

Метод set() возвращает $service

Метод set() возвращает $service, а не ContainerInterface как задумывалось.

Передавать контейнер прямо в конструктор

Я все успокоится не могу :)
Предлагаю ссылку на контейнер передавать не через метод setContainer(), а прямо в конструктор.
Заменяем

$service = new $class(...$args);
if ($service instanceof ContainerAwareInterface) {
    $service->setContainer($this);
}

на

$args[] = $this; // передача контейнера сразу в конструктор
$service = new $class(...$args);

В результате избавляемся от не нужного метода setContainer() (и маленькой кучки дополнительной писанины) и получаем ссылку сразу в конструкторе созданного класса.

А если контейнер в классе не нужен, то в аргументах __construct() не задаем последний аргумент для него.

Метод resolve()

Нужен ли он в публичном доступе?
Не понятно, что можно получить с его помощью извне класса.

Warning: array_key_exists() expects parameter 2 to be array, integer given in ...

Тестировал метод getParameter(). Вероятно аналогичная ошибка будет в методе setParameter().
Тесты с другими вариантами проверок (is_array() + array_key_exists() и isset() + is_array() + array_key_exists() <- это вариант быстрее, особенно на 5.6.x):

1. Нормальное поведение, элемент отличный от null - http://sandbox.onlinephpfunctions.com/code/7219247d1584531490acdb5be4ac71015d17ca28
2. Нормальное поведение, элемент равен null - http://sandbox.onlinephpfunctions.com/code/0acde0551d609bc028096f389e8e4b858f44aa6b
3. Поведение с ошибкой, элемент не null - http://sandbox.onlinephpfunctions.com/code/d2ae6641b87c95b3e0bca00bb3e04fd4b27c7fe6
4. Поведение с ошибкой, элемент null - http://sandbox.onlinephpfunctions.com/code/479e0f1f8d82535cefada8107c17726e83e3cad9