Question

我正在尝试基于链表在C ++中实现Queue容器。我使用相同的结构来实现Stack，它运行良好。

但是现在我对方法“enqueue”有点麻烦。我无法理解究竟是什么问题，虽然我知道指针是我的弱点。

#include <iostream>

template <class N>
class node {
public:
  N data;
  node* next;
};

template <class Q>
class my_queue {
protected:
  node<Q>* m_head;
  unsigned int m_size;

public:
  my_queue() {
    m_head = NULL;
    m_size = 0;
  }

  void enqueue(Q value) {

    node<Q>* newel = new node<Q>; // creating the new element
    node<Q>* last = m_head; // find the last element in the queue

    while(last != NULL) {
      last = last->next;
    }

    newel->data = value;
    newel->next = last->next;
    last->next = newel;

    m_size++;
  }

  void print() {
    node<Q>* element = m_head; // element == each element in the list
    while(element != NULL) {
      std::cout << element->data << std::endl;
      element = element->next;
    }
  }

};

如果我用以下代码编译：

main() {
  my_queue<int> q;
  q.enqueue(1);
  q.enqueue(2);
  q.enqueue(3);
  q.enqueue(4);
  q.enqueue(5);
  q.print();

  return 0;
}

我没有错误，但是当我运行它时，我得到“分段错误”。

Answer 1

在函数

中循环之后

while(last != NULL) {
  last = last->next;
}

指针last将始终等于NULL。因此，由于这些语句，函数具有未定义的行为

newel->next = last->next;
last->next = newel;

该功能可以通过以下方式重写

void enqueue( const Q &value ) 
{
    node<Q> *newel = new node<Q> { value, nullptr };

    if ( m_head == nullptr )
    {
        m_head = newel;
    }
    else
    {
        node<Q> *last = m_head; // find the last element in the queue

        while ( last->next != nullptr ) last = last->next;

        last->next = newel;
    }

    m_size++;
}

为了提高队列效率，最好基于双面列表来实现它。

使用C ++中的链接列表实现队列实现

1 个答案: