1

我有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

在我的项目的许多课程中，我无法执行第二行和第三行并收到相同的错误。我无法自己解释。请教我这个时候。

谢谢 ：）

4

2 回答 2

3

通常，如果您error-type在 MSVC 错误中看到，这是由于未及时包含在该编译单元中的前向声明类型的结果。例如，

// 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*>&）返回向量，以防止客户端代码操纵向量（因此客户端不拥有它），或者使用std::shared_ptr来管理Section对象的共享所有权方案：

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

现在很清楚谁拥有什么。超出您的要求，但希望它有所帮助。

于 2012-11-22T04:49:58.417 回答
0

您需要将向量作为参考返回，否则向量会在返回时被复制。此外，您的函数是 const，因此您也必须将向量返回为 const。

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

现在你应该可以 

const std::vector<Section*>& enrolled = studentOwner->getEnrolledSections();
于 2012-11-22T03:53:00.017 回答