microsoft / microsoft/TypeScript

Should disallow using the result of calling a void-returning function

Đang mở
#40,997 0 bình luận 2 reaction 0 người được giao Xem trên GitHub

Chưa có ai nhận issue này.

Awaiting More Feedback Suggestion
Ngôn ngữ chính
Go
Star
111k
Fork
14.4k
Merge trung bình
1 ngày 19 giờ
Pull request đã merge (30 ngày)
117

Mô tả

TypeScript Version: 4.1.0-dev.20201008

Search Terms: disallow function call void return usage warn error

Code

function doesntReturn() {
    // no return statement
}

const foo: string = doesntReturn() + 'foo'; // should be an error

Expected behavior:

The expression doesntReturn() + 'foo' should be an error because it uses the value of a void-returning function call. As far as I can tell, there is literally no good reason to use the return value of a function with no return statements like this.

This is similar to the current behavior of disallowing void-returning expressions inside control flow (issue #26262). It is also similar to the reasoning behind the warning that occurs when using a function value (instead of calling it) in an if condition.

Actual behavior:

No error message is given.

Playground Link: here

Related Issues: #26262

Hướng dẫn đóng góp

Mở hướng dẫn đóng góp

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. 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.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Hướng nghiên cứu

Bắt đầu với ví dụ Playground được liên kết để tái hiện chẩn đoán còn thiếu, sau đó đọc issue liên quan #26262 về hành vi control-flow tương tự. Công việc được xem là hoàn tất khi biểu thức sử dụng kết quả của một lệnh gọi hàm trả về void báo lỗi, trong khi ví dụ hiện có vẫn tiếp tục hoạt động như mong đợi.

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
Tính năng
Độ 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
38/100

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.