c++ - 有人可以纠正我的重载功能做错了什么吗？

Question

我对模板和重载的使用不是很熟悉，让我改了一段代码作为练习题。我纠正了所有我能设法识别的内容，但我不确定我忽略了什么。如果您觉得变量名称不好，我深表歉意，它们都带有原始损坏的代码。

部分错误信息是

CSL.cpp: In member function 'void CSL::showList() [with T = int]':

CSL.cpp:106: instantiated from here CSL.cpp:26: error: subscripted value is neither array nor pointer

代码本身:

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;
}

Answer 1

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

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

在您的 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 类应该包含作为构造函数输入提供的数组的拷贝。为了避免在动态分配、删除数组、三规则、五规则等方面犯错误，您应该使用 RAII 包装器，例如 std::vector。为此目的。