microsoft / microsoft/TypeScript
Uninitialized local check still permits provably erroneous use-before-assign in closure
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- Go
- Estrellas
- 111k
- Forks
- 14.4k
- Merge medio
- 1 d 19 h
- PR fusionados (30 d)
- 117
Descripción
🔎 Search Terms
uninitialized, local, use before assign, control flow
🕗 Version & Regression Information
- This changed in commit or PR https://github.com/microsoft/TypeScript/pull/55887
⏯ Playground Link
💻 Code
let foo: string;
const setFoo = (): void => {
if (foo.length < 1) {
foo = 'foo';
}
};
export {}
🙁 Actual behavior
No TS Error
🙂 Expected behavior
Variable 'foo' is used before being assigned.(2454)
Additional information about the issue
(this error is correctly reported if we never assign at all, as below)
let foo: string;
const setFoo = (): void => {
if (foo.length < 1) {
}
};
export {}
Related, https://github.com/microsoft/TypeScript/issues/60064.
Otherwise this can be solved by not considering assignments which occur unconditionally after a read.
Guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Línea de trabajo
Comienza con la reproducción enlazada de TypeScript Playground y compárala con el ejemplo adicional que ya informa diagnostic 2454. Lee el cambio de control-flow en PR 55887 y el issue relacionado 60064; se considera terminado cuando la lectura desde la closure informa use-before-assign sin romper el comportamiento existente del diagnóstico no-assignment.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- typescript
- Área
- compilers
- Tipo de issue
- Error
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 38/100