Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

关于ring的一些问题 #20

Closed
x-wesley opened this issue Nov 27, 2020 · 5 comments
Closed

关于ring的一些问题 #20

x-wesley opened this issue Nov 27, 2020 · 5 comments

Comments

@x-wesley
Copy link

1、我看完代码感觉ring的作用只是计算平均值。那么为啥没有用队列,添加元素的时候从队头删除一个,队尾添加一个。

2、如果没有用队列的原因是队列没有像ring那样严格的数量限制容易出现问题,那么为啥没有封装一下官方库提供的ring,而是自己写了一个。自己写的ring里面idx的最大值等于len(ring.data),又有ring.data[ring.idx]的操作,感觉这个有点危险。

@ioworker0
Copy link
Contributor

首先你不懂大佬造轮子的乐趣(开玩笑)

ring.idx是由ring本身维护的,使用上也非常简单。上层使用不存在并发调用,你说的危险性也不存在。

至于使用哪个数据结构或哪个轮子,要看当前的匹配度。(纯属个人意淫)

@x-wesley
Copy link
Author

首先你不懂大佬造轮子的乐趣(开玩笑)

ring.idx是由ring本身维护的,使用上也非常简单。上层使用不存在并发调用,你说的危险性也不存在。

至于使用哪个数据结构或哪个轮子,要看当前的匹配度。(纯属个人意淫)

哈哈哈
总觉得slice[len(slice)]这样的操作有点瘆得慌,像是个不定时炸弹一样。

@cch123
Copy link
Collaborator

cch123 commented Nov 27, 2020

队列也可以呀,就是 Go 标准库里那几个容器 api 实在不好看
如果不需要展示的话,这里甚至不需要有一个容器 :)

@ioworker0
Copy link
Contributor

队列也可以呀,就是 Go 标准库里那几个容器 api 实在不好看
如果不需要展示的话,这里甚至不需要有一个容器 :)

Big old, big old. Haaaaaa~

@x-wesley
Copy link
Author

队列也可以呀,就是 Go 标准库里那几个容器 api 实在不好看

如果不需要展示的话,这里甚至不需要有一个容器 :)

Big old

@x-wesley x-wesley closed this as completed Dec 8, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants