if块中的curCapacity应该只能为0吧,curCapacity精度是否需要提升?
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Idoneità per principianti
- 35/100
Direzione di ricerca
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.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
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),如果短期快速更新,认为可以采用四舍五入的方式,长期来看应该还是提升精度更佳。
- Lingua principale
- Java
- Stelle
- 694
- Fork
- 165
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Preparare l'ambiente
Questo progetto non fornisce container di sviluppo, Dockerfile né guida per i contributori, quindi l'ambiente è a tuo carico: parti dal suo README e consulta la nostra guida al primo contributo per i passaggi generali.
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di youngyangyang04/RPC-Java
-
version5关于令牌Aperta
Difficoltà 4/5 3-5 giorni Idoneità per principianti 45/100
youngyangyang04/RPC-Java#55 ·
-
version5 启动报错Aperta
Difficoltà 3/5 1-2 giorni Idoneità per principianti 30/100
youngyangyang04/RPC-Java#53 · 1 commento ·
-
Version6限流器问题Aperta
Difficoltà 5/5 Più di una settimana Idoneità per principianti 25/100
youngyangyang04/RPC-Java#51 ·
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 28/100
youngyangyang04/RPC-Java#50 ·
-
如何压测Aperta
Difficoltà 5/5 Più di una settimana Idoneità per principianti 15/100
youngyangyang04/RPC-Java#49 · 1 commento ·
Tutte le issue di youngyangyang04/RPC-Java
Issue simili
-
agent:working follow-up
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
Sanne/incus-spawn#1683 · 2 commenti ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
checkstyle/checkstyle#22210 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 75/100
521xueweihan/HelloGitHub#4105 ·
-
devops good first issue security
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
asciidoctor/asciidoctorj#1339 ·