microsoft / microsoft/TypeScript
Never-initialized variable type is not narrowed to undefined
Chưa có ai nhận issue này.
- Ngôn ngữ chính
- Go
- Star
- 111k
- Fork
- 14.3k
- Merge trung bình
- 2 ngày 4 giờ
- Pull request đã merge (30 ngày)
- 132
Mô tả
🔎 Search Terms
narrow uninitialized, uninitialized undefined
🕗 Version & Regression Information
- This is the behavior in every version I tried, and I reviewed the FAQ for entries about
initialized/uninitialized
⏯ Playground Link
💻 Code
const foo = (two: string): void => {
"use strict";
// eslint-disable-next-line @typescript-eslint/init-declarations
let one: string | undefined;
if (one === two) {
// ^? let one: string | undefined
// Noop
}
}
foo("");
🙁 Actual behavior
The type is not narrowed, which prevents typescript-eslint from flagging this comparison that always fails: typescript-eslint playground
🙂 Expected behavior
The one === two comparison should be flagged because it always returns false (there is no way one has any value other than undefined).
Additional information about the issue
Also filed a related issue with ESLint: https://github.com/eslint/eslint/issues/19581
Hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Hướng nghiên cứu
Tái hiện hành vi bằng liên kết TypeScript Playground và ví dụ one === two, sau đó truy vết cách việc narrowing của luồng điều khiển xử lý một biến chưa bao giờ được khởi tạo. So sánh kết quả với phép so sánh được mong đợi là luôn false và sử dụng issue ESLint được liên kết để có thêm ngữ cảnh liên quan. Được xem là hoàn tất khi phép so sánh được nhận diện là không thể xảy ra mà không thay đổi hành vi narrowing không liên quan.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Đánh giá
- Công nghệ
- typescript
- Lĩnh vực
- compilers
- Loại issue
- Lỗi
- Độ khó
- 4/5
- Thời gian dự kiến
- 3-5 ngày
- Mức độ hoạt động
- Đình trệ
- Độ rõ ràng
- Khá rõ ràng
- Mức phù hợp với người mới
- 45/100