NativeScript / NativeScript/android
Improve support for Java varargs methods
Chưa có ai nhận issue này.
- Ngôn ngữ chính
- C++
- Star
- 563
- Fork
- 144
- Merge trung bình
- 10 giờ 46 phút
- Pull request đã merge (30 ngày)
- 14
Mô tả
Java supports varargs methods. For example, [Class.getMethod](https://developer.android.com/reference/java/lang/Class.html#getMethod%28java.lang.String, java.lang.Class<?>...%29)
Method getMethod (String name, Class...<?> parameterTypes)
This is syntax sugar for the actual method signature which is
Method getMethod (String name, Class<?>[] parameterTypes)
Currently, NativeScript for Android does not support calling such method with implicit parameter
var klass = java.lang.Object.class;
// option 1) not supported
var method = klass.getMethod("hasCode");
// option 2) supported
var method = klass.getMethod("hasCode", null);
We can provide another implementation for the current method resolution algorithm so it tries to find a method with one additional parameter (array) as a fallback and if this fail then the method resolution fails as expected.
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
Bắt đầu với thuật toán phân giải phương thức hiện tại trong NativeScript Android runtime và theo dõi cách các lệnh gọi JavaScript được đối sánh với các phương thức Java. Xác minh hành vi bằng các ví dụ varargs ngầm định và tường minh trong issue. Hoàn thành khi một đối số varargs bị bỏ qua được phân giải thông qua fallback của tham số mảng, trong khi các phương thức không phân giải được vẫn thất bại bình thường.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Đánh giá
- Công nghệ
- cpp, java, javascript
- Lĩnh vực
- mobile-dev
- 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
- 35/100