if块中的curCapacity应该只能为0吧，curCapacity精度是否需要提升？
还没有人认领这个 Issue。
评估
调研方向
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.
由索引模型根据 Issue 内容生成。
描述
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），如果短期快速更新，认为可以采用四舍五入的方式，长期来看应该还是提升精度更佳。
- 主要语言
- Java
- 星标
- 694
- 派生
- 166
- PR 合并指标
- 30 天内没有已合并 PR
环境准备
这个项目没有提供开发容器、Dockerfile 或贡献指南，环境需要你自己搭建：先看它的 README，通用步骤见我们的新手贡献指南。
从这里开始
- 先读完整个 Issue，再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库，在一个分支上完成修改。
- 提交 Pull Request，并在描述里引用这个 Issue 编号。
youngyangyang04/RPC-Java 的其他 Issue
-
version5关于令牌未关闭
难度 4/5 3-5 天 新手友好度 45/100
youngyangyang04/RPC-Java#55 ·
-
version5 启动报错未关闭
难度 3/5 1-2 天 新手友好度 30/100
youngyangyang04/RPC-Java#53 · 1 条评论 ·
-
Version6限流器问题未关闭
难度 5/5 一周以上 新手友好度 25/100
youngyangyang04/RPC-Java#51 ·
-
难度 4/5 3-5 天 新手友好度 28/100
youngyangyang04/RPC-Java#50 ·
-
如何压测未关闭
难度 5/5 一周以上 新手友好度 15/100
youngyangyang04/RPC-Java#49 · 1 条评论 ·
查看 youngyangyang04/RPC-Java 的全部 Issue
相似的 Issue
-
难度 1/5 1 小时以内 新手友好度 85/100
-
[Bug]: REST API URL starting with "http" but no scheme (e.g. httpbin.org/get) is not given an http:// prefix可能已有人在做 @ItsDeadlyProgrammer 今天认领。 未关闭
难度 2/5 1-3 小时 新手友好度 68/100
appsmithorg/appsmith#42471 · 1 条评论 ·
维护者通常 2 天内回复
-
code-maintenance
难度 2/5 1-3 小时 新手友好度 77/100
维护者通常 1 天内回复
-
难度 1/5 1 小时以内 新手友好度 85/100
-
bug priority: medium Review effort [1-5]: 3 status: needs-triage type: bug
难度 2/5 1-3 小时 新手友好度 82/100
carlos-emr/carlos#5258 ·
维护者通常 1 天内回复