Instance properties declaration and initialization in constructor failed with useDefineForClassFields turned on

[silentroach]

TypeScript Version: 3.7.2

Search Terms: useDefineForClassFields, constructor

Code

class Test {
    constructor(public readonly something: number) { }
}

const k = new Test(5);

Expected behavior: I expect k.something to be 5

Actual behavior: k.something is undefined, cause TS:

1. writes the value to the property (as it was before)
2. and then rewrite it (with writable: true, I think it is bug too) to void 0

class Test {
    constructor(something) {
        this.something = something;
        Object.defineProperty(this, "something", {
            enumerable: true,
            configurable: true,
            writable: true,
            value: void 0
        });
    }
}

Playground Link: https://www.typescriptlang.org/play/?useDefineForClassFields=true&target=7&ssl=3&ssc=2&pln=1&pc=1#code/MYGwhgzhAEAqCmEAu0DeAoaXrAPYDtkAnAV2CVyIAoAHEgIxAEthoj4wATAkAT2gi4AtvCQALJvgDmALmj4SQ+vCIBKNNAC+6TUA

[RyanCavanaugh]

writable: true is intentional; readonly properties may be reassigned multiple times in the constructor body.

The order is definitely wrong though

[shayded-exe]

Yeah this is a massive issue. It makes useDefineForClassFields: true unusable.

[RizkyArifNur]

for now, you can avoid it by using this solution

class Test {
    readyonly something: number
    constructor(something: number) {
        this.something = something 
}
}

code above will fix the wrong ordering of definition

code playground: https://www.typescriptlang.org/play/?useDefineForClassFields=true&target=7&ssl=10&ssc=2&pln=5&pc=1#code/MYGwhgzhAEAqCmEAu0DeAoaXrAPYDtkAnAV2CVyIAoAHEgIxAEthoj4wATAkAT2gi4AtvCQALJvgDmALmj4SQ+vCIBKNNAC+6belCQYAQXy5xKhMjSYs7Lj36CR4yVICEchUpXWseQklJySipHUQlpD0VlNStsOOhnCAA6UOdpaABeAWEwlx8sbU0gA