Я бы пожалуй использовал это так:
var myModuleNS = extend(myApp, 'modules.module2');
myModuleNS.method = function() {};
//возможно лучше сразу перезаписать прото и не мучиться.
//Но это опасный некроссбраузерный код.
myModuleNS.__proto__ = {
method1: function() {},
publicVar1: 1,
__proto__: myModuleNS.__proto__
}
Этот способ мне кажется не очень удобным (это так же отмечено в оригинальной статье в следующих строках: "This works exceedingly well when defining purely namespaces alone, but can seem a little less flexible when you want to define both functions and properties at the same time as declaring your namespaces."), я бы пожалуй модифицировал функцую следующим образом:
function extend(ns, ns_string, value) {
var parts = ns_string.split('.'),
parent = ns,
pl, i;
if (parts[0] == "myApp") {
parts = parts.slice(1);
}
pl = parts.length;
for (i = 0; i < pl; i++) {
//create a property if it doesnt exist
if (typeof parent[parts[i]] == 'undefined') {
if (value !== undefined && i==pl-1) parent[parts[i]] = value;
else parent[parts[i]] = {};
}
parent = parent[parts[i]];
}
return parent;
}
Теперь этим вполне можно пользоваться:
var myModule = extend(myApp, 'modules.myModule', new MyModule());
//это так же подойдет для расширения существующего пространства имен методами
extend(myApp, 'modules.myModule.method', function myNewMethod() {});
Такая конструкция имеет ряд недостатков, например повторная попытка сделать extend myModule, вернет результат первой такой попытки, я бы не ожидал подобного действия от кода.
Конструкция описанная
Сергей Мельников , сомнительна, он рекомендует вам с помощью этой функции
получать методы искомого модуля. Теперь давайте представим, что запрашиваемого метода не существует - в ответ на extend вы получите пустой объект и попытаетесь его вызвать, а это ой как не понравится программе...
В заключении:
Если бы я и использовал этот паттерн, то пожалуй только внутри конструктора синглтона и только в предложенной мной модификации:
function MyModule() {
var module = extend(NS, 'modules.MyModule', this);
if (module !== this) return module;
//body of module
}
