C++ 中的双链表

Question

我有一个任务要求我们实现一个双向链表类。出于某种原因，他们将节点 struct 定义如下：

struct node {
  node   *next;
  node   *prev;
  T      *o;
};

在我看来，如果结构成员“数据”不是指针，编写类会容易得多。不用说我无法改变它，所以我将不得不解决它。我尝试实现将元素添加到列表开头的方法，如下所示：

template <typename T>
void Dlist<T>::insertFront(T *o) {
    node *np = new node;
    T val = *o;

    np->o = &val;
    np->prev = NULL;
    np->next = first;

    if (!isEmpty()) {
        first->prev = np;
    } else {
        last = np;
    }
    first = np;
}

在使用 ddd 进行调试时，我意识到第一次插入数字时一切正常，但第二次时一切都搞砸了，因为一旦将 'val' 设置为新元素，它就会“覆盖”第一个元素因为使用了 val 的内存地址。我尝试做其他事情，而不是让 'val' 变量执行以下操作：

T *valp = new T;
T val;
valp = &val;
val = *o;

np->o = valp

这似乎也不起作用。我认为这是因为它几乎只是我上面所做的更复杂的形式，只是额外的内存泄漏:)

任何正确方向的想法/指针都会很棒。

Answer 1

您创建的T val 是一个自动变量。您的错误是将地址存储到该堆栈变量中。

正如您所怀疑的，您应该使用new 在堆上分配空间，但您的数据指针需要直接指向new 返回的地址。

你最近一次尝试的错误在这里：

valp = &val;

您正在将valp 更改为指向其他地方（val 的地址），而您可能试图复制 val 的数据，而不是其地址。

应将传递给您的函数的数据复制到valp 指向的新内存中。

Answer 2

您认为指针有效负载不方便。但也许该列表旨在用于在现有对象之上创建一个链接，例如：

struct A { int i; };
A as[10] = { 1,2,3,4,5,6,7,8,9,10 };

LinkedList primes_in_my_data;
insertFront( primes_in_my_data, &as[1] );
insertFront( primes_in_my_data, &as[2] );
insertFront( primes_in_my_data, &as[3] );
insertFront( primes_in_my_data, &as[5] );
insertFront( primes_in_my_data, &as[7] );

// now I have a linked list of primes, and caused no extra memory allocation
// (except for the list overhead)

Answer 3

我认为你不应该这样做：

T val = *o;

由于节点结构中的o 成员是一个指针，而insertFront 的参数也是一个指针，你的导师可能打算让你把你给的指针存储在列表中，不要复制对象并存储指向该对象的指针。只需将传递给insertFront 的o 指针存储为节点的o 成员就可以了。

Answer 4

您的代码不符合您对node 的定义——在node 中您定义了一个data 成员，但在您的代码中您使用的是o。

在任何情况下，您都必须进行两次动态分配来添加每个节点——一次分配给节点本身，另一次分配给它要指向的数据对象。当您分配该数据对象时，您不妨将new 的返回值直接分配给节点中的指针。然后，您需要将指针传递到您刚刚分配的数据对象中的数据对象复制到节点中的指针所指向的数据对象中。

Answer 5

 T val = *o;

    np->o = &val;

这部分是可疑的。提示是，一旦函数超出范围，在函数堆栈上分配的内存将不可用。