使用 std::unique_ptr 和 std::queue

Question

也许这是我的鼻窦问题，而且我今天才开始学习智能指针，我正在尝试执行以下操作：

• 推入队列
• 获取前面的元素
• 弹出元素（我认为一旦地址超出范围它会自动 deque）

这是错误

main.cpp:50:25: error: cannot convert ‘std::remove_reference&>::type’ {aka ‘std::unique_ptr’} to ‘std::unique_ptr*’ in assignment
   50 |   inputFrame = std::move(PacketQueue.front());
      |                ~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~
      |                         |
      |                         std::remove_reference<std::unique_ptr<MyObject::Packet>&>::type {aka std::unique_ptr<MyObject::Packet>}

这里是代码

#include <iostream>
#include <memory>
#include <queue>

using namespace std;

class MyObject
{
    public:
    struct Packet
    {
        uint8_t message;
        uint8_t index;
    };
    
    void pushToQueue(void);
    void FrontOfQueue(std::unique_ptr<Packet> *inputFrame);
    
    private:
    
    std::queue<std::unique_ptr<Packet>> PacketQueue;
};

void MyObject::pushToQueue(void)
{
    Packet frame;
    
    static int counter = 1;
    frame.message = counter;
    frame.index =counter;
    
    counter++;
    
    std::unique_ptr<Packet> passthru_ptr = std::make_unique<Packet>(std::move(frame));
    PacketQueue.push(std::move(passthru_ptr));
    
    cout<<"Pushed to queue\n" ;
}

void MyObject::FrontOfQueue(std::unique_ptr<Packet> *inputFrame)
{
  inputFrame = std::move(PacketQueue.front());
}


int main()
{
    cout<<"Hello World\n";
    
    MyObject object;
    
    object.pushToQueue();
    object.pushToQueue();
    
    {
      // Scope 
    std::unique_ptr<MyObject::Packet> *frame;
    
    object.FrontOfQueue(frame); 
    
    cout<< frame << endl;
    
    }
    
    
    {
      // Scope 
    std::unique_ptr<MyObject::Packet> *frame2;
    
    object.FrontOfQueue(frame2); 
    
    cout<< frame2 << endl;
    
    }
    
    

  
    
    return 0;
}

Link to the code (Online Compiler)

Answer 1

如果我的目标正确，你肯定想要

std::unique_ptr<MyObject::Packet> MyObject::FrontOfQueue()
{
  auto rv = std::move(PacketQueue.front());
  PacketQueue.pop();
  return rv;
}
// ...
std::unique_ptr<MyObject::Packet> frame = object.FrontOfQueue(); 

注意，没有使用原始指针。

我认为一旦地址超出范围，它会自动双端队列。

这个假设是错误的。在调用 .pop() 之前，什么都不会出队。

Answer 2

这是我的示例，其中包含一些额外的日志记录以显示正在发生的事情。 还包括对返回 const 引用的介绍。 现场演示：https://onlinegdb.com/P2nFkdMy0

#include <iostream>
#include <memory>
#include <queue>
#include <string>

//-----------------------------------------------------------------------------
// do NOT use : using namespace std;

//-----------------------------------------------------------------------------

struct Packet
{
    // moved to uint32_t for std::cout reasons.
    // uint8_t is displayed as(special) characters
    std::uint32_t index;
    std::uint32_t message; 
    

    Packet() :
        index{ next_index() },
        message{ index }
    {
        std::cout << "created packet : " << index << "\n";
    }
    
    ~Packet()
    {
        std::cout << "destroyed packet : " << index << "\n";
    }

    // small helper to not have to declare the static variable seperatly
    static std::uint8_t next_index()
    {
        static int counter;
        return counter++;
    }
};

//-----------------------------------------------------------------------------

class MyObject
{
public:
    void push_packet();
    std::unique_ptr<Packet> pop_packet();

    // this function returns a const reference (observation only)
    // of the packet at the front of the queue
    // while leaving the unique pointer on the queue (no moves needed
    // packet will still be owned by the queue)
    const Packet& front();

private:
    std::queue<std::unique_ptr<Packet>> m_queue;
};

void MyObject::push_packet()
{
    std::cout << "push_packet\n";
    // push a packet 
    m_queue.push(std::make_unique<Packet>());
    std::cout << "push_packet done...\n";
}

std::unique_ptr<Packet> MyObject::pop_packet()
{
    std::unique_ptr<Packet> packet = std::move(m_queue.front());
    m_queue.pop();
    return packet;
}

const Packet& MyObject::front()
{
    return *m_queue.front();
}

//-----------------------------------------------------------------------------

int main()
{
    const std::size_t n_packets = 3ul;

    MyObject object;

    for (std::size_t n = 0; n < n_packets; ++n)
    {
        std::cout << "pushing packet\n";
        object.push_packet();
    }

    for (std::size_t n = 0; n < n_packets; ++n)
    {
        std::cout << "packet at front : ";
        std::cout << object.front().index << "\n";

        std::cout << "popping front\n";
        auto packet_ptr = object.pop_packet();
        std::cout << "popped packet : " << packet_ptr->index << "\n";
    }

    return 0;
}