这是我正在编写的将一串单词解析为一组单词的函数。

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函数将这些单词传递给输出迭代器。然后,这有利于直接用于各种目的,例如在一行中输出一个单词,或将它们添加到集合中,或将它们放在向量的末尾,等等。为此,它方便了方便包装的编码。更重要的是,这是一条一般原则,即不必不必要地将自己限制在给定的数据表示形式上。但另一方面,不要浪费希望的概括,因为最后可能会发现它没有被使用。

09-06 06:49