pikachu0310
commented
3 months ago 機能面は良さそうです！ありがとうございます！
ただ、@motoki317 の https://q.trap.jp/messages/e55d6ed9-ad4e-42bc-bce8-f65844b3f34d を参考にすると、かなりコードを簡素にかつ分かりやすくできると気づいたため、リファクタリングをしました。
具体的には、
- sync.RWMutex から sync/atomic にすることで、軽量でより読みやすくしました。
- 初期化とカウンターの増加部分を関数で分離しました。
- Get() と inc() の最初に sync.Once を呼び出すことで、初期化が保証されるようにして、sync.Once の特徴を上手く活かす形にしました。
- count をグローバル変数にして、不要な変数を削除しました。
kaitoyama
Fixes #2157