if块中的curCapacity应该只能为0吧，curCapacity精度是否需要提升？
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
- 35/100
Hướng nghiên cứu
Start with version4/src/main/java/part1/Server/ratelimit/impl/TokenBucketRateLimitImpl.java at line 38 and trace how curCapacity and timestamps are updated when the if block is entered. Determine whether integer truncation permanently loses fractional tokens and whether the proposed readability change preserves behavior. Done means resolving these questions and recording or implementing the agreed correction.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
https://github.com/youngyangyang04/RPC-Java/blob/ab0870519c49f198e0d603315d749e3de34c1537/version4/src/main/java/part1/Server/ratelimit/impl/TokenBucketRateLimitImpl.java#L38
(1) 如果进入if块时curCapacity只能为0，代码是否可以优化成这样
可读性应该会更好。
(2) curCapacity为int类型，若当前时间戳与上次时间戳差值除以RATE得到一个高精度数值，没有进行int强制转换前，得到一种极端情况，令牌为1.98个，此处强行转换int，导致丢失0.98个令牌，且更新当前时间戳，0.98个令牌是否会永久丢失，若1.99时又过来一个请求，应该会被丢弃，拒绝该请求，一定程度上会导致流量较期待降低，如果这个问题确实存在，应该流量是会普遍降低，因为是向左取整（不管是1.001，还是1.999，都取得1），如果短期快速更新，认为可以采用四舍五入的方式，长期来看应该还是提升精度更佳。
- Ngôn ngữ chính
- Java
- Star
- 694
- Fork
- 166
- Chỉ số merge pull request
- Không có pull request nào được merge trong 30 ngày
Chuẩn bị môi trường
Dự án này không cung cấp dev container, Dockerfile hay hướng dẫn đóng góp, nên bạn cần tự thiết lập môi trường: hãy bắt đầu từ README và xem hướng dẫn đóng góp lần đầu của chúng tôi để biết các bước chung.
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 youngyangyang04/RPC-Java
-
version5关于令牌Đang mở
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 45/100
youngyangyang04/RPC-Java#55 ·
-
version5 启动报错Đang mở
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 30/100
youngyangyang04/RPC-Java#53 · 1 bình luận ·
-
Version6限流器问题Đang mở
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 25/100
youngyangyang04/RPC-Java#51 ·
-
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 28/100
youngyangyang04/RPC-Java#50 ·
-
如何压测Đang mở
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 15/100
youngyangyang04/RPC-Java#49 · 1 bình luận ·
Tất cả issue của youngyangyang04/RPC-Java
Issue tương tự
-
code-maintenance
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 77/100
Maintainer thường phản hồi trong vòng 1 ngày
-
bug priority: medium Review effort [1-5]: 3 status: needs-triage type: bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 82/100
carlos-emr/carlos#5258 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
bug: author create/update with an unknown avatarId returns 404 instead of 400 UNKNOWN_REFERENCECó thể đã có người làm @Vinu2111 đã nhận hôm nay. Đang mởbug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 74/100
atomityhq/blog-cms#59 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
PlatformSpecific.hasGreenThreads incorrectly returns false for JDK updates and non-GA versionsĐang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100
Maintainer thường phản hồi trong vòng 1 ngày
-
dev/release: verify-release-candidate.sh runs 0 Java tests with Debian's MavenCó thể đã có người làm @kou đã nhận hôm nay. Đang mởType: bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
apache/arrow-adbc#4947 ·
Maintainer thường phản hồi trong vòng 1 ngày