Я пытаюсь автоматически подключить шаблоны к контейнеру inversifyjs, но все, что я пробую, не работает. Пожалуйста помоги?

private templates = [
    {file: './component.html.tpl', obj: 'HtmlTemplate'},
    {file: './component.tpl.ts', obj: 'ComponentTemplate'}
];
private container = new Container();
bind(){
    // original and working code 
    // this.container.bind<ITemplate>('HtmlTemplate').to(HtmlTemplate);
    // this.container.bind<ITemplate>('ComponentTemplate').to(ComponentTemplate);
    this.templates.forEach(template => {
        import(template.file).then(mod => {
            console.log(mod.default, template);
            // is this correct (seems to work) => 
            this.container.bind<ITemplate>(template.obj).to(mod.default);
            console.log('bound =>', mod.default);
        });
    });
}

И файлы ./component.html.tpl

@injectable() export default class HtmlTemplate implements ITemplate { ... }

И ./component.ts.tpl

@injectable() export default class ComponentTemplate implements ITemplate { ... }

Которая полностью регистрирует консоль, как и ожидалось:

[Function: HtmlTemplate] { file: './component.html.tpl', obj: 'HtmlTemplate' }
[Function: ComponentTemplate] { file: './component.tpl.ts', obj: 'ComponentTemplate' }

Но я действительно ожидал, что код в инструкции foreach:

this.container.bind<ITemplate>(template.obj).to(mod.default);

Чтобы быть эквивалентным этому:

this.container.bind<HtmlTemplate>('HtmlTemplate').to(HtmlTemplate);
this.container.bind<ComponentTemplate>('ComponentTemplate').to(ComponentTemplate);

Но когда я пытаюсь решить его в другом цикле:

this.templates.forEach(template => {
    const tpl = this.container.get<ITemplate>(template.obj);
...

Выдает ошибку:

Error: No matching bindings found for serviceIdentifier HtmlTemplate

Кто-нибудь знает, как это решить?

1
wendellmva 14 Май 2018 в 13:53

1 ответ

Лучший ответ

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

Каждое обещание должно быть сковано цепью. Использование forEach не рекомендуется в ES6 по нескольким причинам, одна из которых заключается в том, что он требует дополнительных действий для работы с обещаниями и не работает с генераторами и функциями async. Код может брать на себя большинство функций async и подвергаться рефакторингу, чтобы сделать поток управления чистым и эффективным:

async bind(){
    for (const template of this.templates)
       const mod = await import(template.file);
       this.container.bind<ITemplate>(template.obj).to(mod.default);
    }
}

Код, который использует bind, должен связывать его и избегать антипаттернов построения обещаний:

async bind() {
    // binding for when the widget is needed;
    for (const component of this.components)
        component.widget = this.container.get<Widget>(component.name);
        if(component.widget) {
            await component.widget.configure();
            await component.widget.bind();
        } else {
            console.error(`widget ${component.name} not resolved`);
        }
    });

    return this;
}

Более эффективный способ - отказаться от процедуры асинхронной инициализации, потому что единственное, что требует этого, - это динамический import(). Промисы import() можно заменить синхронными операторами require, import() в любом случае возвращается к require в Node.js.

1
Estus Flask 15 Май 2018 в 20:38