memcpy或std ::复制我的特定应用程序

Question

我正在尝试为某些旧版C代码撰写C++包装器，以提高其类型的安全性，并使其不再是使用PITA。旧版代码与共享内存中的指针相互作用。豆荚结构是从这些指针中铸造出来的。由于它是共享的内存，因此保证这些结构是POD。

C代码用memcpy实例乱扔。我想知道在这种情况下，使用std::copy而不是memcpy是一个好主意，或者最好将其放置在这种情况下。

例如，原始代码（以伪代码表示）：

// we cast from a void pointer return from the shared memory segment
void *ptr = FunctionToReturnPointer();
descriptor_p = (descriptor_t*)new char[sizeof(descriptor_t)];
memcpy(descriptor_p, ptr, sizeof(descriptor_t));

其中descriptor_t是POD结构。在我的解决方案中，我可能会做这样的事情（在伪代码中表示）：

// provide a casting class that can do everything
template <typename T>
class CastClass {
    static T* Cast() {
        void *ptr = FunctionToReturnPointer();
        T *t = new T;
        std::copy(ptr, ptr + sizeof(T), t);
        return t;
    }
};
// And how I would use the function
descriptor_p = CastClass<descriptor_t>::Cast();

所以这是我的问题：

1. 这样做有益吗？还是这是傻瓜的差事？
2. 我是在代码中添加任何类型安全吗？
3. 我对std::copy算法的使用是否正确？

Answer 1

都不呢？只需使用正确键入的指针，让对象的默认复制分配工作：

descriptor_t* descriptor_p = new descriptor_t;
*descriptor_p = *(descriptor_t*)(FunctionToReturnPointer());

一行：

descriptor_t* descriptor_p = new descriptor_t(*(descriptor_t*)(FunctionToReturnPointer()));

模板版本：

template<typename T>
T* MakeNewCopy(void* p) { return new T(*static_cast<T*>(p)); }
auto p = MakeNewCopy<descriptor_t>(FunctionToReturnPointer());

这将比您现在看起来的清洁要多得多，但是如果您将错误的类型输入模板，则编译器将无济于事，因此"安全"是一个相对术语。每当您使用void*并施放它时，都会脱下手套。尝试尽可能地包含该行为，以便您的代码几乎只处理正确键入的指针。

Answer 2

您真的需要在堆上动态分配副本吗？

// provide a casting class that can do everything
template <typename T>
class CastClass {
    public:
    static T Cast() {
        T *ptr = (T*) FunctionToReturnPointer();
        return *ptr; // return a copy of the data
    }
};

descriptor_t descriptor_p = CastClass<descriptor_t>::Cast();

如果您确实需要指针，则至少使用unique_ptr来管理它：

// provide a casting class that can do everything
template <typename T>
class CastClass {
    public:
    static std::unique_ptr<T> Cast() {
        T *ptr = (T*) FunctionToReturnPointer();
        return new T(*ptr); // return a managed pointer to a copy of the data
    }
};

std::unique_ptr<descriptor_t> descriptor_p = CastClass<descriptor_t>::Cast();

Answer 3

std::copy(ptr, ptr + sizeof(T), t);

不对。

首先，我认为您不能在呼叫std::copy

中传递void*类型的对象

即使您做了类似的事情：

T* ptr = static_cast<T*>(FunctionToReturnPointer());
...
std::copy(ptr, ptr + sizeof(T), t);

有问题。

说T == int和sizeof(int)是4。

std::copy的使用将尝试复制：

*t to *ptr
*(t+1)至*(ptr+1)
*(t+2)至*(ptr+2)和
*(t+3)至*(ptr+3)

您的代码没有为这些对象分配内存，并且会导致不确定的行为。