c++ - C++-创建不带=动态字符串数组分配的Copy函数

尝试为动态分配的数组编写复制功能。

在我的头文件中，我有:

#include <memory>
#include <string>


using std::string;
using std::unique_ptr;
using std::make_unique;

class StringSet{
public:
    //create an empty set
    StringSet() = default;
    StringSet(int capacity);

    //copy a set
    StringSet(const StringSet&);

    StringSet& operator[](const int);

    //Insert a string to the set
    bool insert(string);

    //Remove a string from the set
    bool remove(string);

    //Test whether a string is in the set
    int find(string) const;

    //Get the size of the set
    int size() const;

    //get string at position i
    string get(int i) const;

    //Return the set union of the set and another StringSet
    StringSet setunion(const StringSet&) const;

    //Return the intersection of the set and another StringSet
    StringSet intersection(const StringSet&) const;

    //Return the set diffference of the set and another StringSet
    StringSet difference(const StringSet&) const;

    //prevent default copy assignment
    StringSet& operator=(const StringSet&) = delete;

    int NOT_FOUND = -1;
    static constexpr int def_capacity {4};

private:
    int arrSize {def_capacity};
    int currentSize {0};
    unique_ptr<string[]> arr {make_unique<string[]>(def_capacity)};

};

在我的实现文件中，我有:

#include "StringSet.h"
#include <iostream>
#include <utility>



StringSet::StringSet(int capacity)
: arrSize{capacity},
    arr{make_unique<string[]>(capacity)}
{
}

StringSet::StringSet(const StringSet& a)
{
    auto a2 = StringSet(currentSize);

    for (auto i=0; i < currentSize ; i++ )
        {
        a2[i] = a[i];
        }
}

编译器错误:

error: constructors may not be cv-qualified
error: no match for 'operator=' (operand types are 'StringSet' and 'std::string {aka std::basic_string<char>}')
error: passing 'const StringSet' as 'this' argument discards qualifiers [-fpermissive]
error: use of deleted function 'StringSet& StringSet::operator=(const StringSet&)'

我的任务使赋值操作符=超载，因此我无法在这里使用它。是否有另一种无需使用赋值运算符即可实现复制功能的方法-std::string中是否有任何内容可以让我们以这种方式更轻松地复制内容？

如果还有其他需要在此处添加的详细信息，请告诉我。

谢谢。

最佳答案

此代码的问题:

StringSet::StringSet(const StringSet& a)
{
    auto a2 = StringSet(currentSize);

    for (auto i=0; i < currentSize ; i++ )
    {
        a2[i] = a[i];
    }
}

就是说，即使它已编译，您也不会真正初始化this的成员……您正在初始化一些临时a2，这些临时operator[]在构造函数的末尾超出范围。您实际上想要:

StringSet::StringSet(const StringSet& a)
    : StringSet(a.arrSize)
{
    currentSize = a.currentSize;

    for (auto i=0; i < currentSize; i++ )
    {
        arr[i] = a.arr[i];
    }
}

同样，您的StringSet&返回std::string&，它应该返回std::。

另外，您应该避免像这样做一样将名称带入全局 namespace 。保持本地。编写ojit_code并不是负担。

关于c++ - C++-创建不带=动态字符串数组分配的Copy函数，我们在Stack Overflow上找到一个类似的问题：https://stackoverflow.com/questions/39711294/