Использование функции от каждого объекта в векторе

Question

Im кодирует машину загадки и имеет проблемы с использованием элементов в моем векторе, получая только ошибку сегментации как ошибку. Мне нужно использовать функцию от каждого из роторов в моем векторе на одном символе, преобразовывая символ, когда итератор движется вперед, один раз вперед по списку, один раз назад. Я прокомментировал другие преобразования, чтобы изолировать проблему.

Machine.cpp

#include "Machine.h" 

using namespace std; 

Machine::Machine(const vector<Rotor>& rots, const Plugboard & pb) 
    : rotors(rots), plugboard(pb) {} 

// give c's alphabet index 
int Machine::getPosition(char c) { 
    if (c >= 'A' && c <= 'Z') { 
     return c - 'A'; 
    } else { 
     cout << "not an accepted character"; 
     return -1; 
    } 
} 

// give letter at index i in alphabet 
char Machine::atPosition(int i) { 
    assert(i >= 0 && i <= 25); 
    return "ABCDEFGHIJKLMNOPQRSTUVWXYZ" [i]; 
} 

char Machine::encode(char c) { 
    assert(c >= 'A' && c <= 'Z'); 
    // plugboard 
    // c = plugboard.getMatch(c); 
    // forward pass through rotors 
    for (std::vector<Rotor>::iterator it = rotors.begin(); it != rotors.end(); ++it) { 
     c = it->process(c); 
    } 
    // reflector 
    // c = Reflector::reflect(c); 
    // backwards pass through rotors 
    for (std::vector<Rotor>::iterator it = rotors.end(); it != rotors.begin(); --it) { 
     c = it->processInverse(c); 
    } 
    return c; 

} 

соответствующий вызов часть моей главной функции:

Rotor rot1(argv[1]); 
    Plugboard pb1(argv[2]); 
    vector<Rotor> rotors(1, rot1); 
    Machine machine(rotors, pb1); 
    machine.encode('A'); 

Answer 1

Вы итерация назад неправильно. Итератор, на который указывает rotors.end(), находится за концом вектора, это не действительный объект ... так что вызов функции на нем является неопределенным поведением.

Если вы хотите перебрать в обратном направлении, вы должны использовать обратные итераторы:

for (std::vector<Rotor>::reverse_iterator it = rotors.rbegin(); 
    it != rotors.rend(); ++it) { 
    c = it->processInverse(c); 
}