Question

我有class Student（studentOwner）和class Section。这是我的班级Student：

class Student {
  vector<Section*> enrolledSections;
public:
  vector<Section*> getEnrolledSections() const { return enrolledSections; }
}

所以，当我得到vector<Section*>并分配给另一个向量时，我会遇到错误。我正在使用Microsoft Visual Studio。

// first example: no error, if take size of vector
int a = studentOwner->getEnrolledSections().size();
// second example: error, when only take its vector and assign again
// Error: no suitable user-define conversion from "std::vector<error-type" ....
vector<Section*> enrolled = studentOwner->getEnrolledSections();
// third example: error when take this vector and assign to reference of same type
// Error: initial value of reference to non-const value must be lvalue
vector<Section*>& enrolled = studentOwner->getEnrolledSections();

第二个例子的完整错误是：

Error: no suitable user-define conversion from "std::vector<error-type", std::alocator<<error-type> *>> "to " std::vector<Section*, std::allocator<Section*>>" exists

在我的项目的许多课程中，我不能做第二行和第三行并收到同样的错误。我不能自己解释。请教我这一点。

谢谢：）

Answer 1

通常，如果在MSVC错误中看到error-type，则它是前向声明类型的结果，该类型未包含在该编译单元的时间内。例如，

// Course.h
class Student;

class Course {
     [...]
public:
     Student* getStudent();
}

// Course.cpp
#include "Course.h"

Student* Course::getStudent()
{
    return new Student("Name");  //< Whoops, we never included Student.h!
}

在评论中，您指出循环包含依赖项。正如@Daniel Castro所说，您应该在头文件中转发声明以避免循环包含，然后在.cpp文件中包含所需的头文件（如果您不熟悉，请注意上面的前向声明class Student;）。

顺便说一下，我还会注意到你的例子中的一些设计问题。返回std::vector<Section*>并不能说明谁拥有什么。如果我从函数中获取std::vector的值，那么惯例是我现在拥有向量及其内容。如果我拥有某些东西，那么我有责任删除它。在没有看到实际实现的情况下，大多数编码人员会惊讶地发现他们不应该删除向量的内容。我建议通过const&（例如，const vector<Section*>&）返回向量，这会阻止客户端代码操纵向量（因此客户端不会拥有它），或者使用{ {3}}管理Section个对象的共享所有权方案：

class Student {
    vector<shared_ptr<Section>> enrolledSections_;
public:
    vector<shared_ptr<Section>> getEnrolledSections() const { return enrolledSections_; }
}

现在很清楚谁拥有什么。比你要求的更多，但希望它有所帮助。

Answer 2

您需要将矢量作为参考返回，否则在返回时复制矢量。你的函数也是const，所以你也必须将向量作为const返回。

class Student 
{
  std::vector<Section*> enrolledSections;
public:
  const std::vector<Section*> &getEnrolledSections() const { return enrolledSections; }
}

现在你应该可以

了

const std::vector<Section*>& enrolled = studentOwner->getEnrolledSections();

C ++：不能将指针向量分配给另一个指针向量

2 个答案: