hv第一坑:定时器

错误代码

重试策略:一次延迟1s,最长30s直至事件成功。

int try_count = 0;

//do something
if(not success)
	m_loop->setTimerInLoop((try_count >= 30 ? 30: ++try_count) *1000  , cb, INFINITE, 0x100);
表现现象
  1. cpu 爆了
  2. 内存爆了
总结原因

hv内部代码bug,虽然timerid相同但依旧会再次创建htimer_t,上一次的htimer_t并未释放和删除。所以会累积越来越多的定时器,导致cpu和内存都爆了。

代码修复(设置之前killTimer)
    TimerID setTimer(int timeout_ms, TimerCallback cb, uint32_t repeat = INFINITE, TimerID timerID = INVALID_TIMER_ID) {
        assertInLoopThread();
        if (loop_ == NULL) return INVALID_TIMER_ID;
       
        htimer_t* htimer = htimer_add(loop_, onTimer, timeout_ms, repeat);
        assert(htimer != NULL);
        if (timerID == INVALID_TIMER_ID) {
            timerID = generateTimerID();
        }
        hevent_set_id(htimer, timerID);
        hevent_set_userdata(htimer, this);			
        //在设置timerid之前,删除当前定时器(即使不存在也没关系)
        killTimer(timerID);
         
        timers[timerID] = std::make_shared<Timer>(htimer, cb, repeat);
        return timerID;
    }
代码分析

在这里插入图片描述
在这里插入图片描述
此时将会导致连个问题:

  1. htimer_t对象内存泄露 内存 boom
  2. 定时器未释放,导致定时器积累月累越多,cpu boom
  • 3
    点赞
  • 0
    收藏
    觉得还不错? 一键收藏
  • 1
    评论

“相关推荐”对你有帮助么?

  • 非常没帮助
  • 没帮助
  • 一般
  • 有帮助
  • 非常有帮助
提交
评论 1
添加红包

请填写红包祝福语或标题

红包个数最小为10个

红包金额最低5元

当前余额3.43前往充值 >
需支付:10.00
成就一亿技术人!
领取后你会自动成为博主和红包主的粉丝 规则
hope_wisdom
发出的红包
实付
使用余额支付
点击重新获取
扫码支付
钱包余额 0

抵扣说明:

1.余额是钱包充值的虚拟货币,按照1:1的比例进行支付金额的抵扣。
2.余额无法直接购买下载,可以购买VIP、付费专栏及课程。

余额充值