奇怪的函数行为c + +，不返回相同的答案

Question

我创建一个队列类在c + +中，并遇到麻烦让前台功能工作。它应该打印队列中第一个节点的值。我queue.cpp类是这里

#include "queue.h" 
#include <stdlib.h> 
#include <iostream> 
using namespace std; 

queue::queue() 
{ 
    front_p = NULL; 
    back_p = NULL; 
    current_size = 0; 
} 

void queue::enqueue(int item) 
{ 
    node newnode = node(item, NULL); 
    if (front_p == NULL) //queue is empty 
    { 
     front_p = &newnode; 
     back_p = &newnode; 
    } 
    else 
    { 
     back_p->next = &newnode; 
     back_p = &newnode; 
    } 
    current_size ++; 
} 

int queue::dequeue() 
{ 
    //if there is only one node 
    if (front_p == back_p) 
    { 
     front_p = NULL; 
     back_p = NULL; 
    } 
    //if there are two or more 
    else 
     front_p = front_p->next; 
    current_size --; 
} 

int queue::front() 
{ 
    if (front_p != NULL) 
     return (*front_p).data; 
} 

bool queue::empty() 
{ 
    if (front_p == NULL && back_p == NULL) 
     return true; 
    else 
     return false; 
} 

int queue::size() 
{ 
    return current_size; 
} 

我的头文件（queue.h）是这里

class queue 
{ 
    public: 
    queue(); // constructor - constructs a new empty queue. 
    void enqueue(int item); // enqueues item. 
    int dequeue(); // dequeues the front item. 
    int front(); // returns the front item without dequeuing it. 
    bool empty(); // true iff the queue contains no items. 
    int size(); // the current number of items in the queue. 
    int remove(int item); // removes all occurrances of item 
     // from the queue, returning the number removed. 

    private: 
    class node // node type for the linked list 
    { 
     public: 
      node(int new_data, node * next_node){ 
       data = new_data ; 
       next = next_node ; 
      } 
      int data ; 
      node * next ; 
    }; 

    node* front_p ; 
    node* back_p ; 
    int current_size ; // current number of elements in the queue. 
}; 

测试程序（tester.cpp）

#include <iostream> 
#include "queue.h" 
#include <stdlib.h> 
using namespace std; 

int main(int argc, char * const argv[]) 
{ 
    queue q1; 
    q1.enqueue(5); 
    cout << "front: " << q1.front() << endl; 
    cout << "front: " << q1.front() << endl; 
    cout << "front: " << q1.front() << endl; 
    q1.enqueue(10); 
    cout << "front: " << q1.front() << endl; 
    cout << "front: " << q1.front() << endl; 
    cout << "size: " << q1.size() << endl; 
} 

的makefile

all: tester 

tester: queue.o tester.o 
    g++ tester.o queue.o -o tester 

tester.o: tester.cpp 
    g++ -c tester.cpp 

queue.o: queue.cpp queue.h 
    g++ -c queue.cpp 

clean: 
    rm -f tester *.o 

当我运行我的测试程序时，我得到这个：

front: 5 
front: 6299744 
front: 6299744 
front: 10 
front: 6299744 
size: 2 

正如你所看到的，在第一次入队之后，front返回它应该是的，队列前面的值。但之后它会返回一些奇怪的数字，我不知道它来自哪里！然后当我再次入球时，它再次打印出来。只有在调用前两次后才会开始打印混乱的值。任何人都可以帮助我理解发生了什么？

Answer 1

你的程序运行到不确定的行为，因为你有内存指针你没有自己：

void queue::enqueue(int item) 
{ 
    node newnode = node(item, NULL); 
    if (front_p == NULL) //queue is empty 
    { 
     front_p = &newnode; 
     back_p = &newnode; 
    } 
    else 
    { 
     back_p->next = &newnode; 
     back_p = &newnode; 
    } 
    current_size ++; 
} 

在函数结束时，newnode被破坏，但front_p和back_p仍指向内存位置。无论是动态分配：

node* newnode = new node(item, NULL); 

或使用std::shared_ptr<node>。