Repository navigation
cannot mixin and keep private properties #5070
Description
Hi,
I'm wondering how to achieve this ?
class Test {
public foo: string;
private quux: string;
// ...
}
class Plop {
public bar: string;
// ...
}
class TestPlopMixin implements Test, Plop {
public foo: string;
public bar: string;
private quux: string;
// Class TestPlopMixin incorrectly implements interface Test
// Types have separate declarations of a private property 'quux'
}Thanks
Activity
kitsonk commentedon Oct 2, 2015
I am not 100% certain, but I think you have fallen down a bit of a rabbit hole.
implementstreats the classes as interfaces (and not as classes) and you cannot have private members of an interface (because they aren't part of the exposed API). I think the only way you can solve it, is remove theprivatefrom theTestclass.I think logically it works, even if it was
class TestPlopMixin extends Testyou would run into trouble if you declared another "private" variable with the same name inTestPlopMixinbecause the implementation of the ancestor class is expecting no one else to access it, because it is private... The compiler is keeping you from "stomping" on something, because in your implementation, you shouldn't be using something that someone has "reserved" for their private use. This comes in with the whole challenge in JavaScript that there is not really "private" properties, so I think the TypeScript compiler is trying to keep you from doing something "silly", though obviously the error message lacks a bit of clarity of why it is doing what it is doing.paulsouche commentedon Oct 2, 2015
Thanks for the quick answer.
According to the doc TypeScript allow us to create mixins by creating classes that implements other classes.
I think this is already a bit "silly" for a non-JavaScript developer. So we cannot use this when an implemented class have private properties. Delete privates or make it public work but I don't think this is a good solution.
I'm OK with the idea that the mixin should not access the private of its ancestor but mixin objects that potentially expose "privates" properties is something daily for a JavaScript developer.
Personally I think that it should work without declaring the private on the
class TestPlopMixinas an interface implementation that only declare public properties, keep it private in theclass Testand just access the public API of the ancestor. But in this case it warns about the missing property.jbondc commentedon Oct 2, 2015
I'd say this is a bug, the check happens here:
https://github.com/Microsoft/TypeScript/blob/master/src/compiler/checker.ts#L12869
What should happen in the context of
implements classTypeis only check public members:https://github.com/Microsoft/TypeScript/blob/master/src/compiler/checker.ts#L5141
Can likely submit a patch if TS team agrees.
mhegazy commentedon Oct 2, 2015
The current behavior is by design. private members will stick on the type. if you do not want the privates to appear, do not implement the class, implement an interface that is structurally equivalent to the class's public shape.
Paul Souche (@paulsouche) it is not possible to "implement" a class with a private member. the idea is that privates are unique to a specific class, and can not be accessed, redefined, or assigned to from outside the class. the compiler enforces this by making sure that there is one and only one declaration of a private, and that any implementation of that type has to have the same declaration. when you implement a class, you have to redefine the property, and that will violate the rule.
for mixin purposes you should not use private members in your class. if you must, create interfaces to represent the public shape of the class, which is what your implementation class will use, and the mixin function can then mixin a class implementing the same interface.
paulsouche commentedon Oct 3, 2015
Yeah indeed it works with an interface but I think it's weirder to implements both interfaces and classes. As the implementation of the mixin is up to me, I will only use interfaces that represent the classes.
Thanks for your time on this question.
adhd-cook commentedon Jan 12, 2018
Although this is closed. This looks silly to me, the way we can implement mixins in any practical application using typescript does not make a lot of sense. Is there any plan to support mixins in a better way ?
kitsonk commentedon Jan 12, 2018
@love2dishtech Since TypeScript 2.2 Mixin classes have been supported and allow private members. Is there something missing in that pattern for you?