Question

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

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＆lt; __ gnu_cxx :: __ normal_iterator＆gt;＆gt;}'），指向我的矢量索引？

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

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

Answer 1

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

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

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

不是将迭代器传递给operator []，而应该使用星号来取消引用它们，以便获取它们指向的向量元素：

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

另外，请注意解除引用i + 1：在循环的最后一次迭代中i + 1将等于rend()（反向过去的最后一个元素迭代器）。试图通过这样的迭代器访问任何东西都会导致未定义的行为。

Answer 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;
    }

 }

在您的情况下使用＆＃34; * i＆＃34;代替＆＃34; Close1-＆gt;关闭[i]＆＃34;读取值

使用来自其他类的向量作为反向迭代器

2 个答案: