1

我有一个函数，其中回调接收一个event持有 unique_ptr 实例的data. char*我可以通过它来检索event.data.get()应该给我指针而不是所有权，从而允许对象自由删除它。现在我有另一个函数，它接受一个 unique_ptr 实例，otherdata当然管理所有权，所以该函数可以自由地释放它。所以我试图data从回调中获取并将其传递给安全的函数otherdata。

void cbdata(const DataEvent& event){
                char* datatobewritten = event.data.get();
                stream.write(std::move(datatobewritten),event.length));
            }

上面的示例似乎确实有效，但我不确定这是否是正确的方法。事件是否放弃所有权？如果是这样，放弃所有权是否安全？

为什么这会导致编译器错误：Call to implicitly-deleted copy constructor of 'std::unique_ptr<char []>'？

void cbdata(const DataEvent& event){
                stream.write(event.data,event.length);
            }

如果 event.data 给我一个 unique_ptr 并且 stream.write 需要一个 unique_ptr 那么上面的代码不应该正常工作吗？或这个

stream.write(std::move(event.data), event.length);

抱歉，我的 C++ 知识非常有限，所以 unique_ptr 和 move 语义非常混乱。

4

2 回答 2

2

stream.write您想要拥有正在打印的事件片段的所有权，这有点不寻常。似乎它拥有事件的一部分，而调用者仍然是其余部分的所有者。在那次调用之后，一个event对象仍然是不完整的，因为它event.data被放弃了。

此外，您cbdata通过const&. 这是一种“借用且不改变”的语义。因此，它不应授予其所有权event或其任何部分。

一些选项：

  1. 将整个事件的所有权传递给了stream，而不仅仅是data。不过，您cbdata需要先获得所有权。

  2. 不要给stream. 通常流仅用于向用户呈现数据，而不是以任何方式操纵它们。

  3. 更改您的活动，以便它可以data永久发布。你需要这样的东西：

    std::unique_ptr<DataType> Event::releaseData() {
    return std::move(this->data);
    //after the call, this->data is 'nullptr'!
}

    当然，在其余的课程中，必须支持 nullptrEvent的情况。data

    请注意，这会更改event对象，而不是const函数。您cbdata将不适用于const&事件对象。

  4. 您还可以将数据复制到流中。如果数据很大，可能效率低下，但如果DataType是可复制的，则完全可以。你可以这样做：

    void cbdata(const DataEvent& event){
    stream.write(std::make_unique<DataType>(*event.data),event.length);
}

    与任何其他副本一样，您必须注意，当副本被销毁时，实际上只删除了复制的数据。不应删除原件和副本之间共享的任何内容。

于 2019-09-07T11:55:13.117 回答
1

stream.write()可能需要一个，如果是这样，const char *您不需要std::move()将数据传递给它。std::move()当您需要更改 unique_ptr 的数据所有权时使用。但在这种情况下，数据是自己的event，我们只是暂时允许stream.write()访问它。 

void cbdata(const DataEvent& event){
    const char* datatobewritten = event.data.get();
    stream.write(datatobewritten, event.length);
}

如果stream.write()需要 a std::unique_ptr，那么您需要std::move()从event.data该函数获取数据。在此移动之后，event.data处于“已移出”状态，无法再访问数据。

void cbdata(DataEvent& event){
    stream.write(std::move(event.data), event.length);
    // now 'event.data' owns nothing and holds a null pointer
}
于 2019-09-07T11:47:57.857 回答