0

したがって、基本的には、文字列を受け入れ、その文字列内の単語をユーザーが置き換えたいものに置き換えるフォーマット関数を作成したいと考えています。最初は、文字列のサイズを変更するとイテレータが無効になる可能性があることに気付くまで、非定義可能イテレータにいくつかの問題がありました。出力が入力と同じになり、例外がスローされなくなりました。何かアドバイス？？？

string& formatFn(string& s, string& oldWord, string& newWord)
{
    string word = "";
    for (auto iter1 = s.begin(); iter1 != s.end(); ++iter1)
    {       
            string tmpWord = "";                         
        if (!isblank(*iter1))       // Testing for whitespace
        {
            tmpWord += *iter1;
            if (tmpWord == oldWord)
            {
                string::iterator beg = iter1 - word.size();
                string::iterator end = iter1;
                auto sIter = s.erase(beg, end);                 // Get the position returned by erase
                auto i = sIter - s.begin();                     // Get an index 
                s = s.insert(s[i], newWord);
            }
        }
        if (isblank(*iter1))
        {
            tmpWord.clear();
        }
    }
    return s;
}
4

3 に答える 3

0

すでに文字列を使用している場合、すべてのメソッドを使用しないのはなぜですか?

for (auto it = text.find(o_text); it != string::npos; it = text.find(o_text)){
    text.replace(it, o_text.size(), n_text);
}
于 2013-09-23T20:49:33.393 に答える
0

あなたはそれを過度に複雑にしています:

std::string replace_all(std::string s, const std::string& sOld, const std::string& sNew)
{
    std::size_t p = s.find(sOld);
    while (p != std::string::npos)
    {
        s.replace(p, sOld.length(), sNew);
        p = s.find(sOld, p + sNew.length());
    }
    return s;
}

単語全体のみを置き換えたい場合 (現在の試みではできません):

#include <iostream>
#include <string>

bool test(const std::string& s, const std::string& sOld, std::size_t pos)
{
    return (pos == 0 || !::isalpha(s[pos - 1])) && (!::isalpha(s[pos + sOld.length()]) || pos + sOld.length() >= s.length());
}

std::size_t find_word(const std::string& s, const std::string& sOld, std::size_t pos)
{
    pos = s.find(sOld, pos);
    while (pos != std::string::npos && (!test(s, sOld, pos) && pos < s.length()))
    {
        pos++;
        pos = s.find(sOld, pos);
    }
    return pos;
}

std::string replace_all(std::string s, const std::string& sOld, const std::string& sNew)
{
    std::size_t p = find_word(s, sOld, 0);
    while (p != std::string::npos && p < s.length()) 
    {
        s.replace(p, sOld.length(), sNew);
        p = find_word(s, sOld, p + sNew.length()); 
    }
    return s;
}

int main() 
{
    std::string sOrig = "eat Heat eat beat sweat cheat eat";
    std::string sOld = "eat";
    std::string sNew = "ate";
    std::string sResult = replace_all(sOrig, sOld, sNew);
    std::cout << "Result:  " << sResult << std::endl;
    // Output:  "ate Heat ate beat sweat cheat ate"
    return 0;
}
于 2013-09-23T20:53:35.173 に答える
0 
string::iterator beg = iter1 - word.size();

word実際に何ができるかわかりません。を削除しようとしていますoldWordよね？次に、次のようになります。

string::iterator beg = iter1 - oldWord.size();

編集:これはコードの改良版です:

string formatFn(const string& s, const string& oldWord, const string& newWord) {
    string result = "";     // holds the string we want to return
    string word = "";       // while iterating over 's', holds the current word

    for (auto iter1 = s.begin(); iter1 != s.end(); ++iter1)  {       
        if (!isblank(*iter1))
            word += *iter1;
        else {    // if it is a whitespace, it must be the end of some word
            // if 'word' is not same as 'oldword', just append it
            // otherwise append 'newWord' instead

            if (word == oldWord)
                result += newWord;
            else
                result += word;
            result += *iter1;
            word.clear();    // reset 'word' to hold the next word in s
        }
    }

    // the end of the string might not end with a whitespace, so the last word
    // might be skipped if you don't make this test

    if (word == oldWord)
        result += newWord;
    else
        result += word;

    return result;
}
于 2013-09-23T20:32:40.667 に答える