2

我已从文件中读取数据，并希望将数据存储为对象向量。

vector <Thing*> thingVector;
for (int i = 0; i < 10; i++) {
// Read in contents of file
getline(fileName, v1, ',');
cout << v1 << endl;
getline(fileName, v1, ',');
cout << v2 << endl;
getline(fileName, v3, ',');
cout << v3 << endl;
getline(fileName, v4, '\n');
cout << v4 << endl << endl;
// Store
Thing* thingDetails = new Thing(v1, v2, v3, v4);
thingVector.push_back(thingDetails);
delete thingDetails;
}
thingFile.close();
cout << "Size of THING vector is " << thingVector.size() << endl; // Displays 10

cout << thingVector[0].getV1 << endl; // ERROR HERE

如何将每条记录存储在向量中，然后访问数据？

我也尝试这样做：thingVector.push_back(Thing(v1, v2, v3, v4));

当我像这样尝试时，for语句中没有最后一行和倒数第三行，但我无法访问数据，因此放弃了这种方法。

有什么建议么？

东西.H文件

#ifndef THING_H
#define THING_H

#include <string>

using namespace std;

class Thing {
public:

Thing(string v1, string v2, string v3, string v4);
string getV1();
string getV2();
string getV3();
string getV4();

private:
string v1;
string v2;
string v3;
string v4;
};

#endif

东西.CPP文件

#include "thing.h"

#include <string>

using namespace std;
Thing::Thing(string aV1, string aV2, string aV3, string aV4) {
v1 = aV1;
v2 = aV2;
v3 = aV3;
v4 = aV4;
}

string Thing::getV1(){
return v1;
}

string Thing::getV3(){
return v2;
}

string Thing::getV3){
return v3;
}

string Thing::getV4(){
return v4;
}
4

3 回答 3

3

您的问题是您正在存储指向 的指针Thing，但正在删除指针。所以向量最终充满了悬空指针。您可以通过简单地使用 的向量来避免所有这些麻烦Things：

vector <Thing> thingVector;
...
thingVector.push_back(Thing(v1,v2,v3,v4));

然后你可以像这样访问它：

std::string s = thingVector[0].getV1();

或者

cout << thingVector[0].getV1() << endl;

除非绝对必要，否则不应使用指针动态分配的对象，并且在您的代码示例中似乎没有理由这样做。如果这样做，请考虑使用智能指针来处理内存管理。

请注意，如果您选择了 a vectorofThing指针或智能指针，则必须使用->运算符调用每个元素的方法：

cout << thingVector[0]->getV1() << endl;
                  //   ^ here!

顺便说一句，你真的应该避免using namespace std;，特别是在头文件中。

于 2012-09-02T12:16:51.947 回答
2

您的代码有两个问题：

  • 您正在删除指向Thing您 push_back 的指针，并且
  • 您根本没有阅读v2（复制/粘贴错误；您正在将数据读入v1两次）。

虽然有一个指针向量是可以的，但这很不方便，因为你必须做很多手动工作来管理对象的内存。您可以在向量中使用unique_ptr<Thing>而不是，如下所示：Thing*

vector <unique_ptr<Thing> > thingVector;
于 2012-09-02T12:19:08.097 回答
1

你应该使用

vector <Thing> thingVector;
...
thingVector.push_back(*thingDetails);

这将传递对象而不是指针。

还

cout << thingVector[0].getV1() << endl; // Convert to a function call.
于 2012-09-02T12:18:48.330 回答