Вектор и для не дает результатов

Question

Я работаю над простой игрой (камень, бумага, ножницы), и у меня возникла эта проблема, когда я пытаюсь заполнить вектор с использованием элементов другого вектора в качестве условий. Код может быть более простым для понимания:

#include "stdafx.h"
#include "../../Library/std_lib_facilities.h"

// With the fibonacci series I can generate a secret sequence of number
int fib(int n){
    if (1 == n || 2 == n) {
        return 1;
    }
    else {
        return fib(n - 1) + fib(n - 2);
    }
}

// It shows the list of element in a given vector
void showVector(vector<int>myVector, string nameVector) {
    cout << "\n\n";
    cout << nameVector << " Debug: | ";
    for (int i = 1; i < 10; i++) {
        cout << myVector[i] << " | ";
    }
    cout << "\n\n";
}

// String variant
void showVector(vector<string>myVector, string nameVector) {
    cout << "\n\n";
    cout << nameVector << " Debug: | ";
    for (int i = 1; i < 10; i++) {
        cout << myVector[i] << " | ";
    }
    cout << "\n\n";
}

// Generate a sequence of number in a vector
vector<int> generateCode(int input) {

    vector<int>myVector;

    for (int i = 1; i <= 10; i++) {
        myVector.push_back(fib(i + input) % 3);
    }

    return myVector;
}

int main()
{
    // Secret sequence of moves, based on the value (0, 1 or 2) i will show Rock, Paper or Scissor
    vector<int>fibSeries;
    vector<string>movSeries;

    // Initialization and settings
    int input = 0;

    cout << "Digit an integer to start: ";
    while (cin >> input) {

        // Check the number to make sure is a valid one (i will implement a check later) and generate the secret code
        fibSeries = generateCode(input);

        // For each 0 i'll put Rock in the vector, same for 1 ( in this case paper ) and 2 ( scissor )
        for (int i = 0; i <= fibSeries.size(); i++) {
            if (fibSeries[i] == 0){
                movSeries.push_back("Rock");
            }
            else if (fibSeries[i] == 1) {
                movSeries.push_back("Paper");
            }
            else if (fibSeries[i] == 2) {
                movSeries.push_back("Scissor");
            }
            else {
                movSeries.push_back("Rock");
            }
        }

        // Shows the vector graphically, for debug.
        showVector(fibSeries, "fibisSeries");
        showVector(movSeries, "movSeries");

        // So for a combination of      1 - 2 - 0 - 1
        // The result should be:        Paper - Scissor - Rock - Paper

    }

    return 0;
}

После того, как я выполню код, я получаю (Прервать). Я не понимаю, я новичок в C++, поэтому, пожалуйста, прости меня, если это глупая ошибка. И большая часть кода сложна, потому что у меня есть правила, которым я должен следовать, поэтому, например, если я не изучал что-то, я не могу использовать это здесь. Я просто хочу знать, почему код не хочет, чтобы я использовал for с вектором!

Answer 1

for (int i = 0; i <= fibSeries.size(); i++)

Как упоминает @Yola, вы повторяете на один элемент больше, чем имеете:

если у вас есть вектор std::vector<char> v = {'a', 'b', 'c'};элементы могут быть доступны как:

v[0] (-> 'a')
v[1] (-> 'b')
v[2] (-> 'c')

Как правило, ваш максимальный индекс будет length - 1,

for (int i = 0; i <= fibSeries.size(); i++)

повторяется от 0 до меньшего или равного size (); то есть 0, 1, 2, 3 (4 элемента, в 3 элементах вектора).

Правильный код должен быть:

for (int i = 0; i < fibSeries.size(); i++)
                 ^^^

Answer 2

Итак, как уже было сказано, ваша проблема в том, что вы должны всегда использовать < size потому что индексирование основано на 0 <= size прочтет конец выделенного пространства.

---

Но пара комментариев, если можно, C++11 представил новый стиль на основе диапазона for -loops, которые избавят вас от необходимости писать условия и выражения итерации вообще:

map<int, string> movSeries = { { 0, "Rock"s }, { 1, "Paper"s }, { 2, "Scissors"s } };
vector<int> fibSeries(10);
auto input = 0;

while(cin >> input) {
    for(auto& i : fibSeries) {
        i = fib(input++) % 3;
    }

    cout << "\n\nDebug | ";
    copy(cbegin(fibSeries), cend(fibSeries), ostream_iterator<int>(cout, " | "));
    cout << "\n\n\n\nDebug | ";

    for(const auto i : fibSeries) {
        cout << movSeries[i] << " | ";
    }
    cout << endl << endl;
}