Почему такой паттерн не корректен?

Question

[Rapen]

var obj = {};
  
var Singleton = function(){
 if(!obj) { 
var obj = {};
}
 
  return obj;
  
  }
  
var o  = new Singleton();
var o1 = new Singleton();

o.prop = 's'; 
console.log(o === o1);  //false
console.log(o1.prop); //undefined

Comments

[Kovalsky]

1. В конструкторе не надо возвращать ничего,
2. Зачем нужна проверка if (!obj)? Приведение объекта к булеву значению всегда вернет true,
3. Запоминать инстанс класса лучше например в замыкании, просто в переменной - это очень ненадёжно

[Kovalsky]

Собственно, наверное, не стоило так категорично утверждать, что в конструкторе нельзя возвращать: это делать можно, но предварительно надо ознакомиться вот с этим. Вкратце: при вызове конструктора создается новый объект, ссылка на него лежит в переменной this. Если вы возвращаете из конструктора не объект, то указанное вами возвращаемое значение будет проигнорировано и вернется this. А если объект, то вроде бы никаких нюансов здесь нет, возвращается именно этот объект, разве что связи объекта не будут никак изменены, он будет инстансом того класса, какого был при возврате.

Answer 1

[Толстый Лорри]

Потому что объекты (o и o1) разные, внутри каждого свой obj.

Сама идея синглтона, что это один объект(типа) на всю программу.

function Singleton () {
    if (Singleton._instance) return Singleton._instance;
    Singleton._instance = this;
};

var o = new Singleton();
var o1 = new Singleton();
console.log(o === o1); // true

o.prop = 's';
console.log(o1.prop); // s

Comments

[Kovalsky]

Кроме всего прочего, этот код добавляет метод getInstance к глобальному объекту window, в переменной Singleton хранится ссылка на window, o не является инстансом Singleton, хотя бы потому что Singleton это не конструктор. Вы так и планировали или я чего-то не понял?

[Толстый Лорри]

Никита Полевой: Да, спасибо, была странная фабрика.

Answer 2

[Mikhail Osher]

<сарказм>Потому, что синглтон - антипаттерн.</сарказм>

А вообще, почитайте вот что:
- функции-конструкторы
- hoisting

Answer 3

[T_y_l_e_r]

Синглтон возвращает всегда только 1 обьект а не создает новый.
У вас при каждом new генерируется новый обьект

Comments

[Kovalsky]

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

[Rapen]

А как именно, при том условии, если конструктор возвращает obj из глобального объекта, в принципе новые объекты должны же ссылаться туда?