Почему не получается определить прототип обьекта?

Question

[nak-alexey]

Делаю таймер на JS с применением ООП. Когда хочу к TimerClass через переменную self добавить через прототип метод run выдает ошибку а если добавлю без прототипа все норм, помогите пожалуйста.

var TimerClass = function($node, $days){    
    var self = this;
    
    function Timer(){
        this.node   = document.getElementById($node);
        this.DOM    = [];
        this.interval;
    }
    Timer.prototype = {
        init:   function(){
            console.log('Timer create!!!');
        },
        run:    function(){
            interval = setInterval(timer, 1000);
            function timer(){                
                console.log('Hello! You run timer!!!');
            }
        },
        stop:   function(){
            clearInterval(this.interval);
        }
    };
    function constructor(){
        var object = new Timer();
            object.init();
                           
        self.run = object.run;
        //self.prototype.run = object.run;                
    };    
    return constructor();
};

var timer = new TimerClass('node', 12);   
    timer.run();

Comments

[Dark Hole]

Хороший сборник вредных способов. Начнем с того, что инкапсулировать класс в класс... Э. Зачем? Ровно никакой инкапсуляции не будет. Просто TimerClass будет делать объекты Timer. Что уже не очевидное решение. +доступа к прототипу Timer не будет, т.к. у каждого объекта будет свой прототип.
var TimerClass = function($node, $days){
Во-первых, зачем нужно объвлять функцию через var? Во-вторых, это не PHP, $ в начале не нужны. $days вообще не используется...
this.interval;
Нет, свойство это не создаст.
self.run = object.run;
Мы выкидываем старый контекст при return, что мы вообще пытаемся этим сделать?
По-хорошему надо было бы так:

код

function Timer(node, days){
  this.node   = document.getElementById($node);
  this.DOM    = [];
  this.interval = 0;
  console.log('Timer create!!!');
};
Timer.prototype = {
  run:    function(){
    interval = setInterval(timer, 1000);
    function timer(){                
      console.log('Hello! You run timer!!!');
    }
  },
  stop:   function(){
    clearInterval(this.interval);
  }
}

var timer = new Timer('node', 12);   
    timer.run();

Идеальнее можно сделать только на ES6 классах

Answer 1

[Dima Polos]

Вы обращаетесь к классу, а не к объекту, Timer это класс, Вы к нему обращаетесь через prototype, а самого объекта еще нет. Я думаю, что в этом ошибка.

Comments

[Dima Polos]

Если Вы без прототипа обратитесь, то добавите свойства просто как к объекту, ассоциативному массиву.

[Антон Спирин]

В корне неверно. На момент вызова есть оба инстанса.

[Dima Polos]

Антон Спирин, Антон, вполне может быть, не спорю, надо потестить, эту тонкость мог упустить. Посмотрев на код меня смутило, что к прототипу объекта обращаются до его создания с помощью оператора new. На досуге гляну. Спасибо!