microsoft / microsoft/TypeScript
Checking if Enum | falsely in if condition if value of Enum might be zero
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- Go
- Estrellas
- 111k
- Forks
- 14.3k
- Merge medio
- 1 d 19 h
- PR fusionados (30 d)
- 117
Descripción
Suggestion
🔍 Search Terms
checker, enum, undefined, dx
✅ Viability Checklist
My suggestion meets these guidelines:
- This wouldn't be a breaking change in existing TypeScript/JavaScript code
- This wouldn't change the runtime behavior of existing JavaScript code
- This could be implemented without emitting different JS based on the types of the expressions
- This isn't a runtime feature (e.g. library functionality, non-ECMAScript syntax with JavaScript output, new syntax sugar for JS, etc.)
- This feature would agree with the rest of TypeScript's Design Goals.
⭐ Suggestion
Check if a value has enum | falsely types, and one member of the enum is zero. And the value In top level of If/while/etc's conditions.
It's a common bug case. Even after condition, CFA could strip the zero value. But If we does not use the enum value, we will never find it out.
📃 Motivating Example
enum E {
Foo,
Bar
}
function foo (v: E | undefined) {
if (!v) { // Should be suggestion or error?
throw new Error("Hoops")
}
console.log("Acceptable") // But without E.foo
}
💻 Use Cases
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 el ejemplo motivador E | undefined y revisa el comportamiento del checker para el estrechamiento por veracidad en condiciones if. Determina si el resultado debe ser una sugerencia o un error y, después, define pruebas que cubran un miembro del enum con valor cero y el tipo estrechado; el trabajo estará terminado cuando el comportamiento de diagnóstico previsto esté especificado y cubierto.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- typescript
- Área
- compilers
- Tipo de issue
- Nueva funcionalidad
- Dificultad
- 5/5
- Tiempo estimado
- Más de una semana
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 25/100