Avoid Throwing Exceptions
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 4/5
- Thời gian dự kiến
- 3-5 ngày
- Mức phù hợp với người mới
- 28/100
- Loại issue
- Tái cấu trúc
- Độ rõ ràng
- Khá rõ ràng
- Mức độ hoạt động
- Đình trệ
- Công nghệ
- objective-c
- Lĩnh vực
- authentication, mobile
Hướng nghiên cứu
Bắt đầu bằng cách đọc GoogleSignIn/Sources/GIDSignIn.m và theo dõi nơi các ngoại lệ được cung cấp cho bên gọi. So sánh luồng đó với cách tiếp cận NSError và BOOL được đề xuất trong issue, sau đó xác minh rằng hành vi ở cấp cao nhất của thư viện báo cáo lỗi mà không cung cấp ngoại lệ cho các thư viện máy khách.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
Is your feature request related to a problem you're having? Please describe.
GoogleSignIn/Sources/GIDSignIn.m is the only place that exposes exceptions. This is done in contrast to the Google style guide which in turn is aligned with Apple's Exception Programming Topics.
Describe the solution you'd like
Instead, error objects should be used: you should take care to catch such exceptions at the top level of the library and translate them into an appropriate return code or state. Clang for instance pass a **NSError and return a BOOL.
Describe alternatives you've considered
N/A
Additional context
I am receiving crashes for Google Sign-In iOS plugin for Flutter, and it seems to be the case for react-native as well. Using exceptions is less informative for the client library, and is bound to become 'unhandled'.
- Ngôn ngữ chính
- Objective-C
- Star
- 751
- Fork
- 282
- Merge trung bình
- 2 ngày 15 giờ
- Pull request đã merge (30 ngày)
- 9
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.
Issue khác của google/GoogleSignIn-iOS
-
enhancement triage
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
google/GoogleSignIn-iOS#621 ·
-
enhancement triage
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 76/100
google/GoogleSignIn-iOS#620 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
google/GoogleSignIn-iOS#440 · 4 bình luận · 2 reaction ·
-
enhancement triage
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 48/100
google/GoogleSignIn-iOS#611 ·
-
Remove Cocoapods Support Đang mở
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 45/100
google/GoogleSignIn-iOS#607 ·
Tất cả issue của google/GoogleSignIn-iOS
Issue tương tự
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
-
macOS下查找brew::ncurse失败 Đang mởbug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 84/100
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
noah-nuebling/mac-mouse-fix#2054 ·
-
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 92/100
obsproject/obs-studio#13918 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 88/100
HigherOrderCO/Bend#842 · 2 bình luận ·