Hacktoberfest 2026: le issue che i maintainer hanno segnato per ottobre, aperte e adatte ai principianti. Sfoglia le issue Hacktoberfest

if块中的curCapacity应该只能为0吧,curCapacity精度是否需要提升?

Aperta
#21 1 commento 0 reazioni 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
4/5
Tempo stimato
3-5 giorni
Idoneità per principianti
35/100
Tipo di issue
Bug
Chiarezza
Da chiarire
Stato di attività
Ferma
Stack tecnologico
java
Ambito
backend

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

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di youngyangyang04/RPC-Java

Tutte le issue di youngyangyang04/RPC-Java

Issue simili

Altre issue su Java

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.