ZBLOG

#include #include #include "terra_memory_pool.h" template class LockFreeQueue { public: LockFreeQueue() : head_(new Node), tail_(head_.load()), memory_pool_(sizeof(Node) * 100) {} ~LockFreeQueue() { ...

这个队列的实现是基本正确的,但是可以有一些优化:

  1. 内存池的大小应该根据实际情况进行调整,而不是固定为100个节点大小。可以考虑在构造函数中传入内存池大小参数,或者在运行时动态调整。

  2. push() 中使用了std::make_pair()创建节点值,可以直接使用{}进行初始化,避免不必要的对象拷贝。即 new (new_node) Node({value, nullptr});

  3. try_pop() 中可以在比较并交换head_指针前,先判断当前队列是否为空,如果为空直接返回false。

  4. empty() 中可以直接用size_ == 0来判断队列是否为空,无需使用load()和memory_order_acquire。

  5. 析构函数中可以优化为使用while循环删除节点时,先将当前节点的下一个节点保存起来,再删除当前节点,这样可以减少一次访问操作。即:

    while (Node *const oldhead = head) {

    Node *next_node = old_head->value_.second;
    delete old_head;
    head_ = next_node;
    

    }

本站部分文章来源于网络,版权归原作者所有,如有侵权请联系站长删除。
转载请注明出处:https://golang.0voice.com/?id=1544

分享:
扫描分享到社交APP
上一篇
下一篇
发表列表
游客 游客
此处应有掌声~
评论列表

还没有评论,快来说点什么吧~

联系我们

在线咨询: 点击这里给我发消息

微信号:3007537140

上班时间: 10:30-22:30

关注我们
x

注册

已经有帐号?