Javascript прототип класса Литерал лучшие практики - PullRequest
Новая
       4

Javascript прототип класса Литерал лучшие практики

0 голосов
/ 15 марта 2011

Я новичок в Javascript, поэтому мне интересно, является ли следующий код хорошей практикой. 

CustomClass = function(var1, var2) {
    this.var1 = var1;
    this.var2 = var2;
};
CustomClass.prototype.aMethod = function() {
    console.log("my class method");
};

// Intend as main .js object, if that makes sense
var m = {
    object1:CustomClass.prototype,
    object2:CustomClass.prototype,

    initObjects:function() {
        m.object1 = new CustomClass( value1, value2 );
        m.object1.aMethod();
        m.object2 = new CustomClass( value1, value2 );
        m.object2.aMethod();
    }
};

Или я должен создать свой собственный класс внутри литерала "s"?

Любая помощь будет высоко ценится

1 Ответ

0 голосов
/ 15 марта 2011

Что касается вашего фактического кода: 

// globally scoped. Only avoided with closures and hoisting as shown below
// Also doesn't really make sense to define CustomClass on m unless you want
// To use it directly instead of through a factory function
CustomClass = function(var1, var2) {
    this.var1 = var1;
    this.var2 = var2;
};
CustomClass.prototype.aMethod = function() {
    console.log("my class method");
};

// Intend as main .js object, if that makes sense
var m = {
    // No reason to initialise them to the prototype. Doesn't really make sense
    // m can be edited without needing to intialise object1 or object2 at all
    object1:CustomClass.prototype,
    object2:CustomClass.prototype,

    initObjects:function() {
        m.object1 = new CustomClass( value1, value2 );
        m.object1.aMethod();
        m.object2 = new CustomClass( value1, value2 );
        m.object2.aMethod();
    }
};

Другой шаблон, который может быть полезен, основанный больше на затворах и подъемах.Это требует более функционального подхода и отходит от стандартного классического наследования. 

// Create closure to localise scope.
(function() {
    // global object to store anything that will be hoisted to global scope
    var global = {};

    // Constructor that uses local objects and defines methods on
    // this directly.
    var CustomClass = function(_a,_b) {
         var a = _a;
         var b = _b;

         this.method = function() { console.log("foo"); }
    }

    // init function will be hoisted to global scope.
    global.init = function() {
         global.object1 = new CustomClass( v1, v2 );
         object1.method();
         global.object2 = new CustomClass( v1, v2 );
         object2.method();
    }

    // Hoist you global object into the global variable "m" on window.
    window.m = global;
}());

Конечно, вы теряете цепочку прототипов, поэтому вы должны использовать гораздо больше композиции объектов вместо наследования объектов, и это также медленнеенемного.Потеря скорости заметна только в том случае, если вы создаете 1000+ объектов

...