DI: как инстанциировать реализацию интерфейса с кастомным значением параметра конструктора?

Question

[Посторонним В.]

Юзаю php-di 6.0.10

В DI конфиге указана имплементация интерфейса:

\App\Writer\WriterInterface::class => DI\get(\App\Writer\Text::class)

Реализация имеет вот такой конструктор:

class TextWriter implements WriterInterface {
// ... 
public function __construct(Settings $settings, DataConverter $dataConverter, string $filePath)
{
    $this->init($filePath);
    $this->settings = $settings;
    $this->dataConverter = $dataConverter;
}

В своём приложении я хочу инстанциировать Writer с динамическим значением $filePath в конструкторе:

$this->container->make(WriterInterface::class, ['filePath' => $this->getFilePath()]);

Но вылазит такая ошибка:

DI\Definition\Exception\InvalidDefinition: Entry "App\Writer\Text" cannot be resolved: Parameter $filePath of __construct() has no value defined or guessable Full definition: Object ( class = App\Writer\Text lazy = false __construct( $settings = get(App\Settings) $dataConverter = get(App\DataConverter) $filePath = #UNDEFINED# )

Что не так? Работает только если я явно передаю App\Writer\Text вметод make(). Но мне нужно указывать именно интерфейс, а не реализацию.

Answer 1

[Максим Федоров]

Используйте фабрику:

- В конфиге контейнера

// регайте ваш интерфейс, но ссылаясь на метод фабрики,
// которая и произведет ваш сервис
\App\Writer\WriterInterface::class => DI\factory('\App\Writer\TextWriterFactory::create'),

- Код фабрики:

class TextWriterFactory implements WriterFactoryInterface
{
    public function __construct(Settings $settings, DataConverter $dataConverter)
    {
        $this->settings = $settings;
        $this->dataConverter = $dataConverter;
    }

    public function create(string $filePath): WriterInterface
    {
        return new TextWriter($this->settings, $this->dataConverter, $filePath);
    }
}

Источник: php-di.org/doc/php-definitions.html#factories

Comments

[Посторонним В.]

Но я не хочу использовать new. И фабрику я хочу использовать в другом объекте когда нужно, передавая на лету параметр filePath, а вашим способом всё будет контейнер резолвить заранее, то есть по надобности создавать объект и определять параметры не выйдет.

[Максим Федоров]

Посторонним В.,

а вашим способом всё будет контейнер резолвить заранее

тогда я не понимаю вашей затеи

[Евгений Ромашкан]

Посторонним В., Вы контейнер используете как сервис-локатор, не нужно так.
И суть внедрения зависимостей в том, чтобы не ваш класс-клиент инстанциировал и конфигурировал зависимость.
Хотите явно передать параметр - ну, можно использовать явный вызов фабрики.

[Максим Федоров]

Есть контейнер, есть сервис \App\Writer\Text::class, в него контейнер инжектит то, что ему известно, в случае filePath — пустоту, либо то, что из конфига передадите в определение самого сервиса...

вы хотите в какой-то момент новый объект собрать и изменить в сервисе? Что это за момент и почему нельзя сделать в начале?

налицо плохая архитектура и попытка влезть в холодильник через дезодарант

[Максим Федоров]

Если ваш Writer может писать в разные пути, то зачем вы этот путь задаете в конструкторе?

[Посторонним В.]

Максим Федоров, мне нужно инстанциировать Writer по условию и с определенным параметром.
Клиентский код получает имя файла для райтера, и если всё ок, инстанциирует Writer с этим значением. Хардкодить значение нельзя ни в каком конфиге, оно принимается от юзера при запуске программы.

В конструкторе - потому что при инициализации райтера открывается на запись переданный в конструктор файл.

[Максим Федоров]

Посторонним В., не понимаю я вас — вызвали фабрику, передали туда путь и с готовым объектом работайте, че не нравится — не понимаю

[Максим Федоров]

ClientService
{
    // инициализацию опустим
     public function __construct(WriterFactory $writerFactory);

    public function yourMethod(...)
    {
        $filePath = ...;
        $writer = $this->writerFactory->create($filePath);
        
        // тут хоть уработайтесь с вашим writer
    }
}

[Посторонним В.]

Максим Федоров, дело в том, что я не могу просто юзать new в фабрике и тупо передавать параметр. У меня Writer, как видно из кода, имеет еще другие параметры в конструкторе, и хотелось бы чтобы они инжектились DI автоматически, без дополнительной конфигурации. В этом вся идея. Всё инжектится автоматически, а filePath берется из передаваемого массива параметров. Поэтому я и юзаю container->make:

the parameters provided will be used for the constructor of the object, and the missing parameters will be resolved from the container.

Но это не работает из-за того что я указываю интерфейс, а не реализацию методу make.

С фабрикой разбираюсь, но по-моему так как вы пишете это просто обертка над container->make()

[Максим Федоров]

Посторонним В.,

class TextWriterFactory implements WriterFactoryInterface
{
    public function __construct(Settings $settings, DataConverter $dataConverter)
    {
        $this->settings = $settings;
        $this->dataConverter = $dataConverter;
    }

    public function create(string $filePath): WriterInterface
    {
        return new TextWriter($this->settings, $this->dataConverter, $filePath);
    }
}

[Посторонним В.]

Максим Федоров, а если потом там будет другая реализация WriterInterface, с другими зависимостями? Этот метод не подходит, к сожалению. Похоже, я хочу от php-di то, чего он не умеет.

Я копнул: он определяет мой дефинишн как \DI\Definition\SelfResolvingDefinition и поэтому не передаёт ему при создании параметры, которые я отправляю в make. Если указываю реализацию или указываю в DI конфиге DI\create вместо DI\get, то передаёт, но во втором случае он не резолвит опять же зависимости...

[Максим Федоров]

Посторонним В.,

а если потом там будет другая реализация WriterInterface, с другими зависимостями?

Все ровно также, как и без фабрики, ведь завяжитесь на интерфейс фабрики, а не конкретную фабрику
все как всегда

Похоже, я хочу от php-di то, чего он не умеет.

не до конца понимаю, что вы хотите :(

[Посторонним В.]

Максим Федоров, спасибо, и правда, рабочий вариант.

Answer 2

[Посторонним В.]

То что хотел сделать, не получилось - для интерфейсов make() параметры в конструктор не передаёт.
Вынес выбор реализации в спец класс (по входному параметру и карте) - дальше инстанциирую уже не интерфейс, а реализацию, и параметр работает.

Comments

[Максим Федоров]

так или иначе -- это вышло что-то вроде фабрики, просто с некоторым правилом (маппингом в вашем случае)