B extends A, but B.add populates A.prototype.property

后端 未结 2 1334
不知归路
不知归路 2021-01-16 10:07

I have one class and another that inherits property children from the first one.

function A() {}
A.prototype.children = [];

function B() {}
B.prototype = ne         


        
相关标签:
2条回答
  • 2021-01-16 10:39

    There is only one children array created in your script, but it is referenced by each instance (and even B's prototype) due to inheritance. When you push to it, you will see the changes from everywhere as well.

    Instead, give every instance its own array:

    function A() {
        this.children = [];
    }
    

    And also, don't create only one array for all B instances to inherit from with new A - instead, use

    function B() {
        A.call(this); // do everything the A constructor does on this instance
    }
    B.prototype = Object.create(A.prototype);
    B.prototype.addChild = function(Child) {
        this.children.push(Child);
    };
    
    0 讨论(0)
  • 2021-01-16 11:03

    You shouldn´t be using the prototype to store data that is for the instance. When you do this.children, there are no children in B, thus the prototype chain continues to A. As suggested by @Bergi, you should remove:

    B.prototype = new A
    

    Try defining:

    function A() {
      this.children = [];
    }
    A.prototype.addChild = function (o) { this.children.push(o)};
    var b = new A();
    b.addChild({});
    
    0 讨论(0)
提交回复
热议问题