对指针数组使用begin（），end（）

Question

我有下面的代码

PositionSummary** psArray_= new PositionSummary*[instSize]
for (int k = 0; k < instSize_; k++)
{
    PositionSummary* ps = new PositionSummary();
    psArray_[k] = ps;

}

当我尝试按以下方式进行复印时，它不起作用

std::copy(begin(psArray_),end(psArray_),std::back_inserter(psContainer_));

编译器报告“没有匹配的函数来调用'begin（PositionSummary *＆）”，该如何解决？

Answer 1

psArray只是一个指针。 std::begin和std::end可能无法知道指针指向的数组的长度。 它们适用于纯数组，不适用于指向数组的指针。

您可以改为：

std::copy(psArray_, psArray_ + instSize, std::back_inserter(psContainer_));

假设psContainer_是一个容器，其中包含指向PositionSummary指针。

根据您的应用程序的详细信息，最好使用智能指针或值的std::vector ：

std::vector<some_smart_pointer<PositionSummary>> psArray_;

要么

std::vector<PositionSummary> psArray_;

如果您不需要引用语义，则第二种选择可能更好。

Answer 2

这是行不通的，因为psArray_根本不是数组，而是一个指针。 您可能知道它指向一个特定大小的数组，但是编译器没有（一般）了解这一点的方法。 如果要使其成为数组，则可以正常工作。 尝试像这样声明psArray_：

PositionSummary* psArray_ [instSize];

这将起作用，但是大概不适合您，因为我猜instSize不是编译时间常数。 您可以通过使用向量而不是自己进行内存分配来解决此问题：

std::vector<PositionSummary*> psArray_;
for (int k = 0; k < instSize; k++) 
    psArray_.push_back (new PositionSummary());

std::copy(psArray_.begin()),psArray_.end(),std::back_inserter(psContainer_));

为了进一步改善，您可以考虑在向量中使用smartpointer（因此不必担心以后释放）。 尽管我不知道代码的更大上下文，但我仍然想知道为什么完全需要复制-为什么不直接在最终应该保留它的数据结构中直接构造向量？