Question

我一直收到这个错误 -

错误C2819：类型'列表'没有一个重载的成员'运营商 - ＆gt;'

我无法弄清楚为什么？帮助

Main.cpp -

#include <iostream>
#include <string>
#include <cassert>
using namespace std;

#include "List.h"
#include "Node.h"

此处发生错误：

void PrintList ( List list ) {
 Node * temp = list.getFirst();
 Node * temp2 = list->getLast();
 while ( temp->getItemName() != temp2->getName() ) {
  cout << temp.getItemName() << endl;
 }
}

List.h -

#ifndef LIST_H
#define LIST_H

#include "Node.h"
class List
{
private:
 Node * First;
 Node * Last;
 int num_in_list;
public:
 List () { num_in_list = 0; First = NULL; Last = NULL; }
 int get_num_in_list() { return num_in_list; }
 Node * getFirst() { return First; }
 Node * getLast() { return Last; }
 void del_frnt ();
 void push_front (Node *);
 void push_back (Node *);
 void del_last ();
 void add (Node*);
 Node * pop_back ();
 Node * pop_front ();
 int search_item_list (string);
 Node * get (int);
};
#endif

List.cpp -

#include <iostream>
#include <string>
#include <cassert>
#include "Node.h"
#include "List.h"
using namespace std;

Node * List:: get ( int position_of_node ) {
 assert ( First != NULL);
 Node * temp = First;
 for (int i = 1; i < position_of_node; i++) 
  { temp = temp->getNext(); }
 return temp;
}

int List:: search_item_list (string item_searching_for ) {
 assert (First != NULL && num_in_list != 0);
 int counter = 1;
 Node * temp = First;
 while ( temp->getItemName() != item_searching_for || temp->getNext() == NULL )
  { temp = temp->getNext(); counter++; }
 return counter;
}

void List:: add (Node * node_to_be_added) {
 if (num_in_list == 0) { First = node_to_be_added; Last = node_to_be_added; }
 else if (num_in_list != 0 ) {
  Last->setNext(node_to_be_added);
  node_to_be_added->setPrevous(Last);
  Last = node_to_be_added;
 }
 num_in_list++;
}

Node * List :: pop_back ( ) {
 assert (Last != NULL);
 if ( num_in_list > 1) {
  Node * temp = Last;
  Last = Last->getPrevous();
  Last->setNext(NULL);
  temp->setNext(NULL);
  temp->setPrevous(NULL);
  return temp;
 }
 else if ( num_in_list == 1 ) {
  Node * temp = First;
  First = NULL;
  return temp;
 }
 else return NULL;
}

Node * List:: pop_front ( ) {
 assert ( First != NULL && num_in_list > 0);
 if ( num_in_list > 1 ) {
  Node * temp = First;
  First = First->getNext();
  First->setPrevous(NULL);
  temp->setNext(NULL);
  temp->setPrevous(NULL);
  return temp;
 }

 else if ( num_in_list == 1) {
  Node * temp = First;
  First = NULL;
  return temp;
 }
 else return NULL;
}

void List:: del_last ( ) {
 assert ( Last != NULL );
 if ( num_in_list > 1) {
  Node * temp_node = Last->getPrevous();
  Node * new_last = Last;
  temp_node->setNext(NULL);
  delete new_last;
  num_in_list--;
 }
 else if ( num_in_list == 1) {
  Node * temp = First;
  delete temp;
  num_in_list = 0;
  First = NULL;
 }
}

void List:: del_frnt ( ) {
 assert ( First != NULL);
 if ( num_in_list > 1) {
  Node * saveFirst = First;
  First->getNext()->setPrevous(NULL);
     First = First->getNext( );
     delete saveFirst;
     num_in_list--;
 }
 else if ( num_in_list == 1 ) {
  Node * saveFirst = First;
  delete saveFirst;
  num_in_list = 0;
  First = NULL;
 }
}

void List:: push_back (Node * new_node) {
 assert ( Last != NULL );
 Last->setNext(new_node);
 new_node->setPrevous(Last);
 Last = new_node;
 num_in_list++;
}

void List:: push_front (Node * new_node) {
 if ( First != NULL) {
  First->setPrevous(new_node);
  new_node->setPrevous(NULL);
  new_node->setNext(First);
  First = new_node;
  num_in_list++;
 }
 else if ( First == NULL ) {
  First = new_node;
  Last = new_node;
  num_in_list = 1;
 }
}

Node.h -

#ifndef NODE_H
#define NODE_H
#include <string>
using namespace std;

class Node 
{
private:
 string ItemName;
 string Quantity;
 Node * Next;
 Node * Prevous;
public:
 Node () { ItemName = " "; Quantity = " "; }
 void setItemName (string a) { ItemName = a; }
 string getItemName () { return ItemName; }
 void setQuantity (string a) { Quantity = a; }
 string getQuantity () { return Quantity; }
 void setNext (Node * a) { *Next = *a; }
 Node * getNext () { return Next; }
 void setPrevous (Node * a) { *Prevous = *a; }
 Node * getPrevous () { return Prevous; }
};
#endif

注意：我知道我正在做的只是一个列表，但对于拼贴类我必须这样做:) - 任何帮助/指针/如何做得更好将 GREAT !!!

Answer 1

不能同时做到：

Node * temp = list.getFirst();
Node * temp2 = list->getLast();

前者有效，因为您按值传递list，但要使后者工作，您需要重载operator->。只需使用：

Node * temp2 = list.getLast();

注意：在C ++中，访问聚合成员（数据成员或函数）的语法是：

T.member if T is an aggregate type

和

T->member if T is a pointer to an aggregate type

（好的，我已经简化了一下，但这应该可以帮助你开始。）

Answer 2

在PrintList函数中，您有一个类型为List的参数：

void PrintList ( List list )

当你尝试使用list.getFirst（）访问它时，一切正常。当您尝试执行list-＆gt; getLast（）时，编译器会生成一个错误，告诉您您的变量不是指针类型。在您的简短PrintList功能中，您可能想要替换所有 - ＆gt;随着。运营商。如果您想使用 - ＆gt;，您必须执行以下操作：

List * listptr = new List();
listptr->getFirst();
listptr->getLast();

否则，只需将第二行替换为：Node * temp2 = list.getLast（）;

您可以创建自定义 - ＆gt;操作员，但我认为这不是你想要做的。

另外，我没有仔细研究它，但你的while循环是否会终止？除非对getItemName的调用将改变节点的值，否则它看起来会永远持续下去。

Answer 3

列表不是指针节点* temp2 = list-＆gt; getLast（）;

Answer 4

将Node * temp2 = list->getLast();更改为Node * temp2 = list.getLast();会使其有效，但我认为这不是最佳解决方案。您将按列表将列表传递到PrintList()，这将调用复制构造函数。更好的方法是将const引用传递给列表。您仍然需要从->更改为.。

Answer 5

请注意

*节点* temp2 = list-＆gt; getLast（）; *

列表应该是一个对象，而不是一个指针，因此你不应该使用“ - ＆gt;”，而应该使用“。”

C ++错误C2819：类型'List'没有重载成员'operator - ＆gt;'

5 个答案: