mixin в atom.declare

[llevkin]

Здравствуйте theshock! Прежде всего огромное спасибо за такие замечательные библиотеки - atom.js и libCanvas.

Скажите, судя по коду, atom.declare должен поддерживать свойство mixin, но при подмешивании к классу свойства не появляются. В документации mixin для atom.declare отсутствует. Делаю так:

atom.declare('Test', { test: function() { console.log(1); }, });

atom.declare('Test2, { mixin: [Test], prototype: { initialize: function() { console.log(this.test); //undefined }, }, });

Должен ли поддерживать atom.declare mixin? или это бага? Заранее спасибо.

[theshock]

Извините за столь долгий ответ. Я им давно не пользовался, видимо, это таки баг Миксина. Скорее всего у меня есть мысли вообще удалить Миксин из declare.

[theshock]

Необходима ли эта функциональность?

[llevkin]

На мой взгляд, это очень полезная функциональность, в плане того, что удобно реализовывать общую интерфейсную базу для похожих сущностей, а так же разбивать большие классы на модули. Так же если нет необходимости использовать в проекте mixin, то на мой взгляд проверка на отсутствие свойства mixin не сильно снизит производительность фрэймворка в целом..

В любом случае это имхо, автор фрэймворка - вы и решать вам)

Покопавшись немного в atom.js мне таки удалось заставить mixin работать... Правда в моём случае это получилось довольно (костыльно)/(не элегантно)/(не судите строго) =), ибо не очень хорошо ориентируюсь в atom.js изнутри. На всякий случай выложу участок кода (примерно 1851 строка):

mixin: function (target, items) { if (!Array.isArray(items)) items = [ items ]; for (var i = 0, l = items.length; i < l; i++) { // My code start here var source = methods.proto(items[i]); for (j in source) if (j != 'initialize' && j != 'constructor' && j != 'bindMethods' && j != 'toString' && typeof source[j] === 'function') target.prototype[j] = source[j]; // My code end here

        // original code start here
        //methods.addTo( target.prototype, methods.proto(items[i]) );
        // original code end here
    }
    return this;
},

Если не ошибаюсь, то в вашем случае hasOwnProperty - отсеивает таки методы прототипа mixin.

С уважением, Леонид.

[llevkin]

поправка с учётом аксессоров.. for (j in source) { if (!atom.accessors.inherit(source, target.prototype, j) && j != 'initialize' && j != 'constructor' && j != 'bindMethods' && j != 'toString' && typeof source[j] === 'function') target.prototype[j] = source[j]; }