Как лучше инициализировать объект?

Question

[VolgaVolga]

Сразу после создания объекта TypeScript необходимо вызвать метод init. Как правильнее это сделать, чтобы не лепить вызов в тело конструктора и не делая метод public?

Comments

[Lynn «Кофеман»]

А почему бы не вызвать его в конструкторе?

[VolgaVolga]

Lynn «Кофеман», типа, антипаттерн...

[szQocks]

VolgaVolga, сделай метод init приватным, и вызывай его при помощи другого метода ( публичного ) в конструкторе, тогда получиться что и метод init приватный, и вызываешь его при создании инстансов

[VolgaVolga]

szQocks, но это все равно вызов метода в конструкторе, что не приветствуется.

[Lynn «Кофеман»]

Ну можно как-то так
https://www.typescriptlang.org/play?#code/MYGwhgzh...

class Foo {
    constructor() {
        // do something
    }

    protected init() {
        // init more
    }

    static CreateWithInit() {
        const foo = new Foo();
        foo.init();
        return foo;
    }
}

const myFoo = Foo.CreateWithInit();

[szQocks]

VolgaVolga,

и вызывай его при помощи другого метода ( публичного ) в конструкторе

тогда вызывай его не в конструкторе, не ?

[VolgaVolga]

Lynn «Кофеман», фабричный статический метод может подойти. Спасибо.

[Lynn «Кофеман»]

Или даже так
https://www.typescriptlang.org/play?#code/MYGwhgzh...

Answer 1

[VoidVolker]

типа, антипаттерн...

но это все равно вызов метода в конструкторе, что не приветствуется.

Бред. Вот кем не приветствуется - вот он пускай и не приветствует и тихо миро плавает в своем болотце. Спокойно вызывайте приватный инициализатор в конструкторе - это как раз нормально. Конструктор - это и есть инициализатор объекта. Он даже называется "конструктор" - сборщик объекта. Его для этого и придумали. Выносить часть инициализации из инициализатора - на самом неделе не самая умная идея. А вот разбить инициализацию на разные логические части и разнести в разные методы - как раз таки очень даже хорошая практика. Декомпозиция, небольшие и простые функции/процедуры и все такое прочее.

Comments

[VolgaVolga]

Логично, только вызываемый метод не столько конструирует объект, сколько вешает обработчики событий.

[VoidVolker]

Так и пусть вешает. Разве ж проблема это? Вот если надо хэндлеры цеплять отдельно по каким-то специфичным причинам - то тогда да, использовать отдельный публичный метод самое то. Я всегда так и делаю:

constructor(selectors) {
    this.fieldsInit()
    this.A = this.attachA(selectors.A)
    this.B = this.attachB(selectors.B)
}