Вектор выходит за диапазон

@Protss · Регистрация: 15.08.2016

Author24 — интернет-сервис помощи студентам

Здравствуйте.

Как сделать так чтобы вектор не выходил за диапазон?

C++

#include <iostream>
#include <vector>
using namespace std;
 
int main()
{
    int k;
 
    vector<int> ivec = { 2,3,4,5,6 };
 
 
    for (int k=0; k != ivec[ivec.size()]; ++k)
    {
        cout << (ivec[k] % 2 != 0) ? "Good luck" : "False";
 
        if (ivec[k] % 2 != 0)
            ivec[k]*=ivec[k];
 
        else ivec[k] =ivec[k];
    }
 
 
    for (auto c : ivec)
 
 
    {
        cout << c << endl;
    }
    system("pause");
 
 
 
    return 0;
}

@Hikari · 25.08.2016, 11:36

Сообщение от Protss

k != ivec[ivec.size()]

Какой смысл этого условия?

@John Prick · 25.08.2016, 11:38

Сообщение от Protss

Как сделать так чтобы вектор не выходил за диапазон?

Не писать подобного: ivec[ivec.size()].

@Ferrari F1 · 25.08.2016, 11:44

Protss, для прохода по вектору используй итераторы, ну или цикл for по диапазону

@Protss · 25.08.2016, 12:20 **[ТС]**

Понятно

, а я думал что вектор все равно выходит за диапазон k=5, но условие останавливает его. Но факт что он выходит)
А я тут нагородил, нужно ivec.size()

Добавлено через 9 минут
Скажите, а почему у меня выводится при k=0 010104, а при остальных k ( 9,16....)?

C++

#include <iostream>
#include <vector>
using namespace std;
 
int main()
{
    int k;
 
    vector<int> ivec = { 2,3,4,5,6 };
 
 
    for (int k=0; k != ivec.size(); ++k)
    {
        cout << (ivec[k] % 2 != 0) ? ivec[k]*=ivec[k] : ivec[k]=ivec[k];
 
    }
 
 
    for (auto c : ivec)
 
 
    {
        cout << c << endl;
    }
    system("pause");
 
 
 
    return 0;
}

Добавлено через 22 минуты

Кликните здесь для просмотра всего текста

с помощью итераторов, такой же вывод

C++

#include <iostream>
#include <vector>
using namespace std;
 
int main()
{
    int k;
 
    vector<int> ivec = { 2,3,4,5,6 };
 
 
    for (auto beg=ivec.begin(); beg != ivec.end(); ++beg)
    {
        cout << (*beg % 2 != 0) ? *beg*=*beg : *beg=*beg;
 
    }
 
 
    for (auto c : ivec)
 
 
    {
        cout << c << endl;
    }
    system("pause");
 
 
 
    return 0;

}

@John Prick · 25.08.2016, 13:10

Потому что скобочки надо уважать:

C++

1	cout << ((ivec[k] % 2 != 0) ? ivec[k]*=ivec[k] : ivec[k]=ivec[k]);

@keeperkira · 25.08.2016, 13:50

ivec[ivec.size()]

тут и происходит выход за диапазон. размер массива у тебя равен 5. значит запись имеет вид ivec[5];а это уже 6 элемент массива

C++

ivec[0] //1
ivec[1]//2
ivec[2]//3
ivec[3]//4
ivec[4]//5
ivec[5]// 6 элемен, ошибка!!!

решение ivec[ivec.size()-1]

@Hikari Хитрая блондиночка $) 1472 / 988 / 399 Регистрация: 21.12.2015 Сообщений: 3,785
	25.08.2016, 11:36	2
	Сообщение от Protss k != ivec[ivec.size()] Какой смысл этого условия? 1

@John Prick 2276 / 1767 / 741 Регистрация: 27.07.2012 Сообщений: 5,250
	25.08.2016, 11:38	3
	Сообщение от Protss Как сделать так чтобы вектор не выходил за диапазон? Не писать подобного: `ivec[ivec.size()]`. 1

@Ferrari F1 805 / 532 / 158 Регистрация: 27.01.2015 Сообщений: 3,017 Записей в блоге: 1
	25.08.2016, 11:44	4
	Protss, для прохода по вектору используй итераторы, ну или цикл for по диапазону 1

	25.08.2016, 13:50