使用其他类的向量作为反向迭代器答案

【问题标题】：Using vector from other class as reverse iterator使用其他类的向量作为反向迭代器
【发布时间】：2018-04-20 06:36:54
【问题描述】：

对不起，我在这里如履薄冰。我以前没有使用过反向迭代器，就像你在我的代码中看到的那样，我还想使用另一个类中的向量作为迭代器对象：

double indicators::sRSItemp(input* Close1, int StartDay) {
  int n = 14;
  double rs;
  double rsi;
  double tmpavl;
  double tmpavg;


if (!RSI.empty()) {
for ( vector<double>::reverse_iterator i = Close1->Close.rbegin(); i != Close1->Close.rend(); ++i ) {

    if (Close1->Close[i] < Close1->Close[(i + 1)]){
        tmpavl = ((AVL[0] * 13 ) + (Close1->Close[(i +1)] - Close1->Close[i] ) / n);
        cout << "AVLtmp " << AVL[0] << endl; 
        cout << "tmpavl " << tmpavl << endl;
        AVL.insert(AVL.begin(), tmpavl);
        cout << "AVL is " << AVL[0] << endl;

        tmpavg = ((AVG[0] * 13 ) / n );
        AVG.insert(AVG.begin(), tmpavg);
        // cout << "AVG is " << AVG[i] << endl;
        }

        else if  (Close1->Close[i] > Close1->Close[(i + 1)]) { 
            tmpavg = ((AVG[0] * 13 ) + (Close1->Close[i] - Close1->Close[(i +1)]) / n );
            AVG.insert(AVG.begin(), tmpavg);
            // cout << "AVG is " << AVG[i] << endl;

            tmpavl = ((AVL[0] * 13 ) / n );
            AVL.insert(AVL.begin(), tmpavl); 
            // cout << "AVL is " << AVL[i] << endl;

            }

            rs = AVG[0] / AVL[0];
            rsi = (100.0 - (100.0 / (1.0 + rs)));
            RSI.insert(RSI.begin(), rsi);

            }
    }
return 0;
}

但是当我编译这段代码时，我得到了几个这样的错误：错误：'operator[]' 不匹配（操作数类型为'std::vector' 和'std::vector::reverse_iterator {aka std::reverse_iterator<:__normal_iterator> >}'），指向我的矢量索引？？

if (Close1->Close[i] < Close1->Close[(i + 1)]){

就像我说的，这对我来说是一个新领域，我猜错误在于迭代器的声明？当我在其他代码中迭代相同的向量（从前到后）时，没有问题。非常感谢您的帮助！

【问题讨论】：

input 是什么类型？您能否发布最少的代码，我们可以用来重现错误？
您取消引用反向迭代器的方式与执行迭代器的方式相同。所以你的比较应该是if (*i < *(i+1))。

标签： c++ c++11 stdvector reverse-iterator

【解决方案1】：

std::vector 的方括号运算符接受索引，而不是迭代器。

在这里您尝试使用迭代器作为索引：

if (Close1->Close[i] < Close1->Close[(i + 1)]) {

不要将迭代器传递给运算符 []，您应该只使用星号来取消引用它们，以便获取它们指向的向量元素：

if (*i < *(i + 1)) {

此外，请注意取消引用 i + 1：在循环的最后一次迭代中，i + 1 将等于 rend()（反向过去最后一个元素迭代器）。尝试通过此类迭代器访问任何内容将导致未定义的行为。

【讨论】：

非常感谢！那行得通！只是关于取消引用最后一个元素的后续问题。有没有办法可以在该元素之前停止迭代？
@PushT 是的，您应该可以在 for 的条件中添加 -1：i != Close1->Close.rend() - 1。

【解决方案2】：

要查看你做错了什么，请注意这两个是等价的

int main(){
    vector<int> myVec {{1,2,3,4}};
    //read then print each value in vector
    for(vector<int>::iterator i=myVec.begin();i!=myVec.end(); ++i){
    //here i is an iterator not an index
         int val = *i; //get value in current position within vector
         cout<<val<<endl;
    }

    for(int i=0; i!=myVec.size(); ++i){
    // here i is an index
         int val = myVec[i];//get value in current position within vector
         cout<<val<<endl;
    }

 }

在您的情况下，使用“*i”而不是“Close1->Close[i]”来读取值

【讨论】：