我希望对您有所帮助的一些观察(但请注意:以下代码未经测试)。
如果你的Passenger 需要名字、姓氏和目的地,我想有一个Passenger 构造函数带有三个字符串参数。
我想你的Read_from_file() 应该变成这样的
void read_from_file(list<Passenger> & flist, string const & filename)
{
string firstN, lastN, dest;
ifstream input(filename.c_str());
while (input >> firstN >> lastN >> dest)
{
flist.push_back(Passenger(firstN, lastN, dest)); // for C++98
// flist.emplace_back(fistN, lastN, dest); // for C++11/14
}
// input.close(); no need for close: file closed by destructor
}
建议:通过引用传递参数,并尽可能使用const。
同样,insert() 可以通过简单的方式进行调整(两种不同的模式,取决于您使用的是最新的 C++ 编译器还是 C++11 之前的编译器)
void insert(list<Passenger> & flist, string const & firstN,
string const & lastN, string const & dest)
{
flist.push_back(Passenger(firsN, lastN, dest)); // C++98 version
// flist.emplace_back(firsN, lastN, dest); // C++11/14 version
}
如何check_reservation() 不太明显。并且真正依赖于 C++98/C++11 问题。
我想你想要一个check_reservation(),它想找到一个Passenger,给出名字、姓氏和目的地。
在 C++11 中相当简单(假设您的 Passenger 包含一个 firsN、一个 lastN 和一个 dest 成员)
bool check_reservation(list<Passenger> const & flist, string const & firstN,
string const & lastN, string const & dest)
{
return flist.cend() != find_if(flist.cbegin(), flist.cend(),
[&](Passenger const & p)
(return (fistN == p.fistN)
&& (lastN == p.lastN)
&& (dest == p.dest);));
}
在 C++98 中,您没有 lambda 函数,因此更加棘手。您必须构造一个函子进行比较;像
struct person_eq : std::unary_function <Passenger, bool>
{
string const & f;
string const & l;
string const & d;
person_eq (string const & f0, string const & l0, string const & d0)
: f(f0), l(l0), d(d0)
{}
bool operator() (Passenger const & p) const
{ return (p.firstN == f) && (p.lastN == l) && (p.dest == d); }
};
你的check_reservation() 应该变成类似
bool check_reservation(list<Passenger> const & flist, string const & firstN,
string const & lastN, string const & dest)
{
return flist.cend() != find_if(flist.cbegin(), flist.cend(),
person_eq(fistN, lastN, dest));
}
我再说一遍:这段代码未经测试,所以要小心。
ps:对不起,我的英语不好。
编辑
抱歉:如果为您的Passenger 定义了operator==,那么您的check_reservation() 可以非常简单
bool check_reservation(list<Passenger> const & flist, string const & firstN,
string const & lastN, string const & dest)
{
return flist.cend() != find(flist.cbegin(), flist.cend(),
Passenger(firstN, lastN, dest));
}