microsoft / microsoft/TypeScript
False positive ts(2373)
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- Go
- Sterne
- 111k
- Forks
- 14.4k
- Ø Merge
- 1 T. 19 Std.
- Gemergte PRs (30 T.)
- 117
Beschreibung
🔎 Search Terms
Parameter cannot reference identifier declared after it, ts(2373)
🕗 Version & Regression Information
This is the behavior in every version I tried, and I reviewed the FAQ for entries about anything related (and found nothing).
⏯ Playground Link
💻 Code
/**
* This function throws `ReferenceError: Cannot access 'length' before initialization` at runtime.
*
* TS correctly reports `ts(2373)`. 👍
*/
export const truePositive = <const T>({ [length - 1]: finalItem, length }: readonly T[]): T | undefined => {
return finalItem;
};
/**
* This function works fine at runtime.
*
* TS **in**correctly reports `ts(2373)`. 👎
*/
export const falsePositive = <const T>({ length, [length - 1]: finalItem }: readonly T[]): T | undefined => {
return finalItem;
};
🙁 Actual behavior
A false positive ts(2373) is being reported in a situation where we can determine statically that what the report warns about is not taking place.
Ironically, the false positive ts(2373) might represent a real bug in TS' own modeling of declaration order.
🙂 Expected behavior
No false positive ts(2373) being reported.
Additional information about the issue
To be clear:
In the falsePositive case, we can statically determine that length is declared prior to being used.
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne mit der verlinkten TypeScript Playground-Reproduktion und vergleiche die Parameter-Muster von truePositive und falsePositive. Verfolge die Implementierung der Diagnose ts(2373) und füge einen Regressionstest für den gezeigten Fall der Deklarationsreihenfolge hinzu. Als abgeschlossen gilt die Aufgabe, wenn das falsePositive-Beispiel keine Diagnose erzeugt, während truePositive weiterhin die Laufzeitgefahr meldet.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- typescript
- Bereich
- compilers
- Issue-Typ
- Bug
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 35/100