0

Хочу автоматически создать классы и распихать их по переменным. Пример:

Есть переменная var classes = {}; и есть классы MyClass, MyClass2, MyClass3, MyClass4. Все они - синглтоны.

Хочу инициализировать все классы и записать в этот объект так, чтоб обращаться к классам через classes.MyClass, classes.MyClass2 и т.д.

Вот пример класса (другие классы подобные):

let _myClassSingleton = Symbol();

class MyClass {

    constructor(enforcer) {
        if (enforcer != _myClassSingleton) 
            throw "Cannot construct singleton";
    }

    static getInstance() {
        if(!this[_myClassSingleton]) {
            this[_myClassSingleton] = new MyClass(_myClassSingleton);
        }
        return this[_myClassSingleton];
    }

    someMethod() { console.log('MyClass/someMethod'); }
}

Т.е. обратиться к экземпляру можно только так MyClass.getInstance();

Имена всех классов записаны в массиве, т.е.

let applicationClasses = ['MyClass', 'MyClass2', 'MyClass3', 'MyClass4'];

Как мне инициализировать все эти классы динамически и сложить в объект classes ?

Пытаюсь через цикл:

for (let className of applicationClasses) {
    // код инициализации классов
}

Но если в цикле писать так:

classes[className] = className.getInstance();

будет выдавать, конечно же, ошибку, что нет метода getInstance. Что логично. Ибо это получается, что вызываем метод у строки.

Можно написать так:

classes[className]  = eval(className + ".getInstance()");

тогда будет все будет работать и можно будет обращаться к классам через classes.MyClass, classes.MyClass2 и т.д. Однако через eval не хочется делать.

Вопрос: можно ли как-то еще сделать данную вещь или единственный вариант в данном случае только eval ?

4
  • какой-то у тебя неправильный синглтон, класс MyClass, а возвращаешь объект Settings
    – Grundy
    22 авг 2016 в 8:29
  • @Grundy прошу прощения. Просто для примера переименовывал имя. Недопечатал 22 авг 2016 в 9:52
  • классы объявлены в глобальном контексте? или внутри каких-нибудь самовызывающихся функций?
    – Grundy
    22 авг 2016 в 10:03
  • @Grundy глобально. собраны в одном файле 22 авг 2016 в 10:41

2 ответа 2

2

Вы все переусложняете.

Во-первых, нет смысла заводить класс для синглтона:

var MyObject = {
  someMethod() { console.log('MyClass/someMethod'); },
  // И никаких конструкторов с getInstance!
};

Во-вторых, нет никакого смысла писать имена переменных в массиве:

var objects = {
   MyObject: MyObject,
   MyObject2: MyObject2,
   MyObject3: MyObject3,
   MyObject4: MyObject4,
};
9
  • в третьих можно использовать computed property
    – Grundy
    22 авг 2016 в 8:19
  • Да, знаю, что можно писать var MyObject = { someMethod() { ... }, }; Прост хотел сделать это на уровне ES6. Может это лишняя гонка за новым и привычном глазу. По поводу var objects = { MyObject: MyObject, MyObject2: MyObject2, ... }; тоже знаю. Но для чего я хотел через цикл делать-чтоб как бы автоматически пробежалось и всё подключилось. Типа красивее выглядит, нежели вручную такой столбик городить некрасивый. То есть предлагаете принебречь излишней красотой в виде массива и цикла в угоду простоты добавления нужных классов в столбик? Типа не экономь на спичках? :) 22 авг 2016 в 9:47
  • 1
    @СергейМишин так ведь массив-то вы автоматически все равно не сформируете 22 авг 2016 в 9:48
  • @PavelMayorov да. но он просто спокойно в одну строку написан и не занимает много места :) 22 авг 2016 в 9:49
  • 2
    @СергейМишин литерал объекта тоже можно в одну строку написать :) 22 авг 2016 в 9:49
1

Вместо строк, в массиве можно хранить непосредственно ссылки на классы, тогда работа с ними упроститься до работы с элементами массива

let _myClassSingleton = Symbol();
class MyClass {
  constructor(enforcer) {
    if (enforcer != _myClassSingleton)
      throw "Cannot construct singleton";
  }
  static getInstance() {
    if (!this[_myClassSingleton]) {
      this[_myClassSingleton] = new MyClass(_myClassSingleton);
    }
    return this[_myClassSingleton];
  }
  someMethod() {
    console.log('MyClass/someMethod');
  }
}
let _myClassSingleton2 = Symbol();
class MyClass2 {
  constructor(enforcer) {
    if (enforcer != _myClassSingleton2)
      throw "Cannot construct singleton";
  }
  static getInstance() {
    if (!this[_myClassSingleton2]) {
      this[_myClassSingleton2] = new MyClass2(_myClassSingleton2);
    }
    return this[_myClassSingleton2];
  }
  someMethod() {
    console.log('MyClass2/someMethod');
  }
}

let applicationClasses = [MyClass, MyClass2];

let classes = applicationClasses.reduce((classes, c) => {
  classes[c.name] = c.getInstance();
  return classes
}, {});

classes.MyClass.someMethod();
classes.MyClass2.someMethod();

2
  • я сглупил и забыл что в массиве/объекте можно хранить имя класса не как строку а ссылку. Так что этот вариант подходит. Спасибо. 22 авг 2016 в 11:47
  • @СергейМишин, в любом случае это больше из разряда потому что могу, а не из-за какой-то практической ценности :)
    – Grundy
    22 авг 2016 в 11:49

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.