问题描述
|
我编写了一个读为“ 0”的类。但是我需要地图在课程之外是可编辑的。所以我的问题是如何返回
map
。
class ReadMap
{
string fileName;
public:
//constructors and destructor
ReadMap(){fileName=\"blank.txt\";}
ReadMap(string name){fileName=name;}
~ReadMap(){}
//Function to print out visible list
void show()
{
LineDatabase Entry;
int LineNumber=100;
string buffer;
ifstream myfile (fileName.c_str() );
while (myfile.good())
{
myfile >> LineNumber >> ws;
getline (myfile,buffer);
Entry.insert(pair<int,string>(LineNumber,buffer));
cout <<buffer << endl;
}
//return Entry;
}
};
解决方法
让show()的调用者传递要填充的地图的引用可能会更好,因为返回地图往往会产生高开销。像这样:
void show(LineDatabase& Entry) {
// do your map readin logic
return;
}
,您可以像返回其他内容一样使用return4ѭ关键字返回地图。如果您想知道为什么注释return
不起作用,那是因为您声明declare6ѭ返回void
。将void show()
更改为LineDatabase show()
。
另外,请尝试使变量和类型名称保持小写。典型的约定是使用大写的名称作为模板参数,因此读取时有点混乱。
,有4个选项。
最简单的选择是更改show的返回类型并取消注释您的返回,但是这将按值返回地图,该值将涉及一个副本,并且可能(取决于大小)效率很低(可能很危险,取决于LineDatabase的复制运算符)。
LineDatabase show()
{
LineDatabase Entry;
// .... ommited
return Entry;
}
第二个选项是按照user258808的建议进行操作,并创建一个新对象,然后通过指针返回它,这种方法的问题是,您的客户端必须在完成后知道在此指针上调用delete,否则您将创建泄漏。
第三个选项是将Entry作为ReadMap的字段,然后返回引用。这是我的个人喜好,因为它使客户负担最小,但是,它可能还要求您在每次新运行之前“重置”条目。
像这样
class ReadMap
{
string fileName;
LineDatabase Entry;
public:
//constructors and destructor
ReadMap(){fileName=\"blank.txt\";}
ReadMap(string name){fileName=name;}
~ReadMap(){}
//Function to print out visible list
LineDatabase& show()
{
int LineNumber=100;
string buffer;
ifstream myfile (fileName.c_str() );
while (myfile.good())
{
myfile >> LineNumber >> ws;
getline (myfile,buffer);
Entry.insert(pair<int,string>(LineNumber,buffer));
cout <<buffer << endl;
}
return Entry;
}
};
这样做的问题是,它会将您的内部状态暴露给修改,可以返回const引用,但是客户端不能修改Map。
最后,您可以按照bodes的建议进行操作。但是,这需要客户端传递地图以供您使用。
您的选择将取决于您希望客户执行多少工作以及需要和/或不需要在数据结构上施加哪些约束。