WebServer icon indicating copy to clipboard operation
WebServer copied to clipboard

定时器问题

Open JsonJsons opened this issue 4 years ago • 12 comments

楼主你好,如果客户端主动关闭,定时器队列里对应的定时器是在哪里剔除掉的?我只找到在将关闭的连接通过runInLoop函数加入到队列,从而自动执行httpClose函数,但是在httpClose函数中只是剔除掉了与之对应的channel对象,没有找到在哪里剔除掉对应的定时器队列中的对应的定时器。 还有一个问题就是,当有新的请求来的时候,在执行handleConn()函数时,是怎么更新的对应的定时器?我顺着程序找,怎么发现是往定时器队列中添加的新的定时器呢?不是应该更新对应的定时器队列中对应的那个定时器吗? 楼主辛苦了!

JsonJsons avatar Jul 19 '19 08:07 JsonJsons

@JsonJsons 每次事件来的时候(不管什么事件),都会分离定时器和事件,其实就是把定时器关了,然后你说的是对滴,我没有更新,是直接添加了新的,你可以试着看怎么优化一下

linyacool avatar Jul 20 '19 03:07 linyacool

@JsonJsons 这个是不好的地方,事件频繁的时候,频繁创建和删除定时器

linyacool avatar Jul 20 '19 03:07 linyacool

@JsonJsons 微信:linyacool

linyacool avatar Jul 20 '19 03:07 linyacool

@JsonJsons 微信:linyacool

加你了,谢谢

JsonJsons avatar Jul 20 '19 03:07 JsonJsons

@linyacool 你好,我能不能加你的微信,谢谢!

tiankaijin avatar Jul 31 '19 11:07 tiankaijin

@linyacool 你好,我能不能加你的微信,谢谢!

可以的哇

linyacool avatar Aug 04 '19 03:08 linyacool

你好,我想问一下定时器是有事件来的时候,之后才进行一个判断小根堆是吗? 如果长时间没有事件到来,是不会主动断开链接的,是这样的吗

yinzhengxin avatar Sep 21 '19 09:09 yinzhengxin

我看的版本是前第0.5版。timer_manager.handle_expired_event();函数是在epoll_wait函数最下面执行。不知道理解的对吗?谢谢!

yinzhengxin avatar Sep 21 '19 09:09 yinzhengxin

@yinzhengxin 是这样的,不是精准的时间控制

linyacool avatar Sep 28 '19 15:09 linyacool

这个定时器实现逻辑是什么样子的,我知道超时他会删除优先队列里面的对象,可是他如何影响具体的连接呢?

LLBRYU avatar Feb 18 '20 08:02 LLBRYU

@LLBRYU 一个连接超时了,不会马上就他这个连接剔除,而是等下一次的任何一个连接到来的时候,顺便去检查定时器,如果发现超时请求,一并剔除

linyacool avatar Feb 21 '20 03:02 linyacool

楼主你好,如果客户端主动关闭,定时器队列里对应的定时器是在哪里剔除掉的?我只找到在将关闭的连接通过runInLoop函数加入到队列,从而自动执行httpClose函数,但是在httpClose函数中只是剔除掉了与之对应的channel对象,没有找到在哪里剔除掉对应的定时器队列中的对应的定时器。 还有一个问题就是,当有新的请求来的时候,在执行handleConn()函数时,是怎么更新的对应的定时器?我顺着程序找,怎么发现是往定时器队列中添加的新的定时器呢?不是应该更新对应的定时器队列中对应的那个定时器吗? 楼主辛苦了!

针对第二个问题,其实没有必要对堆中的定时器进行更新,因为对其更新的话需要找到定时器->修改->调整堆结构,这还是一个比较耗时的过程,因此这个文件描述符被封装在了HttpData类中,而整个server是通过shared_ptrHttpData进行使用的,因此对于加入定时器的连接,即使第二次刷新,只要添加第二次的shared_ptr<HttpData>定时器,第一次的定时器直接pop就好了,不会析构这个HttpData对象,所以就不会关闭文件描述符,只是在这个对象内部减少了一次引用计数而已。

Hanwn avatar Apr 19 '21 08:04 Hanwn