-
Notifications
You must be signed in to change notification settings - Fork 12.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. Weβll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Allow non-polymorphic ("private") extension of base classes #57979
Comments
This is also possibly related to #4628. Not in the sense that my issue is affected by static inheritance, but because if this issue were resolved by the addition of something like the |
It occurs to me that one radical way to solve this problem would be to add some syntax that allows you to suppress creating the interface portion of the class altogether. Then you could simply define the interface separately with whatever subset of the actual functionality you want to expose, and then the ES6 class definition just gets matched against that interface, rather than generating the interface itself. ...Huh. This actually feels like a useful and actionable feature. You could rewrite the original example like this: // This line is the only definition of the MyTuple<T> type
export interface MyTuple<T> extends ReadonlyArray<T> { }
// A private class is one whose types are not visible to the outside. It has signature { } or possibly object.
export private class MyTuple<T> extends Array<T>
// This one implements the MyTuple<T> interface to ensure that it matches its public API
implements MyTuple<T> {
constructor(...args: readonly T[]) {
super();
this.push(...args);
Object.freeze(this);
}
} What do you think? |
@dmchurch You can use a class expression to make it "private". // This line is the only definition of the MyTuple<T> type
export interface MyTuple<T> extends ReadonlyArray<T> { }
// A private class is one whose types are not visible to the outside. It has signature { } or possibly object.
export const MyTuple = class<T> extends Array<T>
// This one implements the MyTuple<T> interface to ensure that it matches its public API
implements MyTuple<T> {
constructor(...args: readonly T[]) {
super();
this.push(...args);
Object.freeze(this);
}
} Note that you MUST NOT use the name |
@whzx5byb Yes, I've used that workaround before, and it's not applicable in this case. I need the emit to contain an ECMAScript top-level class export, which has slightly different semantics than an anonymous class assigned to a const (or a let, or a var, etc etc). If you have any suggestions that actually fit my needs, I'm all ears. |
I was looking for something similar, I wanted my subclass to be a readonly array. After playing with all the possibilities I managed frankenstein this together from the bits an pieces on StackOverflow and this repos issues. function ReadonlyArrayCtor(): { new <T>(...items: readonly T[]): ReadonlyArray<T> } {
return Array as any;
}
export class MyTuple<T> extends ReadonlyArrayCtor()<T> {
constructor(...args: readonly T[]) {
super(...args);
Object.freeze(this);
}
}
let vec = new MyTuple(1, 2);
vec[1] = 100; // error Kinda works. But couldn't make it an actual tuple though. Even casting the constructor factory to function ReadonlyArrayCtor(): { new (x: number, y: number): readonly [number, number] } {
return Array as any;
}
export class MyTuple<T> extends ReadonlyArrayCtor() {
constructor(x: number, y: number) {
super(x, y);
Object.freeze(this);
}
}
let vec = new MyTuple(1, 2);
let x = vec[0]
let y = vec[1]
let z = vec[2] // no error
vec[1] = 100; // error |
π Search Terms
private non-polymorphic non-assignable inheritance extension subclass change method signature of base class
β Viability Checklist
β Suggestion
I would like to write an ES6 class that does not inherit the typing of its base class (and thus doesn't expose any properties or methods defined in the base class, unless they are overridden in the subclass). In C++ I would call this "private inheritance", but in short: I would like to be able to use the implementation of a class (by using the
super
keyword in a method) without declaring my instances to be assignment-compatible.π Motivating Example
To the best of my knowledge, there are currently no ways to express this in TypeScript. In pure JavaScript, I can write the following:
Instances of the
MyTuple
class, despite extending theArray
prototype, don't conform to the Array contract, as all of the mutation methods will throw a TypeError. Even worse, direct array element assignment will silently fail, as assignment to a read-only property is simply ignored. However, the only way to get TypeScript to output this JS code is to use a TypeScript class declaration, and these always propagate the base class typings.In this particular case, the class is a better match for the
ReadonlyArray
interface, so ideally I could write the following in TypeScript:π» Use Cases
In my current project, I'm writing a variable-dimensionality geometry library in which the
Point
class is a specialization ofArray<number>
. Point is a generic class with the signaturePoint<Dims extends number>
, so a 2D point is aPoint<2>
, a 3D point is aPoint<3>
, etc. Using Array as the base class provides a number of advantages, like being able to use.map()
in order to create a new Point with the same dimensionality, based on the existing. However, most of the Array functionality shouldn't be exposed to consumers; in particular, anything that modifies the length of the array or creates an array of different length (push
,slice
, etc) would cause improper typing.Well, first off, the return type of
.map()
is incorrect because of Array inheritance in ES6 declaration file (lib.es6.d.ts)Β #10886; it and all the other copy-instantiating methods should return aPoint
but instead returnnumber[]
. I can't usedeclare
to override the method signature directly because of Providedeclare
keyword support for class methodsΒ #38008 and Allow 'declare' on methods and constructorΒ #48290, and TS won't allow me to override it as a property because the types don't match.I'm removing the typing entirely by casting
Array
to a dummy constructor ofObject
:readonly number[]
, but that didn't work because, as mentioned above, I need to declaremap()
as a method that returns a Point, and the existingnumber[]
return signature can't be cast to a Point derived fromreadonly number[]
. So now I'm casting it to Object, which comes with a fairly serious drawback: I can no longer use thesuper
keyword in my methods, because TS now thinks thatsuper
is type Object, and I can't even correct it - thesuper
keyword can't have a type assertion applied to it, as mentioned in Allow type assertion for "super" keyword in method.Β #41034.Now I go to write a lot of type assertions and
// @ts-expect-error
comments, because so far as I can tell, there is no other way around this.The text was updated successfully, but these errors were encountered: