这是我正在编写的将一串单词解析为一组单词的函数。
std::set<std::string> parseStringToWords(string rawWord)
{
getline(cin,rawWord);
std::set<std::string> myset;
istringstream iss(rawWord);
string word;
while(iss >> word) {
myset.insert(word);
}
}
我很确定到目前为止我已经做对了,但是我不确定在while循环中该做什么。有什么提示吗?
最佳答案
这是一些代码,希望可以引导您正确使用:
#include <iostream>
#include <string> // std::string
#include <set> // std::set
#include <sstream> // std::istringstream
namespace my {
using std::cin;
using std::cout;
using std::istringstream;
using std::set;
using std::string;
auto set_of_words( const string& s )
-> set<string>
{
set<string> result;
istringstream iss( s );
string word;
while( iss >> word )
{
result.insert( word );
}
return result;
}
auto getline( const string& prompt )
-> string
{
string result;
cout << prompt;
getline( cin, result );
return result;
}
} // namespace my
auto main() -> int
{
using namespace my;
using namespace std;
const set<string> words = set_of_words( getline( "A line of words, please: ") );
cout << "You entered these unique_words:" << endl;
for( const string word : words )
{
cout << "* " << word << endl;
}
}
此代码的主要问题是它不检查或处理故障。在专业工作中,许多代码通常与故障检查和处理有关。特别是
my::getline
函数不应仅在输入失败时静默返回结果。另一个问题是由于缺乏抽象,因此缺乏可重用性。正如我在对该问题的评论中已经提到的那样,对于经验丰富的程序员而言,自然的方法是让split-into-words函数将这些单词传递给输出迭代器。然后,这有利于直接用于各种目的,例如在一行中输出一个单词,或将它们添加到集合中,或将它们放在向量的末尾,等等。为此,它方便了方便包装的编码。更重要的是,这是一条一般原则,即不必不必要地将自己限制在给定的数据表示形式上。但另一方面,不要浪费希望的概括,因为最后可能会发现它没有被使用。