从Java学习C++，尝试制作链表

Question

我刚开始学习C++（来自Java）并且在做任何事情时都遇到了一些严重的问题：P 目前，我正在尝试制作一个链接列表，但一定是在做一些愚蠢的事情，因为我不断得到“ void 值没有被忽略，因为它应该是”编译错误（我已经将它标记在下面抛出它的位置）。如果有人可以帮助我解决我做错的事情，我将非常感激:)

另外，我不习惯在一般情况下选择通过引用、地址或值传递以及内存管理（目前我的所有节点和数据都声明在堆上）。 如果有人对我有任何一般性建议，我也不会抱怨：P

来自 LinkedListNode.cpp

的关键代码

LinkedListNode::LinkedListNode()
{
    //set next and prev to null
    pData=0; //data needs to be a pointer so we can set it to null for
             //for the tail and head.
    pNext=0;
    pPrev=0;
}

/*
 * Sets the 'next' pointer to the memory address of the inputed reference.
 */
void LinkedListNode::SetNext(LinkedListNode& _next)
{
    pNext=&_next;
}

/*
 * Sets the 'prev' pointer to the memory address of the inputed reference.
 */
void LinkedListNode::SetPrev(LinkedListNode& _prev)
{
    pPrev=&_prev;
}
//rest of class

来自 LinkedList.cpp 的关键代码

#include "LinkedList.h"

LinkedList::LinkedList()
{
    // Set head and tail of linked list.
    pHead = new LinkedListNode();
    pTail = new LinkedListNode();

     /*
      * THIS IS WHERE THE ERRORS ARE.
      */
    *pHead->SetNext(*pTail);
    *pTail->SetPrev(*pHead);
}
//rest of class

Answer 1

另外，我不习惯在一般情况下选择通过引用、地址或值传递以及内存管理（目前我的所有节点和数据都声明在堆上）。如果有人对我有任何一般性建议，我也不会抱怨：P

前 Java 程序员总是这样做。它是颠倒的。 您实际上应该从不堆分配数据。对象应该在堆栈上声明，如果他们需要堆分配的内存，他们应该在内部处理它，在它们的构造函数中分配它并在它们的析构函数中释放它。

这会导致代码更干净、更安全。

类成员也应该是值，而不是指针/引用，除非您特别需要在不同对象之间共享该成员。如果该类独占其成员，只需将其设为非指针值类型。这样它就在类本身内部进行分配，您不需要跟踪新/删除调用。

最简单的经验法则是不使用指针，除非你有。您需要将对象分配到别处吗？为什么不能here分配，按值访问？即使对象必须从一个函数返回，或者作为参数传递给另一个函数，复制通常会解决这个问题。只需定义适当的复制构造函数和赋值运算符，并在必要时复制对象。

Answer 2

中领先的*

*pHead->SetNext(*pTail);
*pTail->SetPrev(*pHead);

不需要。

pHead 是一个指向节点的指针，您可以在其上调用 SetNext 方法作为 pHead->SetNext(..) 通过引用传递 object。

-> 的 precedence 高于 *

因此，您实际上是在尝试取消引用函数 SetNext 的返回值，该函数不返回任何内容，从而导致此错误。