Извлёк из своей CMS контейнер для внедрения зависимостей, покрыл тестами, задал новый неймспейс.

Собственно сабж: https://packagist.org/packages/cheevauva/pdic

_____________
Люди, имеющие низкий уровень квалификации, делают ошибочные выводы, принимают неудачные решения и при этом неспособны осознавать свои ошибки в силу низкого уровня своей квалификации

Ну молодец, чо.

Разреши задать вопрос. По большому счету он не только тебе адресован. Тебе в частности.

Если хватает квалификации делать что-то более-менее вменяемое – почему бы не помочь улучшить что-то существующее, более популярное?. Я понимаю, что делать свое прёт, и всё такое... Но как теперь ориентироваться во всем этом https://packagist.org/search/?q=di%20container ?

Потому что идеология моего решения разительно отличается от других. Если коротко, то в самом приложении построенном на этом контейнера, сам контейнер можно увидеть в двух местах: где инициализируемый он сам и критические зависимости, а также в фабриках. Хоть есть и реализация Service Locator, она по факту сделана для того что-бы была. При этом есть возможность задать конкретные зависимости для определенного компонента (читай объекта).

Ну и последнее, я вырвал этот контейнер из CMS'ки, потому что его нужно дать программисту, который будет строить ООП систему в качестве обучения, при этом система должна будет строиться на основе какого-то DI контейнера. Мой контейнер, наверно самый простой из существующих, как и в использовании, так и в изучении.

_____________
Люди, имеющие низкий уровень квалификации, делают ошибочные выводы, принимают неудачные решения и при этом неспособны осознавать свои ошибки в силу низкого уровня своей квалификации

Изменения:
- Удаление поддержки логических зависимостей

Тесты также обновлены и проходят, качать тут

Кстати, у кого есть годный гайд по git на русском языке, чет он вообще работает не как mercurial

_____________
Люди, имеющие низкий уровень квалификации, делают ошибочные выводы, принимают неудачные решения и при этом неспособны осознавать свои ошибки в силу низкого уровня своей квалификации

T1grOK, ну я пока адекватно оценить не могу, мне пока что mercurial удобнее всего так как я в основном с ним и работал, буду смотреть, а то даже ветку с релизом не могу создать


_____________
Люди, имеющие низкий уровень квалификации, делают ошибочные выводы, принимают неудачные решения и при этом неспособны осознавать свои ошибки в силу низкого уровня своей квалификации

chee, а можно услышать почему не Pimple? =)

Ron, недостатки Pimple
1. Ручное конфигурирование, через анонимные функции
2. Использование sl (хотя судя по примерам, в приделах самого контейнера)
3. Сам контейнер нарушает принцим разделения ответственности, точнее позволяет нарушить, это я веду к фиче, которая позволяет хранить в контейнере конфигурационные параметры.

Это не фатальные недостатки, но в моём pet-проекте я не могу себе их позволить.

_____________
Люди, имеющие низкий уровень квалификации, делают ошибочные выводы, принимают неудачные решения и при этом неспособны осознавать свои ошибки в силу низкого уровня своей квалификации

chee, и снова без документации. =) Как и фреймворк, только тут есть шанс все-таки понять что к чему, потратив лишнее время.

Мог бы посоздавать alias для Cheevauva.
А то за собой тащить такие макороны не удобно.

\Cheevauva\Contract\Container\UseServiceLocator

   if ($object instanceof \Cheevauva\Contract\Container\UseServiceLocator) {
            $object->setContainer(new \Cheevauva\Container\ServiceLocator($properties, $this));
            return;
        }

Не хватает комментариев к коду.Особенно для метода classUsesDeep.
Название метода вообще ничего не говорит.

Не поняты название переменной $tratis

        $traits = array();
        do {
            $traits += class_uses($class);
        } while ($class = get_parent_class($class));
        foreach ($traits as $trait => $same) {
            $traits += $this->classUsesDeep($trait);
        }
        return array_unique($traits);

https://github.com/cheevauva/container/blob...a/Container.php

      $object = new $className;
        $this->objects[$className] = $object;
        $this->handleObject($object);
        
        if ($object instanceof \Cheevauva\Contract\Container\Mediator) {
            $this->objects[$className] = $object->get();
        }

Не нравится такой подвязанной подход с условиями.

Что-то типо $this->objects[$className] = $object->getClassObject(); подошло бы для всех.