0

我写了一个函数来计算两组的并集。

我遇到了几个编译错误，我相信这部分是由于我如何制作StringUnion数组并声明它，但到目前为止我没有做任何事情。

这是我的头文件。

#ifndef StringSet_header
#define StringSet_header
#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)};

};

#endif

这是我对我的SetUnion功能的实现。

StringSet StringSet::setunion(const StringSet &Array2) const
{
    StringSet StringUnion = make_unique<string[]>(arrSize);

    if (currentSize > 0)
    {
        for (auto i=0; i < currentSize; i++)
        {
            auto s = arr[i];
            StringUnion.insert(s);
        }
        for (auto i=0; i < Array2.currentSize; i++)
        {
            auto s = Array2[i];
            if (StringUnion.find(s) == NOT_FOUND)
            {
                StringUnion.insert(s);
            }
        }
    }
    else    
    {
        auto result = StringSet();
        return result;          //return empty StringSet}
    }
}

错误：

|error: conversion from 'std::_MakeUniq<std::basic_string<char> []>::__array {aka std::unique_ptr<std::basic_string<char> []>}' to non-scalar type 'StringSet' requested|

error: passing 'const StringSet' as 'this' argument discards qualifiers [-fpermissive]

error: no matching function for call to 'StringSet::find(StringSet&)'

error: no matching function for call to 'StringSet::insert(StringSet&)'

按预期插入和查找工作，我能够在我的删除功能和其他一些功能中使用插入和查找功能，为什么我不能在这里使用它们？

4

2 回答 2

1

编译器提供的错误看起来很清楚。让我们检查一下。

  • 从请求转换std::make_unique ...为非标量类型StringSet

这是因为函数的定义std::make_unique，它返回一个std::unique_ptr<T>. 但是您正在尝试将其分配给 type 的值StringSet。没有用于StringSet从 a创建 a 的构造函数或运算符std::unique_ptr，因此编译器抱怨他不能这样做。

  • 错误：没有匹配的调用函数'StringSet::find(StringSet&)'

你的类StringSet有一个operator[]返回一个对StringSetso的引用auto s = Array2[i];的类型StringSet。但是您的功能find并insert要求std::string. 由于没有构造函数可以提供从StringSettostd::string的隐式转换，因此编译器会抱怨。

于 2016-09-29T07:14:36.830 回答
1

在你的行中

StringSet StringUnion = make_unique<string[]>(arrSize);

RHS 使用 c++14构造，它接受一个std::size_t，并返回一个std::unique_ptr<std::string>内部指向一个数组。

然而，LHS 是一个StringSet对象。

您没有定义采用这种类型的构造函数，所以这是一个问题。

查看您的代码，StringSet确实有一个std::unique_ptr<std::string>成员，因此您可以添加一个 ctor 获取这样的对象，并从中初始化成员。但是，目前还不清楚这样一个演员有什么好处，因为你已经有了一个演员

StringSet(int capacity);

这基本上已经做了同样的事情。

正如 Leon 所写，你应该只使用这个而不是你拥有的那一行

StringSet StringUnion(arrSize);
于 2016-09-29T07:06:01.343 回答