如何正确使用这个迭代器？

Question

我正在尝试使用容器来存储字符串 s 的迭代器，但是当我运行它时出现了段错误。正如您将在下文中看到的，错误似乎来自 char temp = **itbegin;，这可能意味着我无法通过先前的迭代器分配任何值。

这是为什么？我是不是用错了迭代器？如何正确使用迭代器？

#include <iostream>
#include <vector>
using namespace std;
string reverseVowels(string s);

int main()
{
    string s ="hello";
    cout << reverseVowels(s);
}
string reverseVowels(string s) {
    string::iterator iter = s.begin();
    string::iterator iterend = s.end();
    vector<string::iterator> iteratorbox;
    for(;iter != iterend ; ++iter){
        if((*iter) == 'a' &&
           (*iter) == 'e' &&
           (*iter) == 'i' &&
           (*iter) == 'o' &&
           (*iter) == 'u'){
               iteratorbox.push_back(iter);
           }

    }
    auto itbegin = iteratorbox.begin();
    auto itend =   iteratorbox.end() ;
    --itend;
    //for(;itbegin < itend ; ++itbegin, --itend)
    {
        char temp = **itbegin;
    //    *(*itbegin) = *(*itend); 
    //    *(*itend) = temp;
    }
    return s;
}

Answer 1

您的问题来自于您在 iteratorbox 向量中插入迭代器的条件。

您使用了 && 运算符，这意味着字符串中的每个字母必须同时 等于所有元音 。这意味着不会插入任何迭代器，然后您将尝试取消引用向量的 begin() 迭代器，它恰好是它的尾后迭代器。这会导致未定义的行为，在您的情况下表现为崩溃。

您可能打算使用

((*iter) == 'a' ||
(*iter) == 'e' ||
(*iter) == 'i' ||
(*iter) == 'o' ||
(*iter) == 'u')

作为条件。

Answer 2

@rems4e 的回答非常好，但我发现如果将元音字母放入数组中，这样的代码更易于阅读且不易出错

char const vowels[] = { 'a', 'e', 'i', 'o', 'u' };

以便您可以将 reverseVowels 中的匹配逻辑封装到标准算法中 any_of

if (std::is_any_of(std::begin(vowels), std::end(vowels), [](auto const& v) {
   return *iter == v;
}) {
    iteratorbox.push_back(iter);
}

这避免了很多重复测试（如果您使用不同的字母表（例如德语），可能会很快失去同步）

如何正确使用这个迭代器？

How to use this iterator properly?

c++

containers

stl

c++11