1

我对模板和重载的使用不是很熟悉，我被要求纠正一个代码作为练习。我纠正了所有我能识别的内容，但我不确定我忽略了什么。如果变量名称对您来说听起来很糟糕，我深表歉意，它们都带有原始损坏的代码。

部分错误信息是

CSL.cpp：在成员函数“void CSL::showList() [with T = int]”中：

CSL.cpp:106：从这里实例化 CSL.cpp:26：错误：下标值既不是数组也不是指针

代码本身：

template<class T>
CSL<T>::CSL(T *d, int s) : data(*d), size(s)
{
}

template<class T>
void CSL<T>::showList()                 //Function with problem.
{
  cout<<"Comma separated list:"<<endl;
  for(int x = 0; x < size; ++x)
  {
    cout << data[x];
    if(x != size + 1)
      cout << ": ";
  }
  cout << endl << endl;
}

int main()
{

  someCustomers[0].setCustomer("Zaps", 23.55);
  //...
  someCustomers[5].setCustomer("Curtin",56999.19);

  CSL_Size = sizeof(someInts)/sizeof(someInts[0]);
  CSL<int> CSL_Integers(someInts, CSL_Size);
  //...    
  CSL_Size = sizeof(someCustomers)/sizeof(someCustomers[0]);
  CSL<Customer> CSL_Customers(someCustomers, CSL_Size);

  CSL_Integers.showList();   //Problem starts here
  CSL_Doubles.showList();
  CSL_Videos.showList();
  CSL_Customers.showList();

  return 0;
}
4

1 回答 1

1

您的CLS类模板接受类型参数T并定义data该类型的数据成员。

因此，当您提供int的类型参数时T，内部数据成员的类型data是int。

在您的showList()函数中，您尝试将下标运算符应用于该int，将另一个整数值作为输入传递给operator []：

cout << data[x];

但这是非法的，因为不涉及指针算术（data不是数组，也不是指针，也不是x）。这基本上相当于按照以下方式做一些事情：

42[1729]

这显然是无稽之谈。您可能打算拥有T*的类型data，并将构造函数重写为：

template<class T>
CSL<T>::CSL(T *d, int s) : data(d), size(s)
//                         ^^^^^^^
//                         Just pass d instead of *d
{
}

但是请注意，将封装容器的所有权留给客户是一个尴尬的设计决策。这样，您将必须确保该CSL对象不会超过数组（否则，data将是一个悬空指针）。

因此，您的CLS类应包含作为构造函数输入提供的数组的副本。为了避免在动态分配、删除数组、三规则、五规则等方面出错，您应该std::vector为此目的使用 RAII 包装器。

于 2013-05-17T11:17:44.477 回答