模板化的“strdup()”？

Question

template<typename T>
static T *anydup(const T *src, size_t len) {
    T *ptr = malloc(len * sizeof(T));
    memcpy(ptr, src, (len * sizeof(T)));
    return ptr;
}

这样合适吗？使用 int、long 等时，我可以期待任何错误吗？我对泛型编程非常陌生，正在努力学习更多。

Answer 1

不，这是不对的！当你在 C++ 代码中有一个malloc() 时，你应该变得非常怀疑：

• malloc() 分配内存，但没有正确创建对象。使用此类内存的唯一方法是使用新放置。
• memcpy() 不尊重 C++ 对象的复制语义。这只能与可复制的类一起使用。我会在其他地方很难找到错误（浅拷贝和其他导致 UB 的可怕事情）。

对于 char、int、double 等基本类型，它可以工作。但不适用于更复杂的类型。

---

备选方案 1：调整您的代码以正确创建和复制对象

template<typename T>
T *anydup (const T *src, size_t len) {
    T *ptr = new T[len];                        // requires that T has a default constructor
    copy (src, src+len, ptr);                   // requires that T is copyiable
    return ptr;
}

注意：如果用户忘记删除数组，则存在内存泄漏风险，如果用户不使用delete[]，则存在UB！为避免这种情况，您可以选择返回unique_ptr<T[]>。

---

备选方案 2：摆脱数组和指针以及内存噩梦：使用向量！

template<typename T>
vector<T> anydup (const vector<T> src) {
    vector<T> v(len);                        // requires that T has a default constructor
    copy (src.cbegin(), src.cend(), v);      // requires that T is copyable
    return v;
}

您可以考虑使用 Remy Lebeau 和 FDinoff 在 cmets 中建议的复制构造函数创建向量，无论是在函数中还是直接在使用代码中。

如果你在使用代码中直接使用copy()，你很快就会发现还有copy_if()、copy_backwards()和其他一些不错的<algorithms>可以根据情况使用。